Add user picture to comment previews

Here's the patch that abstracts the logic for presets and comments by using hook_comment('view'). This also fixed a possibly destructive change with the last commit by only replaceing the part of $account->content['user_picture'] we care about in hook_profile_alter().

One too many comment->picture assignments...

Did it work before with panels? :) If it's not using the proper theme functions it's kinda hard to support.

Good catch! :)

+++ imagecache_profiles.module	27 Jul 2010 20:01:40 -0000
@@ -266,12 +262,26 @@ function imagecache_profiles_user($op, &
+  if ($op == 'view' && isset($comment->uid) && theme_get_setting('toggle_comment_user_picture')) {
+    // If this is a comment preview, we need to manually add $comment->picture.
+    if (!isset($comment->picture)) {

We need to remove the check for isset($comment->uid) and move it two lines down as !empty($comment->uid).

Other modules (like Gravatar) may inject something into $comment->picture for anonymous users but still want imagecache to load its style. This would prevent it.

Powered by Dreditor.