I have traced the user profile image not resizing to OpenID Profile (I think)

i tried to recreate the config created by the Omniauth Relying Party feature and it requires OpenID Profile

when I enable either the feature or the module alone the user profile images no longer resize

makes me think it might be related to a recent bug in ImageCache or ImageCache Profiles ?

http://drupal.org/node/1109804

on OpenAtrium installs without enabling OpenID Profiles the user profile uploads work fine

latest versions of modules from contrib, sandbox and gitorious (omniauth);
Omniauth Relying Party 6.x-1.0-alpha1

CommentFileSizeAuthor
#7 sync-user-profile-pictures.patch6.53 KBmarkwk
#1 user-image-openid.png298.62 KBniccolox
#1 whacky.png168.91 KBniccolox
#1 working.png172.95 KBniccolox

Comments

niccolox’s picture

niccolox commented
StatusFileSize
new working.png172.95 KB
new whacky.png168.91 KB
new user-image-openid.png298.62 KB

attached screenshots

I am guessing its either Strongarm or a versioning issue with one of the ImageCache types modules

will come back to it next week

niccolox’s picture

niccolox commented

looking at the code I guess its something to do with 

/**
 * Process variables for user-picture.tpl.php.
 * This is a copy from user.module, only modified to allow remote pictures
 */
function openid_profile_preprocess_user_picture(&$variables)

will check if user-picture.tpl.php is any different between stock D6 and Open Atrium .. maybe thats it

niccolox’s picture

niccolox commented
Title: User Profile Image Resize Breaks when installing OpenID Profile when part of Omniauth on OpenAtrium » User Profile Image Resize Breaks when installing OpenID Profile on OpenAtrium

removed the Omniauth reference from title, am now testing with installation of modules, not using feature or profile except as reference

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Status: Active » Fixed

Removed the functionality to synchronize the user picture.
It needs some more thinking / work to implement it right.
That should fix the behavior described in the initial issue.

Status: Fixed » Closed (fixed)

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

markwk’s picture

markwk commented
Status: Closed (fixed) » Active

Have similar problem. I'm currently working on it. A bit of a difficult one this syncing of user profile pictures. You are right that it needs some rethinking.

I'm currently setting it up so that the hub site has the imagecache profile presets setup in a way that parallels what's needed on the atrium sites.

Thoughts on this approach?

markwk’s picture

markwk commented
StatusFileSize
new sync-user-profile-pictures.patch6.53 KB

Here's the patch to fix this problem, at least in my case. Not sure if this is a universal fix though.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Category: Bug report » Feature request
Issue summary: View changes