Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Sorry about that. Didn't really have the time to put together a proper issue summary at the time.
The problem is regression from Author Pane implementing its own Author Pane User Profile templating instead of overriding template_preprocess_user_picture(). When Gravatar was installed, it wouldn't use the Gravatar profile image, as Author Pane did its own stuff for the avatar. Although I've only had a quick look, the attached patch is headed in the right direction. With this patch, it uses Drupal's user picture templating, so Gravatar is still a consideration.
Hmm... I could have sworn I added Gravatar support in D6. Keep in mind that Author Pane intentionally uses its own method for images because it's doing something slightly different than core (linking to the larger profile image as you'd expect rather than the redundant link to the profile page).
I'm on a plane this afternoon so can't look at this any more right now, sorry.
Yup! This still supports different image sizes as it only overrides if $variables['caller'] is set. But again, this probably still needs testing. Thanks Michelle! :-) ... See you next week!
Comments
Comment #1
MichelleUh... Could you give some more info there? :) Is there something broken with the gravatar support?
Comment #2
RobLoachSorry about that. Didn't really have the time to put together a proper issue summary at the time.
The problem is regression from Author Pane implementing its own Author Pane User Profile templating instead of overriding template_preprocess_user_picture(). When Gravatar was installed, it wouldn't use the Gravatar profile image, as Author Pane did its own stuff for the avatar. Although I've only had a quick look, the attached patch is headed in the right direction. With this patch, it uses Drupal's user picture templating, so Gravatar is still a consideration.
Probably needs more testing before committing it.
Comment #3
MichelleHmm... I could have sworn I added Gravatar support in D6. Keep in mind that Author Pane intentionally uses its own method for images because it's doing something slightly different than core (linking to the larger profile image as you'd expect rather than the redundant link to the profile page).
I'm on a plane this afternoon so can't look at this any more right now, sorry.
Comment #4
RobLoachYup! This still supports different image sizes as it only overrides if $variables['caller'] is set. But again, this probably still needs testing. Thanks Michelle! :-) ... See you next week!