This module implements hook_access () very strangely.

Check out the userplus_access($perm) function and the doco for hook_access and you will see what I mean.

Comments

agileware’s picture

agileware commented
Title: Strange usage of hook_access() » Rename userplus_access to avoid naming conflict with hook_access

OK, I see why now.

It is called from hook_menu to determine menu access.

In that case userplus_access($perm) should be renamed to userplus_menu_access($perm) or something like that so there is no naming conflict with hook_access().

marcp’s picture

marcp commented
Status: Active » Fixed

Thanks for catching that! Commited a fix to DRUPAL-6--2 branch.

Status: Fixed » Closed (fixed)

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