I constantly have trouble understanding this function so I wrote this. Maybe it needs to be in @return, maybe not, I am not sure.

#7 1973762_7.patch727 byteschx
PASSED: [[SimpleTest]]: [MySQL] 54,710 pass(es). View
docs.patch667 byteschx
PASSED: [[SimpleTest]]: [MySQL] 54,484 pass(es). View


cosmicdreams’s picture

looks good to me but you might want someone on the documentation team to review this. Is there anyway to test the syntax to see if it properly documents (perhaps using simplytest.me?)

chx’s picture

Assigned: Unassigned » yched
yched’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me. Thanks !

chx’s picture

Issue tags: +needs backport to D7

also for D7.

alexpott’s picture

Assigned: yched » jhodgdon

This one's for Jennifer.

jhodgdon’s picture

Status: Reviewed & tested by the community » Needs work

I think the example does belong in the @return section, since it describes the return value (and when re-patching, don't leave blank lines within the @return). Thanks!

chx’s picture

Status: Needs work » Reviewed & tested by the community
727 bytes
PASSED: [[SimpleTest]]: [MySQL] 54,710 pass(es). View
jhodgdon’s picture

Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Fixed

Thanks! Committed to both 8.x and 7.x.

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

Anonymous’s picture

Issue summary: View changes

Updated issue summary.