| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.0.2 |
| Bug #3042 | auto-detection based validation harmful | ||
|---|---|---|---|
| Submitted: | 2004-12-27 20:52 UTC | ||
| From: | andrew dot clark at ucsb dot edu | Assigned: | justinpatrin |
| Status: | Closed | Package: | DB_DataObject_FormBuilder |
| PHP Version: | 4.3.10 | OS: | FreeBSD 4.10 |
| Roadmaps: | (Not assigned) | ||
[2004-12-27 20:52 UTC] andrew dot clark at ucsb dot edu
Description:
------------
DO_FormBuilder tries to guess validation rules, but overrides rules added to form (through postGenerateForm, for example).
For integer types that are not integers in the form (IP address converted from dotted decimal to long notation, for example), this is harmful.
Here's the affected code in FormBuilder.php:
// Auto-detect field types depending on field's database type
switch (true) {
case ($type & DB_DATAOBJECT_INT):
if (!isset($element)) {
$element =& $this->_createIntegerField($key);
}
$elValidator = 'numeric';
Comments
[2004-12-28 02:46 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2004-12-28 11:56 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2004-12-28 18:59 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2005-01-02 00:40 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!