[#4994] - [imp] Multilanguage: Breadcrumbs does not display submenus if the default Home menu item set to All languages is deleted
- Closed
- 5 Nov 2014
- Medium
- Build: staging
- # 4994
- Diff
- infograf768:multilangpathway
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Test:
Install a basic multilingual site from scratch.
See: #4988
Create a submenu in the mainmenu of one of the languages.
Using PhpMyadmin, delete in the _menu table the row mainmenu with Home set to Language All, i.e. *
Display the site and the submenu
The breadcrumbs will not show the right items
you will also get an error:
PHP Notice: Trying to get property of non-object in ROOT/modules/mod_breadcrumbs/helper.php on line 51
Patch and test again.
(Don't forget to kill that test site as it is not normal to delete the mainmenu home menu item set to All languages in a multilingual site
jissues-bot
- | Labels |
Added:
?
|
||
The test works as described. However, when I click on "Joomla" in the "Popular Tags" module - I have installed "Default English (GB) Sample Data" - I get several notices, like this one:
Notice: Trying to get property of non-object in C:\xampp\htdocs\joomla33_test\modules\mod_menu\helper.php on line 40
moving to RTC based on tests by @fontanil and @waader
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4994.
| Category | ⇒ | Multilanguage |
| Status | Pending | ⇒ | Ready to Commit |
However, when I click on "Joomla" in the "Popular Tags" module - I have installed "Default English (GB) Sample Data" - I get several notices, like this one:
Notice: Trying to get property of non-object in C:\xampp\htdocs\joomla33_test\modules\mod_menu\helper.php on line 40
@waader Do you get that error also without the patch applied? Or only when it is applied? Sounds like a different issue to me.
That was meant as a remark. To be more precise: I get these notices with or without applying the patch under the above testing assumption - deleting that row in the _menu table.
| Labels |
Added:
?
|
||
Merged into staging, thanks!
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-11-05 20:25:49 |
@test
PR works as intended
Thanks