[wp-testers] rc2 and rc3 doesn't add extra classes to comments
Philip M. Hofer (Frumph)
philip at frumph.net
Thu Jan 27 14:12:25 UTC 2011
I'm not seeing anything off hand.. besides not passing the depth and
maxdepth = get_option('thread_comments_depth') to the comment_reply_link, --
you're missing this: $GLOBALS['comment_depth'] = $depth; at the top, while
the $depth is passing through the callback it's not setting the globa, heck
I have no idea if that's important or not, that being said your code -- that
being said, you should be using 'default' instead of case '';
http://danixland.pastebin.com/RL8pkSiA .. switched things around on ya.
Have you removed the callback from wp_list_comments function to see if it
works without your call back? I've seen something like this before when
there was a plugin/or add_filter('comment_class', not returning properly
past the first item that was assigned to return which is ..
----- Original Message -----
From: "danilo" <danixland at gmail.com>
To: <wp-testers at lists.automattic.com>
Sent: Thursday, January 27, 2011 5:56 AM
Subject: Re: [wp-testers] rc2 and rc3 doesn't add extra classes to comments
> On 27/01/2011 14:34, Philip M. Hofer (Frumph) wrote:
>> sorry meant filter on the comment_class, and would like to see your
>> callback code in a pastebin just to see if it might be something you
>> overlooked
>
> Thanks for replying, here's the pastebin for my comment's callback
> function:
> http://danixland.pastebin.com/GHNXDqqz
> it's almost the same as the one you may find in the toolbox framework by
> automattic, I just modified it a bit but this same function works on
> WordPress 3.0.4, I don't understand why it doesn't work on the nightly
> test site...
>
> Let me know if you need to see some other code, I'm testing it locally, so
> I'll have to use pastebin...
>
> --
> Danilo aka danix
> http://danixland.net
> _______________________________________________
> wp-testers mailing list
> wp-testers at lists.automattic.com
> http://lists.automattic.com/mailman/listinfo/wp-testers
>
More information about the wp-testers
mailing list