Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Currently it is not possible to add the 'same link' but with different query parameters on the same level to admin_menu.
For example if you combine this with admin_menu_sources. You can add 2 links in the menu for example
-civicrm/contact/add?reset=1&ct=Individual
-civicrm/contact/add?reset=1&ct=Organisation
If you choose this menu as admin_menu, you only get one of the links. The reason is inside the admin_menu_links_menu($tree) function. (That's why i file this issue here instead of in admin_menu_source).
We use only the href element as key inside the generated tree, if we could use a combination of href and optional query's as identifier this could be possible.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | admin_menu-allow_same_path-2416543-6.patch | 1.66 KB | skylord |
| #3 | admin-menu-samelink.patch | 1.7 KB | mallezie |
| #1 | admin-menu-samelink.patch | 1.72 KB | mallezie |
Comments
Comment #1
mallezieThis could be a start patch to use the query parameters as well as identifier.
Comment #3
mallezieYou shalle not maually edit patches ;-)
Comment #4
truls1502I am sorry for no reply until now.
There are many issues regarding this module admin_menu which is a bit difficult for us to follow up since some of the issues might be already outdated, or is already fixed by the module or any other modules or itself core which means that the problem might no longer need to be fixed.
We can see that the issue has been created for a few years ago, I hope it is okay for you that I am postponing the issue, and give you around two weeks. If you still face the problem, could you tell us the step by step when until you get the error message or what is frustrated you, and a list of modules you are using related to admin_menu and a screenshot that might help us? So it makes us easier to reproduce your issue.
However, after two weeks with no feedback - we will close this issue. So in case, you noticed it after the issue is closed, do not hesitate to reopen it like and fill information which is mentioned above.
So before giving us a feedback, do you mind to test it again with our latest 7.x-3.x-dev?
Thank you for understanding! :)
Comment #5
truls1502This issue has been automatically marked as closed because it has not had recent activity after the last post.
However, if you or someone is still facing the same issue as described to the issue, could you please to re-open the issue by changing the status of the issue, and add an explanation with more details which can help us to reproduce your situation.
Again, thank you for your contributions! :)
Comment #6
skylord CreditAttribution: skylord commentedThe issue is still present. Just try to add multiple items with same path but different query (i.e. views with different prepopulated exposed filters). Rerolled patch from #1.