The second "sentence" of the second paragraph has no verb, plus it has back-to-back contradictory prepositions:
"When altering a node form, the node object retrieved at from $form['#node']."

Does it mean to say:
"When working with a node form, the node object can be retrieved from $form['#node']."?

See page
http://api.drupal.org/api/function/hook_form_alter/6

CommentFileSizeAuthor
#3 743124-3.patch895 bytesldpm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Issue tags: +Novice

Good catch! Needs to be fixed in Drupal 7 first, then backported to Drupal 6.

Good project for a new doc contributor.

ldpm’s picture

ldpm CreditAttribution: ldpm commented
Assigned: Unassigned » ldpm
ldpm’s picture

ldpm CreditAttribution: ldpm commented
FileSize
743124-3.patch895 bytes

Here is a patch for D7

ldpm’s picture

ldpm CreditAttribution: ldpm commented
Assigned: ldpm » Unassigned
Status: Active » Needs review
crotown’s picture

crotown CreditAttribution: crotown commented
Status: Needs review » Reviewed & tested by the community

Fixed the grammar.

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Version: 6.x-dev » 7.x-dev

just changing version

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Reviewed & tested by the community » Needs work

DELETED COMMENTS

I was wrong, hooks have to be in the imperative form...

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Needs work » Needs review

After editing #7

jhodgdon’s picture

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

The patch in #3 looks fine to me. Not sure what all the comments up there are about.

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented

#10 I was confused about some verb standards, but after some reading I know we have different ones for hooks :)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Good catch! Committed to HEAD.

Welcome to the core team, ldpm! :)

Status: Fixed » Closed (fixed)
Issue tags: -Novice

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

Issue tags: +Novice

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.