'Best comment' avatar is incorrect.

phonydream2 - March 29, 2008 - 15:36
Project:Comment bury/promote
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

The following piece of code shows the best comment within node.tpl.php:

  if(module_exists('comment_bury_promote')) {
    print comment_bury_promote_display_best_comment($node);
  }

Unfortunately, it is showing the author of the nodes avatar in the comment and not the person that wrote the comment.

#1

mercmobily - March 29, 2008 - 17:43

Hi,

...?
Are you sure...?
We are using _that_ very code in Free Software Magazine:

http://www.freesoftwaremagazine.com/forum/1257

And it seems to work fine!

Can you confirm this problem? Can you send me an URL?

Merc.

#2

phonydream2 - March 30, 2008 - 13:03

Hi Merc,

Thanks for trying to help out. I was mistaken in my original post. The best comment is not showing the original authors avatar, but the default site avatar. I think this is because there is no value for $picture.

When I add this code, print_r($comment); into comment.tpl.php I get the following variables for the best comment:

[cid][pid][nid][uid][subject][comment]
[hostname] [timestamp]
[score][status]
[format][thread]
[users][name][mail][homepage]
[new][fivestar_rating][fivestar_view]

However, for the remaining comments I get all of the above variables except [hostname] plus a lot more in addition:

[registered_name]
[picture]
[data]

...and a lot more including private message variables, picture delete and avatar variables, guestbook variables, tac_lite variables etc.

#3

mercmobily - March 30, 2008 - 14:10

Hi,

I fixed this version in the current version of this module.
Please download the latest version of the module and confirm that the problem is no longer there.

Merc.

#4

phonydream2 - March 30, 2008 - 16:15

The latest version from CVS solves this problem!
Thanks a million, this module is great.

#5

mercmobily - March 30, 2008 - 17:40
Status:active» fixed

Hi,

I am glad somebody is using it :-D
I have the feeling you might be the only person using it...!

Bye!

Merc.

#6

mercmobily - April 10, 2008 - 09:35
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.