API page: http://api.drupal.org/api/drupal/includes%21form.inc/function/drupal_val...

Suggest rewording in:

"Validation handlers can also $form_state to pass information on to submit handlers."

to:
"Validation handlers can also pass information on to submit handlers."

CommentFileSizeAuthor
#3 form-form-state-docs-1601136-3.patch784 bytesChaulky
#3 D7-form-form-state-docs-1601136-3-do-not-test.patch764 bytesChaulky

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Title: Reword Form_State Paragraph » $form_state parameter docs in drupal_validate_form() need typo fixed
Version: 7.x-dev » 8.x-dev
Priority: Normal » Minor
Issue tags: +Novice, +Needs backport to D7

Actually, I think the word "use" is missing: Validation handlers can also *use* $form_state...

JSCSJSCS’s picture

JSCSJSCS commented

That does make more sense.

Chaulky’s picture

Chaulky commented
StatusFileSize
new D7-form-form-state-docs-1601136-3-do-not-test.patch764 bytes
new form-form-state-docs-1601136-3.patch784 bytes

Attached the very minor patch to add the missing "use". Included D7 patch since I can't imagine we'll need to go back and forth on this one.

Applies to latest...
D8: commit ca334de86
D7: commit f20658a8

Chaulky’s picture

Chaulky commented
Status: Active » Needs review

Forgot to change status

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Thanks! I'll get this committed shortly. And by the way, it wasn't even necessary to make the 7.x patch, since the 8.x patch would have applied to 7.x just fine (with -p2). :)

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed to 8.x/7.x. Thanks again!

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