Hi. As long as fileds in the email to a friend form are all required and skipping Message text returns validation error that this field must be filled in - I have added '#required' => TRUE, to this form and the patch agains v. 6.10 is attached.

Please review and port to the next module revision.

The advantages of having those additions are:
1. Forms API cares about validation (if fields left empty)
2. Usability improvement as user can see asterisk next to the required field

CommentFileSizeAuthor
form_required1.patch13.98 KBivrh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jcnventura’s picture

Status: Patch (to be ported) » Fixed

Thanks for the patch! I have just committed it to CVS.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

tsi’s picture

Status: Closed (fixed) » Active

User message shouldn't be required or at least optional in settings page.

jcnventura’s picture

Status: Active » Closed (fixed)

See #703296: Cancel shouldn't required field input for more on this error. It is now fixed.