The module pulls in the user's Facebook picture but when the size "Normal (200px wide)" is chosen, it displays 100px wide.

Selecting "Small (100px) wide produces a 50px wide image.

Thumb (50px wide) produces a missing image icon.

Square (50px by 50x) seems to work fine.

CommentFileSizeAuthor
#6 user_image_1005646.patch6.99 KBAnonymous (not verified)

Comments

g000fy’s picture

g000fy commented

I figured that out too. It seems fb has changed definition of picture sizes.

Normal is now 100px and large is 200px. I manually changed fbconnect.admin.inc to fix it.

$options_pic_size = array(
    'thumb' => t('Thumb (50px wide)'),
    'normal' => t('Normal (100px wide)'),
    'large' => t('Large (200px wide)'),
    'square' => t('Square (50px by 50px)'),
  );
alaugh’s picture

alaugh commented

Thanks g000fy. I really appreciate it. That did the trick. My next step is to now get the 200px size to display in user profiles when using Author Pane and Advanced Profile Kit. If you have any insight into this one, please let me know.

Thanks again for the response and help.

alaugh’s picture

alaugh commented

Thanks g000fy. I really appreciate it. That did the trick. My next step is to now get the 200px size to display in user profiles when using Author Pane and Advanced Profile Kit. If you have any insight into this one, please let me know.

Thanks again for the response and help.

ric_rac’s picture

ric_rac commented

i also have this problem... where you able to get the 200px wide image to display on profiles?

mrp40’s picture

mrp40 commented

From http://developers.facebook.com/docs/reference/api/

You can specify the picture size you want with the type argument, which should be one of square (50x50), small (50 pixels wide, variable height), and large (about 200 pixels wide, variable height): http://graph.facebook.com/profileid/picture?type=large.

As g00fy did above, I fixed the problem also by simply changing fbconnect.admin.inc:146 to

  $options_pic_size = array(
    'square' => t('Square (50x50px)'),
    'small' => t('Small (50px wide, variable height)'),
    'large' => t('Normal (200px wide, variable height)'),
  );

This then matches the API description... ie, removes "normal" altogether.

Once changed you will need to go to /admin/settings/fbconnect/apperance and set the default sizes again. Seems to be working for me now.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs review
StatusFileSize
new user_image_1005646.patch6.99 KB

Patch for this issue and some other tidying up. It was diffed against the current Github repo.

FBConnect uses Graph API to insert profile images. Graph API currently supports these 4 image types:
1. square - 50px x 50 px thumbnail
2. small - 50px wide, variable height
3. normal - 100px wide, variable height
4. large - 180px wide, variable height
(Large was previously 200px. Newly uploaded facebook profile images are supplied at 180px width, whilst older images remain at 200px width)

Changed some description text in the admin form to be non gender specific.

Facebook logo on user pic is not available using Graph API. Removed the Facebook Logo checkbox from admin form and $logo parameter in functions.

theme_fbconnect_user_picture_override () does not check for 'access user profiles' before providing a link to user profiles. This will result in access denied errors. Changed the way images are themed to use core functions, theme('image'...) and link l() after checking for permissions. Removed $user_url parameter in functions as this is now provided in the l() function.

Tested on Firefox, Chrome, IE, Safari and all seems to work as expected :)

ps: apologies for the three 'no newline at end of file' in the patch, some kind of bug I'm having with my version of git :(