When warden_menu() is ran it checks the 'warden_access_callback' variable() before deciding if the menu path should be available. The correct approach with this should be to include the variable check within the access callback.

CommentFileSizeAuthor
#3 warden-n2781467-3.patch1.4 KBdamienmckenna

Comments

DamienMcKenna created an issue. See original summary.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #2781509: Plan for Warden 7.x-1.0 release
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new warden-n2781467-3.patch1.4 KB

This removes the lines from hook_menu(), and adds an update script to trigger the menus to be rebuilt so that the path can be handled properly.

asrob’s picture

asrob
he/him
Primary language Hungarian
Location Hungary 🇭🇺 🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me, that's why RTBC'ed!

mike.davis’s picture

mike.davis commented

Thanks for this Damien, this has now been committed.

mike.davis’s picture

mike.davis commented
Status: Reviewed & tested by the community » Fixed

Updated status!

Status: Fixed » Closed (fixed)

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