Bug with removing Top Level Menus

Communities ContentBox CMS
#1

Found a bug in AdminMenuService

The following code needs to be applied to make it work

AdminMenuService function removeTopMenu(required topMenu){

for(var x=1; x lte arrayLen(menu); x++ ){

if( menu[x].name eq topMenuMap[arguments.topMenu].name ){

arrayDeleteAt( menu, x );

structDelete(topMenuMap, arguments.topMenu);

break;

}

}

// return

return this;

}

#2

Found a bug in AdminMenuService

The following code needs to be applied to make it work

AdminMenuService function removeTopMenu(required topMenu){

for(var x=1; x lte arrayLen(menu); x++ ){

if( menu[x].name eq topMenuMap[arguments.topMenu].name ){

arrayDeleteAt( menu, x );

structDelete(topMenuMap, arguments.topMenu);

break;

}

}

// return

return this;

}

#3

fixed

signature0.jpg

Luis F. Majano
CEO
Ortus Solutions, Corp
www.ortussolutions.com

ColdBox Platform: http://www.coldbox.org
Linked In: http://www.linkedin.com/pub/3/731/483
Blog: http://www.luismajano.com
IECFUG Manager: http://www.iecfug.com

Social: twitter.com/lmajano facebook.com/lmajano

#4

I see this has not been fixed yet either :frowning:

#5

You are more than welcome to submit a pull request as well.

Luis Majano
CEO
Ortus Solutions, Corp
Toll Free/Fax: 1-888-557-8057
Direct: 909-248-3408
www.ortussolutions.com
Twitter: @lmajano, @ortussolutions