Feeling Lucky
?
[#2559] - Fixing notice in mod_breadcrumbs with multi-lang
- Closed
- 14 Nov 2014
- Medium
- Build: master
- # 2559
- Diff
- Hackwar:mod_breadcrumbs
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This fixes a notice that is thrown in mod_breadcrumbs when we have a multi-language site without a default menu item for the language "All".
brunobatista
- comment
- 19 Nov 2013
brianteeman
- comment
- 27 Jul 2014
Closed as per the comment on the tracker
brianteeman
- comment
- 22 Aug 2014
reopened as requested
| Build | ⇒ | . |
| Status | New | ⇒ | Pending |
| Category | ⇒ | Multilanguage |
| Category | Multilanguage | ⇒ | Modules Multilanguage |
Hackwar
- comment
- 14 Nov 2014
Since no one wanted to test or review this in one year and since we have merge conflicts here, I'm closing this PR. Maybe we can fix this another time.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 2014-07-27 10:04:56 | ⇒ | 2014-11-14 09:49:27 |
infograf768
- comment
- 14 Nov 2014
This has been solved already
we have now
// Look for the home menu
if (JLanguageMultilang::isEnabled())
{
$home = $menu->getDefault($lang->getTag());
}
else
{
$home = $menu->getDefault();
}
Tracker item here: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=32754&start=0