Looking at a user's profile page (http://example.com/user/6), Gravatar shows a broken image.

View source shows me an img src of:

http://www.lea-sl.org/sites/default/files/styles/thumbnail/http/www.gravatar.com/avatar/9a9448e47d947a385db1963ea8fdd6f5.jpg%3Fd%3Dhttp%253A//www.lea-sl.org/sites/default/files/AnonymousPicture.gif%26s%3D100%26r%3DPG

Clearly this can't be right. :-)

I don't have the time at the moment to isolate and see if it's some other module having fun at my expense, but I'm wondering if anyone else has run into this.

CommentFileSizeAuthor
#19 gravatar-image_url_munged_when_pic_display_style_selected-1689850-19.patch763 bytesdeggertsen
#17 gravatar-image_url_munged_when_pic_display_style_selected-1689850-17.patch763 bytesacouch
#11 interdiff.txt364 bytesandreymaximov
#11 gravatar-image_url_munged_when_pic_display_style_selected-1689850-11.patch735 bytesandreymaximov
#9 image-url-munged-when-pic-display-style-selected-1689850-09.patch733 bytesacouch

Comments

verscienta’s picture

verscienta commented

That is exactly what has happened to me. I can't quite figure out what the problem is. Using Drupal 7.15.

verscienta’s picture

verscienta commented
Priority: Normal » Major
vacilando’s picture

vacilando commented

Same here, even in the current dev version.

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Version: 7.x-1.1 » 7.x-1.x-dev
vacilando’s picture

vacilando commented
Version: 7.x-1.x-dev » 7.x-1.1
Status: Active » Fixed

OK, you need to go to /admin/config/people/accounts and "Picture display style" from "thumbnail" (or whatever style) to "".

Then go to the user account in question and re-save it to get your Gravatar to display.

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented
Status: Fixed » Active

Not sure I'd call that a fix, though it could be a workaround if you don't care about applying image styles to profile pics.

deggertsen’s picture

deggertsen commented

Yeah, I personally want image styles for my profile pics. Has anybody come up with an actual solution to this one?

deggertsen’s picture

deggertsen commented

Related issue #334630: Cache gravatar images locally

acouch’s picture

acouch commented
Title: Image URL munged » Image URL munged when user picture display style selected
Status: Active » Needs review
StatusFileSize
new image-url-munged-when-pic-display-style-selected-1689850-09.patch733 bytes

Attached is a patch. This keeps the module from trying to use image_style if there is no user pic, which is what screws up the image.

Status: Needs review » Needs work

The last submitted patch, image-url-munged-when-pic-display-style-selected-1689850-09.patch, failed testing.

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Forgot to mark URL as code.

andreymaximov’s picture

andreymaximov commented
Version: 7.x-1.1 » 7.x-1.x-dev
Issue summary: View changes
StatusFileSize
new gravatar-image_url_munged_when_pic_display_style_selected-1689850-11.patch735 bytes
new interdiff.txt364 bytes

#9 Patch has error fixed here.

... $style = variable_get('user_picture_style', '') && $account->picture ...

Code leads to $style taking incorrect value.

andreymaximov’s picture

andreymaximov commented
Status: Needs work » Needs review

The last submitted patch, 11: gravatar-image_url_munged_when_pic_display_style_selected-1689850-11.patch, failed testing.

andreymaximov’s picture

andreymaximov commented

11: gravatar-image_url_munged_when_pic_display_style_selected-1689850-11.patch queued for re-testing.

The last submitted patch, 11: gravatar-image_url_munged_when_pic_display_style_selected-1689850-11.patch, failed testing.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Nice catch!

acouch’s picture

acouch commented
StatusFileSize
new gravatar-image_url_munged_when_pic_display_style_selected-1689850-17.patch763 bytes

Attached is a patch that makes sure that the account picture exists before asking for it. This fixes an error that occurred while adding a new user.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 17: gravatar-image_url_munged_when_pic_display_style_selected-1689850-17.patch, failed testing.

deggertsen’s picture

deggertsen commented
Status: Needs work » Needs review
StatusFileSize
new gravatar-image_url_munged_when_pic_display_style_selected-1689850-19.patch763 bytes

Patch wouldn't apply to dev. Here's a new patch that does.

Status: Needs review » Needs work

The last submitted patch, 19: gravatar-image_url_munged_when_pic_display_style_selected-1689850-19.patch, failed testing.

deggertsen’s picture

deggertsen commented
Status: Needs work » Needs review

Is something wrong with the test bot? I swear I applied the patch to the correct dev...

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented

19: gravatar-image_url_munged_when_pic_display_style_selected-1689850-19.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 19: gravatar-image_url_munged_when_pic_display_style_selected-1689850-19.patch, failed testing.

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented

The error is: Output: [ERROR: No valid tests were specified.].

Filing this issue: #2218283: gravatar.test missing from .info file