Closed (fixed)
Project:
SpamSpan filter
Version:
6.x-1.5-beta2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Jan 2009 at 09:54 UTC
Updated:
4 Dec 2010 at 16:46 UTC
Hello,
I found that if I want to uncheck the checkbox in SpamSpan module settings page I cannot do so.
Comments
Comment #1
robert castelo commentedCould you double check that the SpamSpan checkbox is not able to uncheck because of Checkbox Validate module:
Disable Checkbox Validate module and test to see if the problem is still there, if the checkbox behaves correctly then it is a problem with Checkbox Validate.
Comment #2
sunThis is plain wrong:
An administrative module settings checkbox cannot be required. I can only guess that this is a copy+paste error.
Comment #3
djg_tram commentedI second that, Robert, I also had to modify the code to remove the #required attribute. This is not a Checkbox Validate issue. A checkbox never can be required by definition because it denotes an optional item that the user is allowed to leave emtpy. The core Drupal bug that forgets to validate checkboxes, the bug that Checbox Validate is written to fix, allows you to get away with this incorrect semantics without that additional module but that doesn't shift the responsibility to Checkbox Validate. Make sure you never #require a checkbox, plain and simple, because it is logically incorrect to do so.
Bye,
Comment #4
lakka commentedFixed in 6--1.5. Thanks
Comment #5
lakka commented