phptemplate_theme doc headers need standardization

Agreed. The doc headers for http://api.drupal.org/api/function/phptemplate_theme in both Drupal 6 and Drupal 7 need a clean up.

Since they are actually hook_theme() implementations, they should just say:

Implements hook_theme().

as the first line, then maybe have a blank line and some more information there. The current doc is lacking () after hook_theme(), so there is no link to hook_theme(), which would have said what the args were.

See http://drupal.org/node/1354 for standards. And please patch in 7.x first, then backport to 6.x.

Looking at this function, I don't think it needs doc beyond what is in hook_theme(). Our standard is only to document hook implementations extensively if they are doing something really noteworthy. So, here's a patch that standardizes the doc header for this function, and the other one in the same file (which completely lacked doc before).

Should be ported to D6 if accepted.

Committed to HEAD, thanks!

Needs port to Drupal 6 as well.

Version for D6.

Sigh. sender - I forgot to mention that in D6, all the other hook implementations are still following the old standard, so they say
Implementation of hook_foo().
rather than
Implements hook_foo().

So we should probably also do the same here.

No biggie. Here's a new version with the change you mentioned.

Looks good to me. Thanks!

Thanks, committed.