theme_comment_folded and theme_comment_wrapper were converted. Most of the functions don't need to be changed.
I changed the parameters for comment_wrapper to include a node. The function that calls it already had it available and I think it can help themers in a big way by providing more context. With that node in place, it also provides a template suggestion based on the parent node type. Garland had an override for the comment wrapper -kinda hackish? and it now uses this new node parameter.
Also added a few bug fixes. A lot of the theme_flat/threaded_expanded/collapsed functions had undefined * messages. The existing comment.tpl.php from the comment folder and Garland had an anchor tag with an id of "new" set. It's already printed out from theme_comment_view() so it was removed from the .tpl files.
Comment | File | Size | Author |
---|---|---|---|
tplify_comment_1.patch | 10.96 KB | dvessel | |
Comments
Comment #1
merlinofchaos CreditAttribution: merlinofchaos commentedThat id="new" needs to be put back -- that's how the node/2134#new got to the first new comment.
Comment #2
dvessel CreditAttribution: dvessel commentedBut it's already outputted through http://api.drupal.org/api/HEAD/function/theme_comment_view. And it checks for the first new comment. Just checking for new status the way the .tpl did would output it multiple times.
Comment #3
Dries CreditAttribution: Dries commenteddvessel is right (according to my testing). Committed to CVS HEAD. Thanks.
Comment #4
(not verified) CreditAttribution: commented