http://api.drupal.org/api/drupal/core!modules!user!user.module/function/...

  foreach (module_implements('permission') as $module) {
    $function = $module . '_permission';
    if ($permissions = $function('permission')) {

should be

  foreach (module_implements('permission') as $module) {
    $function = $module . '_permission';
    if ($permissions = $function()) {

Exists back to d7, introduced in #334030: replace module_list with module_implements when calling a hook

CommentFileSizeAuthor
#6 1864188-6-user_filters_d7.patch608 bytesjohnshortess
#2 1864188-1-user_filters_hook_permission.patch628 bytesjohnshortess

Comments

johnshortess’s picture

johnshortess
he/him
Primary language English
Location Fairfax, Virginia, USA
commented
Assigned: Unassigned » johnshortess
johnshortess’s picture

johnshortess
he/him
Primary language English
Location Fairfax, Virginia, USA
commented
Status: Active » Needs review
StatusFileSize
new 1864188-1-user_filters_hook_permission.patch628 bytes

I think this should fix it.

marvin_b8’s picture

marvin_b8 commented

sry tested and everything nice

hefox’s picture

hefox commented
Status: Needs review » Reviewed & tested by the community

Looks good to me

dries’s picture

dries commented
Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed to 8.x. Sounds like this needs to be backported to 7.x.

johnshortess’s picture

johnshortess
he/him
Primary language English
Location Fairfax, Virginia, USA
commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new 1864188-6-user_filters_d7.patch608 bytes

And here's a patch for D7.

marvin_b8’s picture

marvin_b8 commented
Status: Needs review » Reviewed & tested by the community

Looks good to me, no problems.

David_Rothstein’s picture

David_Rothstein commented
Status: Reviewed & tested by the community » Fixed

Fun stuff. Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/5c9c9a6

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