BadBehavior: Drupal 6 update

edit: Use the release below instead. No sense in filling up this thread with old releases...

Pobster

Also... If you're particularly bothered about uninstall hooks removing entries from your variable table, change the uninstall hook in badbehavior.install to;

function badbehavior_uninstall() {
  drupal_uninstall_schema('badbehavior');
  db_query("DELETE FROM {variable} WHERE name LIKE 'badbehavior_%'");
  cache_clear_all('variables', 'cache');
}

Apologies, didn't think to put it in the code before... Not that it really matters I guess! Just for neatness!

Pobster
PS. I've also included a nice hook_requirements to check that badbehavior/bad-behavior actually exists and that it contains at least core.inc.php and version.inc.php and removed the watchdog error message from hook_init (as the requirements hook negates it) and finally, I've removed the unnecessary admin/modules#description from hook_help.

Okay I changed the code from hook_init to hook_boot, so now badbahavior is called on *every* page request regardless of whether it's cached or not. Sorry didn't think about doing this before as I don't use page caching myself...

Tested and working...

Pobster

Ah my bad, forgot about this http://drupal.org/node/140311

/**
  * Implementation of hook_menu().
  */
function badbehavior_menu() {
  $items = array();

  $items['admin/settings/badbehavior'] = array(
    'title' => 'Bad behavior',
    'description' => 'Configure automatic spam blocking for your site.',
    'page callback' => 'drupal_get_form',
    'page arguments' => array('badbehavior_settings'),
    'access arguments' => array('administer bad behavior')
  );
  $items['admin/reports/badbehavior'] = array(
    'title' => 'Bad behavior',
    'description' => 'Examine the spam blocking logs for your web site.',
    'page callback' => 'badbehavior_overview',
    'access arguments' => array('administer bad behavior')
  );
  $items['admin/reports/badbehavior/event'] = array(
    'title' => 'Details',
    'page callback' => 'badbehavior_event',
    'access arguments' => array('administer bad behavior'),
    'type' => MENU_CALLBACK);

  return $items;
}

Sorry!

Pobster
PS. Thanks for the credit in the README! ;o)

Hiya!

In Drupal 6.x the t() calls are unnecessary in title and description for hook_menu items. The items are automatically passed through t() now - that was the only change though! :o)

Thanks,

Pobster