akismet.admin.inc lines
'#default_value' => variable_get('akismet_check_nodetypes', NULL)
'#default_value' => variable_get('akismet_antispambot_rules', NULL)
render PHP warnings:
warning: Invalid argument supplied for foreach() in /includes/form.inc on line 1181
setting default values to empty arrays should fix this issue
'#default_value' => variable_get('akismet_check_nodetypes', array())
'#default_value' => variable_get('akismet_antispambot_rules', array())

CommentFileSizeAuthor
#5 akismet_242339_2.patch2 KBdrolp
#1 akismet_242339.patch1.85 KBdrewish
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drewish’s picture

drewish CreditAttribution: drewish commented
Status: Active » Needs review
FileSize
akismet_242339.patch1.85 KB

try this out.

drewish’s picture

drewish CreditAttribution: drewish commented
Status: Needs review » Fixed

committed to HEAD.

yraffah’s picture

yraffah CreditAttribution: yraffah commented

I'm still getting the same exact error using the HEAD version

axyjo’s picture

axyjo CreditAttribution: axyjo commented
Status: Fixed » Active
drolp’s picture

drolp CreditAttribution: drolp commented
FileSize
akismet_242339_2.patch2 KB

I only see these prior to entering my api key.

That being said the original patch only touches akismet.module, akismet.admin.inc have some instances as well. See attached patch. (applies to head)

drewish’s picture

drewish CreditAttribution: drewish commented
Status: Active » Fixed

thanks committed to HEAD.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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