$form_state parameter docs in drupal_validate_form() need typo fixed [#1601136]

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

Comment	File	Size	Author
#3	form-form-state-docs-1601136-3.patch	784 bytes	Chaulky
#3
#3	D7-form-form-state-docs-1601136-3-do-not-test.patch	764 bytes	Chaulky
#3

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 29 May 2012 at 17:20

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

Comment #2

JSCSJSCS CreditAttribution: JSCSJSCS commented 30 May 2012 at 00:48

That does make more sense.

Comment #3

Chaulky CreditAttribution: Chaulky commented 30 May 2012 at 04:02

File	Size
D7-form-form-state-docs-1601136-3-do-not-test.patch	764 bytes

form-form-state-docs-1601136-3.patch	784 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

Comment #4

Chaulky CreditAttribution: Chaulky commented 30 May 2012 at 04:03

Status:

Active

» Needs review

Forgot to change status

Comment #5

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 30 May 2012 at 15:01

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