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.Ran into this:
Notice: Undefined index: menu_name in admin_links_access_filter_preprocess_menu() (line 9 of modules/contrib/admin_links_access_filter/admin_links_access_filter.module).
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | ensure-menu-name-exists.patch | 949 bytes | a.hjelm |
Comments
Comment #2
a.hjelm CreditAttribution: a.hjelm commentedI'll admit that I'm not 100% sure that this is actually related to this module or if something else has been messed up, but this is where I ran into it.
menu_name seems to not be available inside $variables at all times, so here's a small patch that looks out for that and grabs the menu_name from the first loaded menu item if needed.
Comment #3
agoradesign CreditAttribution: agoradesign commentedInteresting,,, haven't seen this before. Would be interesting, how this can happen?!?
However, adding these extra lines won't hurt anyway and makes it more bullet proof :-)
Thank you very much for the patch, will commit this asap
Comment #4
a.hjelm CreditAttribution: a.hjelm commentedYeah, it's a bit strange that the menu_name is not available.
Also, realised I exported the patch with local paths. Here's a properly formatted one.
Comment #6
agoradesign CreditAttribution: agoradesign commentedCommitted, thanks! :-)
(I'll tag a new alpha release, as I don't think that we'll have many upcoming changes in the near-future)
Comment #7
agoradesign CreditAttribution: agoradesign commented