[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