Hi!

I have found ugly bug in gravatar.module function: gravatar_preprocess_user_picturem. When user sends picture, gravatar module ignores it and display avatar from their website.

it was:

    // Decide with picture to use.
    if (!empty($account->picture->filepath)) {
      $picture = $account->picture->filepath;
    }

it should be:

    // Decide with picture to use.
    if (!empty($account->picture->uri)) {
      $picture = $account->picture->uri;
    }
CommentFileSizeAuthor
user_default_picture-gravatar.patch700 bytesquex
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Assigned: Unassigned » Dave Reid
Status: Active » Reviewed & tested by the community

Thanks for the patch, will commit shortly. At some point this API changed for file objects, yay!

Dave Reid’s picture

Status: Reviewed & tested by the community » Fixed

Thanks! Committed the patch to CVS!
http://drupal.org/cvs?commit=456396

Dave Reid’s picture

Version: 8.x-1.x-dev » 7.x-1.x-dev

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.