$info['report access'] is an array, so $access_permissions[] = $info['report access']; makes $access_permissions into an array of arrays instead of a flat array.

CommentFileSizeAuthor
#6 2680597-6.diff1 KBB_man
#2 2680597.diff1022 bytesdrumm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drumm created an issue. See original summary.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm at Drupal Association commented
Status: Active » Needs review
FileSize
2680597.diff1022 bytes

Attached is a patch to correct this.

The PHP warning this causes is:

Warning: Illegal offset type in isset or empty in user_access() (line 833 of /var/www/dev/spam-drupal.dev.devdrupal.org/htdocs/modules/user/user.module).

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm at Drupal Association commented
Issue tags: +affects drupal.org
drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm at Drupal Association commented
Assigned: drumm » Unassigned
drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm at Drupal Association commented
Issue tags: -affects drupal.org
B_man’s picture

B_man
he/him
Location California, USA
CreditAttribution: B_man at Drupal Association commented
Title: Improve $access_permissions array handling in _mollom_actions_access_callbacks() » Improve $access_permissions array handling in _akismet_actions_access_callbacks()
Project: Mollom » Akismet (Mollom replacement)
Version: 7.x-2.x-dev »
FileSize
2680597-6.diff1 KB

Attaching an updated diff for akismet.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm at Drupal Association commented
Issue tags: +affects drupal.org
drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm at Drupal Association commented
Project: Akismet (Mollom replacement) » Akismet
Version: » 7.x-2.x-dev