CMS MADE SIMPLE FORGE

Menu Manager

 

[#8274] cms_breadcrumbs return an empty string if you set show_all option to 1

avatar
Created By: Leeroy Brun (leeroybrun)
Date Submitted: Fri Aug 17 05:46:48 -0400 2012

Assigned To:
Version: v1.1
CMSMS Version: 1.11.1
Severity: None
Resolution: Fixed
State: Closed
Summary:
cms_breadcrumbs return an empty string if you set show_all option to 1
Detailed Description:
If you set the show_all='1' option to the breadcrumbs action of the MenuManager
module, it returns an empty string.

Examble : {cms_breadcrumbs show_all='1'}

The bug is located in the "modules/MenuManager/action.breadcrumbs.php" file.

It's because of this condition :
if( $content->ShowInMenu() && $content->Active() && $showall == 0 )

If show_all is set to something else than 0, it will never return something...
:-)

I changed it to :
if( $content->ShowInMenu() && $content->Active() || $showall == 1 &&
$content->Active() )

And now it works fine ! :-)


History

Comments
avatar
Date: 2012-08-17 09:47
Posted By: Robert Campbell (calguy1000)

Yes there is a bug... yes it is in that line, but this 'fix' is incorrect.
      
avatar
Date: 2012-08-18 18:36
Posted By: Leeroy Brun (leeroybrun)

Ok sorry... I just wanted to help... :-)
      
avatar
Date: 2012-11-30 13:07
Posted By: Rolf (rolf1)

Fixed and shipped in previous release
      
Updates

Updated: 2012-11-30 13:07
state: Open => Closed

Updated: 2012-08-17 09:47
resolution_id: => 7