Since in most cases the call will be performed, there is no time penalty by moving the call at the beginning of the function. And it makes the $id variable available to the spam_log() call within the 1st if() condition.

CommentFileSizeAuthor
spam-6.x-id_not_defined.patch1 KBAlexisWilke
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

AlexisWilke’s picture

AlexisWilke CreditAttribution: AlexisWilke commented
Status: Needs review » Fixed

Committed here:
http://drupal.org/cvs?commit=478128

Status: Fixed » Closed (fixed)

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