Running the latest 7.x-dev of Commerce with numerous child-modules. Enabled admin_menu and got tons of 'duplicate' links in the menu: http://i.imgur.com/YhIaT.png

This is happening because the "Manage Fields" and "Manage Display" links are being added by the core fields_ui module, which sets their 'type' to MENU_LOCAL_TASK, resulting in these links showing up. These links have to do with items in the commerce_line_item_ui, commerce_customer_ui, and commerce_product_ui modules. All three modules implement hook_menu_alter() to set the 'context' property for their respective links, and the attached patch also changes the 'type' to MENU_CALLBACK, which fixes the issue.

Patch attached. Please advise if there's a reason not to do this.

(Menu registry needs to be cleared after applying the patch.)

CommentFileSizeAuthor
menu_type_conflict.patch2.81 KByuriy.babenko

Comments

Status: Needs review » Needs work

The last submitted patch, menu_type_conflict.patch, failed testing.

rszrama’s picture

rszrama commented
Category: bug » support
Status: Needs work » Postponed (maintainer needs more info)

Hmm, I always use Drupal Commerce with Admin Menu and never get links showing up like this. What version of Admin Menu are you on? And can you duplicate this if you install using the Commerce Kickstart installation profile? It may have something to do with the installation process itself, as I have noticed differences before depending on whether or not Admin Menu was installed before or after the Commerce modules.

n8tron’s picture

n8tron
he/him
Primary language English
Location Indiana
commented

Not sure if this is my issue, but now when I go to "Store" in admin menu I only see "Configuration". I have to use the URI to navigate commerce. Previous something blew out all my router menu items and I had to totally rebuild all menu items. I couldn't even get to /user.

I'm using the 7.x-1.0 release tho.

Subscribe.

rszrama’s picture

rszrama commented
Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)