In Drupal 6.2, menu items became responsible for declaring their own access callbacks, as menu item inheritance was no longer supported.
The trigger module was not updated to reflect this.
Thus, the menu items created as local tasks for third-party modules do not have the correct access callback. Instead, the callback defaults to user_access().
This patch makes it explicit that the access callback should be trigger_access_check().
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | trigger_access_checkjv.patch | 655 bytes | jvandyk |
| trigger_access_checkjv.patch | 607 bytes | jvandyk |
Comments
Comment #2
jvandyk commentedPatch now made from Drupal root.
Comment #3
jvandyk commentedComment #4
Anonymous (not verified) commentedThe last submitted patch failed testing.
Comment #5
drewish commentedthose tests look unrelated. bumping to get a retest.
Comment #6
cburschkatestbot must now be explicitly told to retest a patch. Done.
Comment #8
andypostSuppose better to rework menu #544318: Rework trigger_menu()