Closed (fixed)
Project:
AntiSpam
Version:
6.x-1.3
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Dec 2010 at 01:07 UTC
Updated:
4 Aug 2017 at 08:50 UTC
Jump to comment: Most recent
Hi there,
Thanks for a great module.
I updated to the latest version of devel today and found a number of warning for your module.
notice: Undefined variable: moderator_types in antispam_menu() (line 453 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
notice: Undefined variable: moderator_types in antispam_menu() (line 467 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
notice: Undefined variable: moderator_types in antispam_menu() (line 477 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
notice: Undefined variable: moderator_types in antispam_menu() (line 487 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
notice: Undefined variable: moderator_types in antispam_menu() (line 497 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
notice: Undefined variable: moderator_types in antispam_menu() (line 507 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
notice: Undefined variable: moderator_types in antispam_menu() (line 517 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
notice: Undefined variable: moderator_types in antispam_menu() (line 527 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
notice: Undefined variable: moderator_types in antispam_menu() (line 537 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
notice: Undefined variable: moderator_types in antispam_menu() (line 547 of /vpfa-dev/hr_drupal6_template/sites/all/modules/antispam/antispam.module). =>
I asked about it in devel issues and found out that
"notice: Undefined variable" used to be ignored, but it's good practice to always give your variables a value before referencing them; these notices often point to bugs, and that's why we're not suppressing them anymore.
Comments
Comment #1
pixture commentedThanks for the information.
I found what's wrong. I will fix this within a few days.
Meanwhile, if you want to avoid these warning messages, add the following line to antispam.module file.
Comment #2
_vid commentedThat's awesome.
Thanks.
Vid
Comment #3
cfinke commentedThis appears to have been fixed at some point. Closing.