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().

Comments

Status: Needs review » Needs work

Patch failed to apply. More information can be found at http://testing.drupal.org/node/14518. If you need help with creating patches please look at http://drupal.org/patch/create

jvandyk’s picture

StatusFileSize
new655 bytes

Patch now made from Drupal root.

jvandyk’s picture

Status: Needs work » Needs review
Anonymous’s picture

Status: Needs review » Needs work

The last submitted patch failed testing.

drewish’s picture

Status: Needs work » Needs review

those tests look unrelated. bumping to get a retest.

cburschka’s picture

testbot must now be explicitly told to retest a patch. Done.

Status: Needs review » Needs work

The last submitted patch failed testing.

andypost’s picture

Status: Needs work » Closed (duplicate)

Suppose better to rework menu #544318: Rework trigger_menu()