API page: https://api.drupal.org/api/drupal/core%21modules%21system%21system.api.p...

The documentation for hook_schema() needs improvement. It has at least the following issues:

  • It doesn't specify its @return types.
  • It doesn't @see closely-related functions.
  • It has minor comment formatting issues.

Patch to follow.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TravisCarden’s picture

Status: Active » Needs review
Issue tags: +Novice
FileSize
1.9 KB
2.64 KB

Most of the changes in the attached patch are merely justifying text to fill the available line length. A word-diff makes it easier to see at a glance that the text itself is not being changed, so I'm attaching one. Other than that, I think this is pretty self-explanatory.

Status: Needs review » Needs work

The last submitted patch, word-diff.diff, failed testing.

TravisCarden’s picture

Status: Needs work » Needs review
FileSize
1.9 KB
2.64 KB

Mmm. I was afraid the testbot might want to test the word-diff. Let's try again.

TravisCarden’s picture

Assigned: TravisCarden » Unassigned
jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Thanks! good to go.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks again! Committed to 8.x.

jhodgdon’s picture

Committed it to 7.x too.

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