Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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
Comment | File | Size | Author |
---|---|---|---|
form_required1.patch | 13.98 KB | ivrh | |
Comments
Comment #1
jcnventura CreditAttribution: jcnventura commentedThanks for the patch! I have just committed it to CVS.
Comment #3
tsi CreditAttribution: tsi commentedUser message shouldn't be required or at least optional in settings page.
Comment #4
jcnventura CreditAttribution: jcnventura commentedSee #703296: Cancel shouldn't required field input for more on this error. It is now fixed.