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.1. Add one Primary menu item named "Link 1" with path "node/1".
2. Add second Primary menu item named "Link 2" with path "node/1".
3. Menu item named "Link 1" is removed and "Link 2" is kept
If i do the same with "" the "Link 1" is not deleted.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | multi-node-links-179578-4.patch | 1.21 KB | pwolanin |
Comments
Comment #1
pwolanin CreditAttribution: pwolanin commentedI'll take a look - in D6 you should be able to create multiple links to the same path (if you really want to)
Comment #2
pwolanin CreditAttribution: pwolanin commentedok, I can reproduce this with a node link, but menu module will happily add two links to ''
Comment #3
pwolanin CreditAttribution: pwolanin commentedah, all the node links are in the DB table, they are just not being displayed for some reason! So the initial description of the bug is wrong - a new link is created with a new mlid, but for some reason only one of the two (or more) links is displayed.
Comment #4
pwolanin CreditAttribution: pwolanin commentedthe bug is in function menu_tree_collect_node_links() and function menu_tree_check_access().
patch attached, seems to fix the problem in a reasonable way.
Comment #5
pwolanin CreditAttribution: pwolanin commentedComment #6
chx CreditAttribution: chx commentedyes, this works well. we can change the node_links array as we want because it's only used at these two places.
Comment #7
hass CreditAttribution: hass commentedPatch works for me, too.
Comment #8
Gábor HojtsyThanks, committed.
Comment #9
(not verified) CreditAttribution: commented