This sentence doesn't make any sense:

The string "hook" is used as a placeholder for the module name is the hook definitions.

Found on: http://api.drupal.org/api/group/hooks

CommentFileSizeAuthor
#6 624882-D6.patch1.91 KBjhodgdon
#1 624882.patch1.9 KBjhodgdon

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Title: Documentation problem with Hooks » Typos on Hooks section page
Version: 6.x-dev » 7.x-dev
StatusFileSize
new 624882.patch1.9 KB

Good catch, thanks for reporting! The attached patch fixes this typo and one other one in the same page (and some line wrapping issues).

Fix is for Drupal 7 (same doc there).

Should be backported to Drupal 6 as well if accepted.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Active » Needs review
Bojhan’s picture

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

Ahh, yup- that looks better! RTBC

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Version: 7.x-dev » 6.x-dev
Status: Fixed » Patch (to be ported)

Needs port to Drupal 6.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Patch (to be ported) » Reviewed & tested by the community
StatusFileSize
new 624882-D6.patch1.91 KB

Here's the identical patch (or at least, a patch that will make the output identical) for Drupal 6. Taking the liberty of pre-marking it as reviewed, since it's the same as the D7 reviewed patch, and it's just a doc patch so shouldn't require any testing.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

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