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.").

CommentFileSizeAuthor
forward_message_require.patch343 bytespopulist

Comments

seanr’s picture

Status: Needs review » Fixed

Fixed in CVS DRUPAL-5, will be in the next release.

Status: Fixed » Closed (fixed)

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