After an illuminating thread on the translators' list, we decided it'd be nice to backport some of the PHPDoc fixes from the t() function in D6 into D5. The docs were nicely fleshed out, even though the function itself changed very little. I'm working on a patch now, stay tuned.

Comments

dww’s picture

Status: Active » Needs review
StatusFileSize
new6.82 KB
new3.02 KB
new3.02 KB

This is the entire D6 version of the PHPdoc comment, except I stripped out the D6-only changes:

- url() args in an example.
- the D6-only $langcode argument.

I also fixed a few lingering formatting problems from the D6 and HEAD version, included in the D5 patch, and attached here as separate patches for D6 and HEAD (the D6 patch applies to HEAD with fuzz, but it was easy to generate a clean patch). I don't think the usual "rule" about committing in HEAD and backporting applies to doc formatting patches, and the most important part is getting the content into the D5 version, so I'm leaving the version of this as 5.x-dev.

Thanks,
-Derek

drumm’s picture

Title: Backport PHPDoc improvements on t() from D6 » t() documentation cleanup
Version: 5.x-dev » 7.x-dev

Committed to 5.x.

drumm’s picture

Status: Needs review » Reviewed & tested by the community
dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to DRUPAL-6 and CVS HEAD. Thanks dww!

dww’s picture

No problem, thanks Drumm and Dries. Happy holidays! :)

Status: Fixed » Closed (fixed)

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