Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This text: "The title of this node, always treated a non-markup plain text"
should be changed to this text:
"The title of this node, always treated as non-markup plain text"
There's a typo. The word "a" should be changed to "as"
Comment | File | Size | Author |
---|---|---|---|
#2 | 884718.diff | 566 bytes | ctmattice1 |
Comments
Comment #1
jhodgdonThis is in the example function body for hook_schema, and is present in Drupal 7 as well. Should be fixed in D7 first.
Good project for a novice Drupal contributor...
Comment #2
ctmattice1 CreditAttribution: ctmattice1 commentedI'll Bite, diff attached.
question though, why is the comment
// example (partial) specification for table "node"
under hook_schema()
shouldn't it be in the doc block above something like
* A schema definition structure array. For each element of the
* array, the key is a table name and the value is a table structure
* definition. The following example is a partial specification for the
* "node" table.
Comment #3
jhodgdonRegarding your question about comment placement -- No. The docblock should document what the hook does and what its params and return value are. The function body gives an example, and any documentation that is specific to that particular example should go as code comments in the function body.
So, after you submit a patch, you should set the status to "needs review" so that (a) patch reviewers are aware that you have a patch that you think is ready to go and (b) the testing bot will launch to make sure the patch is OK.
In this case, I will go ahead and bypass "needs review" and mark it "reviewed".
Thanks!
Comment #4
jhodgdonPlease mark this as D6 / to be ported if committing, thanks!
Comment #5
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD.
Comment #6
jhodgdonI fixed it in the D6 contrib repository too.