Looking at this, it seems like there should be some defaults for flood items, rather than relying only on the flood table- this makes it possible for admins to install the module and configure it rather than waiting for flood events.

This patch has a default for contact, and t() fix.

Oh and it would be funny to rename this module "Moses"

CommentFileSizeAuthor
flood_exemption.diff1.07 KBarthurf

Comments

nedjo’s picture

nedjo
he/him/his
Primary language English
commented

Good idea on the recognizing contact. Pls reformat (for coding standards) as:


        // We know some basic types are in core 
	if (module_exists('contact')) {
         $types[] = 'contact';
        }

Permissions aren't passed through t().

Pls patch HEAD and then DRUPAL-5. I made some significant changes since the stable release, need to issue a new one when your change is in.

nedjo’s picture

nedjo
he/him/his
Primary language English
commented

oops, that should have been:

  // We know some basic types are in core.
  if (module_exists('contact')) {
    $types[] = 'contact';
  }