Closed (fixed)
Project:
Forward
Version:
5.x-1.12
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
16 Apr 2008 at 18:59 UTC
Updated:
20 Jul 2009 at 21:50 UTC
The personal message field of the forward form is set to #required = 1 in the form definition. However, in the forward_form_validate there is an additional validation check ala: if (!$form['message']) { form_set_error('message', t('You must enter a message.')); }. This prevents a hook_form_alter from setting the #required bit to 0 because the secondary check will still make the field required. This patch just removes the secondary check. The only downside here is that the error message is not as nice as before ("You must enter a message" versus "Your Personal Message field is required.").
| Comment | File | Size | Author |
|---|---|---|---|
| forward_message_require.patch | 343 bytes | populist |
Comments
Comment #1
seanrFixed in CVS DRUPAL-5, will be in the next release.