Hi everyone,

I think I've discovered a bug in forward 5.x-1.15. When I'm logged out (i.e., don't have the "administer forward" permission), forward completely ignores the flood control limit I've set on the form.

I traced this down to line 430 of forward.module. By default, it looks like this (abbreviated):

form_set_error(null, variable_get( ... ));

When I change it to this, flood control works properly:

form_set_error('recipients', variable_get( ... ));

I'm not sure why this is happening exactly, perhaps something was changed in form_set_error?

Anyway, I hope this is helpful to someone. If anyone can shed anymore light on this issue, it'll be greatly appreciated. Cheers :)

Comments

phenaproxima’s picture

FYI, I am using Drupal 5.7.

seanr’s picture

Title: Flood control bug » form_set_error doesn't accept null for field name
Version: 5.x-1.15 » 5.x-1.x-dev
Status: Needs review » Fixed

Fixed. New build will be up momentarily.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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