small issue: would be *much* easier to read if the field options were a second level list, as at


#5 948520-reroll.patch1.67 KBjhodgdon
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 948520-reroll.patch. View
#4 database.inc_.doxygen.patch1.56 KBmvc
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch database.inc_.doxygen.patch. View


jhodgdon’s picture

Title: Documentation problem with Schema API » Schema API list formatting is off
Issue tags: +Novice

Yes, the formatting is wrong, thanks for reporting!

Fixing that would be a good project for a novice contributor.
There is information about list formatting on

mvc’s picture

Where is this text? How can I fix it?

jhodgdon’s picture

The text is inside a topic/group section in the file includes/

You can fix it by making a patch.

There's information about topic/group sections on (see link above for list formatting bottom of that page has info about topic/group sections).

There's info on creating patches at

And you can get additional help on IRC -- see -- on the #drupal-contribute channel is the place to get help starting as a contributor. Is that enough to get you started?

mvc’s picture

1.56 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch database.inc_.doxygen.patch. View

yes, thanks! i just didn't know which file to patch. i foolishly grep'd a fresh copy of drupal 6 without realizing that my search phrase would be broken across a newline, and when i saw no results i assumed this documentation was maintained outside drupal someplace. *blush*

i didn't actually install doxygen and the api module and properly test this patch, but i compared includes/ against includes/database/ in drupal 7 (where the formatting works) and i believe this will fix the problem.

jhodgdon’s picture

Status: Active » Reviewed & tested by the community
1.67 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 948520-reroll.patch. View

Please read -- patches need to be rolled from the Drupal root, and should be done with cvs diff if possible. I could not get this patch to apply without editing the top.

Also, when you attach a patch, please set the status to "Needs review", so that we know there is a patch that is waiting for a review.

Apart from that, you did the exact right thing in that patch... so to save time, I just rerolled it. Thanks!

mvc’s picture

thanks for your all your feedback, jhodgdon. i've submitted lots of patches for modules, but only a few for core, so i didn't think of rolling the patch from the drupal root directory. and yep, i definitely should have known to select "needs review"!

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 948520-reroll.patch, failed testing.

Gábor Hojtsy’s picture

Status: Needs work » Fixed

Committed, thanks.

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

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