? Pending
Referenced as Duplicate of: # 8011

User tests: Successful: Unsuccessful:

avatar teeuwis
teeuwis
9 Sep 2015

In the breadcrumb module a menu item is not shown when it lacks a link (like a menu heading)
Fixes issue #7838 in the issue tracker

avatar teeuwis teeuwis - open - 9 Sep 2015
avatar teeuwis teeuwis - change - 9 Sep 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2015
Labels Added: ?
avatar Bakual Bakual - change - 9 Sep 2015
Title
Update default.php - fixing invisible menu heading
Breadcrumbs - fixing invisible menu heading
avatar Bakual
Bakual - comment - 9 Sep 2015

Copying original issue description:

Steps to reproduce the issue

Create a menu
Add a menu item of menu item type "Menu Heading"
Add a submenu item of menu item type "Individual Article"

Create a breadcrumb module for this menu and navigate to the submenu item.

Expected result

The breadcrumb module should show
Home > Menu Item > Sub Menu Item

Actual result

The breadcrumb module shows
Home > Menu Item

System information (as much as possible)

Not relevant

avatar justinherrin
justinherrin - comment - 9 Sep 2015

Looks like the issue came from the recent #7613 PR which re-worked the breadcrumbs to include microdata.

@test tested and now the Menu Heading type shows back up in the breadcrumb trail as expected.

avatar zero-24 zero-24 - alter_testresult - 9 Sep 2015 - justinherrin: Tested successfully
avatar zero-24 zero-24 - change - 9 Sep 2015
Easy No Yes
avatar zero-24 zero-24 - change - 9 Sep 2015
Category Modules
avatar wilsonge
wilsonge - comment - 9 Sep 2015

Merging on reivew with the test

avatar wilsonge wilsonge - close - 9 Sep 2015
avatar wilsonge wilsonge - reference | a6b7e05 - 9 Sep 15
avatar wilsonge wilsonge - merge - 9 Sep 2015
avatar wilsonge wilsonge - close - 9 Sep 2015
avatar wilsonge wilsonge - change - 9 Sep 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-09-09 19:40:42
Closed_By wilsonge
avatar wilsonge wilsonge - change - 9 Sep 2015
Milestone Added:
avatar teeuwis teeuwis - head_ref_deleted - 9 Sep 2015
avatar zero-24 zero-24 - change - 28 Oct 2015
Milestone
avatar zero-24 zero-24 - change - 28 Oct 2015
Milestone Added:
avatar zero-24 zero-24 - change - 28 Oct 2015
Milestone Added:
avatar zero-24 zero-24 - change - 28 Oct 2015
Milestone
avatar uudruid74
uudruid74 - comment - 11 Jan 2016

I manually fixed this once before. Just updated to 3.4.8 and its still there? How long until this gets pushed to stable?

avatar rdeutz
rdeutz - comment - 11 Jan 2016

It is merged for 3.5.0 as you can see from the milestone, meanwhile you can copy your fixed version to the template and create an override

avatar uudruid74
uudruid74 - comment - 11 Jan 2016

I know how to fix it as I've done it before. My question was what makes this 3.5 and not a bugfix that should have made it into 3.4.x a couple revisions ago. When I update, bugs should be fixed, not re-added! Nor should I have to go through a complicated overlay procedure to stop new releases from re adding bugs that were fixed months ago!

This is clearly not a new feature nor anything that would break compatibility with existing modules or cause new issues. It's a typo, exactly the sort of thing that new releases are supposed to fix. Bug has been here ... 6 months?

Sure. It's my fault for not using an overlay to fix it. That's it!

avatar rdeutz
rdeutz - comment - 11 Jan 2016

can't say why it wasn't merged for 3.4.5 from the timeline it should have be merged/included, after that we had only security releases and we decided to merge only the security fixes

Add a Comment

Login with GitHub to post a comment