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

Comments

leksat’s picture

leksat commented
StatusFileSize
new cck_phone-required_validation-1659068-1.patch648 bytes

Attached patch fixes the bug.

leksat’s picture

leksat commented
Title: Module doesn't respect "required" setting » Validation issues
Status: Active » Needs review
StatusFileSize
new 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.