I ran into a strange problem with this module. I am using i18n_menu as well, and additionally some custom modules to extend that.

What i'm noticing is that on my local dev station, i am able to modify menu_views links by invoking hook_translated_menu_link_alter(), but for some unknown reason, it is not working on my actual servers.

After much digging, it seems that for some reason i haven't sorted out yet, the link object is not alterable...

It seems that adding one line to the menu_views hook_menu_link_alter will tell the alter system that the link should be alterable.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jannis created an issue. See original summary.

jannis’s picture

Here is a patch that adds a line to menu_views implementation of hook_menu_link_alter() that adds the alter option to every views_menu menu link and sets its value to TRUE. This allows views_menu links to be alterable by other modules.

jannis’s picture

Status: Active » Needs review

  • markcarver committed c8dc5bf on 7.x-2.x authored by jannis
    Issue #2939616 by jannis: menu views links don't have 'alter' option
    

  • markcarver committed c8dc5bf on 8.x-3.x authored by jannis
    Issue #2939616 by jannis: menu views links don't have 'alter' option
    
markhalliwell’s picture

Status: Needs review » Fixed
markhalliwell’s picture

Category: Feature request » Bug report

Also, this is really a bug, not a feature.

jannis’s picture

that was fast

markhalliwell’s picture

Heh, it was a relatively simple enough change and I figured you've already spent the time to debug it. This project isn't exactly on my fast track of things to thoroughly test, so I'm trusting the patch :D

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.