Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I am creating a 'useful-links' menu for my site. When I navigate to:
'admin/build/menu-customize/menu-useful-links/add'
I get the page for:
'admin/build/menu-customize/menu-useful-links/container-add'
I can add 'containers' but I can't add 'items'. When I disable the module the 'add-item' page returns and I can add items. When I added an item underneath a container (with the module still disabled) it worked perfectly and displayed in a block; but this would mean every time I wanted to add an item I would have to disable the module.
Comments
Comment #1
ajzz CreditAttribution: ajzz commentedSame problem after Jan 23rd upgrade! This is indeed critical bug and a problem with the code (changed from UI).
Comment #2
Roulion CreditAttribution: Roulion commentedSame here
Comment #3
mikeker CreditAttribution: mikeker commentedLooks like a logic error in
_menu_item_container_form_menu_edit_item_alter()
. As a quick fix you can changemenu_item_container.admin.inc
at line 88 fromto
I'll roll a proper patch later unless someone else beats me to it.
Comment #4
TonyBy CreditAttribution: TonyBy commentedThank you for your work on the new patch. I'm sorry to say that it has introduced a new problem:
When I add a menu item under a container item (which I could not do before the patch) the menu item is not accessible from the 'useful-links' menu in my block. A solid right arrow to the left of the container indicates that there are other items 'inside' the container but they cannot be accessed. When I disable the module I can access these items.
Comment #5
JohnAlbinPlease open a separate support issue for that problem along with details about how that module provides that block and how it is configured. A normal core menu block should be working fine. Also, thanks for opening up this issue. :-)
Crap. I can see I was making sure the "edit item" page worked, but I didn't test the "add item" page when I committed that code. :-p
@mikeker your patch works great, but you don't need all those parenthesis.
Fixed. http://drupal.org/cvs?commit=495424
Comment #6
mikeker CreditAttribution: mikeker commentedYeah, I'm old school... *smirk*
I think it helps with readability, but perhaps that's because my editor does automatic parenthesis matching/pairing. To each his own, I suppose!
Thanks again for a great utility module.