Added field has a wrong value for "destination" property.
Drupal function drupal_get_destination() doesn't return a string but a single element array:
array('destination' => ...dest_url...);

Spamicide will send this value to drupal_goto(), generating warnings.

See the attached patch for a fix on function spamicide_form_alter().

CommentFileSizeAuthor
bad_destination_value.diff1013 bytesnicorac
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

seandunaway’s picture

seandunaway CreditAttribution: seandunaway commented
Priority: Normal » Major
Status: Needs review » Reviewed & tested by the community

I've been using this in production. This is a major fix as it throws half a dozen warnings to watchdog for every submission.

Thanks nicorac!

nicorac’s picture

nicorac CreditAttribution: nicorac commented

You're welcome.
Thanks for testing it.

Hope it will be included soon.

lipcpro’s picture

lipcpro CreditAttribution: lipcpro commented
Assigned: Unassigned » lipcpro
Status: Reviewed & tested by the community » Closed (fixed)