Hi!
Currently module does nothing if field is not filled but field marked as required (no message, no field highlighting).

CommentFileSizeAuthor
#2 cck_phone-validation-1659068-2.patch6.77 KBLeksat
#1 cck_phone-required_validation-1659068-1.patch648 bytesLeksat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Leksat’s picture

Leksat CreditAttribution: Leksat commented
FileSize
cck_phone-required_validation-1659068-1.patch648 bytes

Attached patch fixes the bug.

Leksat’s picture

Leksat CreditAttribution: Leksat commented
Title: Module doesn't respect "required" setting » Validation issues
Status: Active » Needs review
FileSize
cck_phone-validation-1659068-2.patch6.77 KB

I found another validation errors and fix them:
1. Removed double field validation via #element_validate and hook_field_validate().
2. Removed validation of generic things like maxlength of select option, because Drupal do this job as well.
3. $errors variable is used instead of form_set_error().

Please review the patch.