The facebook picture is showing on the Profile, but on all other pages it is missing, and I get this error:

Warning: Illegal string offset 'fb_avatar' in fbconnect_theme_user_picture_override() (line 935 of /var/www/studentango/sites/all/modules/contrib/fbconnect/fbconnect.module).

I'm working with an up to date Drupal 7.19.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SebaSOFT’s picture

suscribing

rashmiarora’s picture

I am facing the same problem. Found any solution ??

lhridley’s picture

Status: Active » Needs review
FileSize
668 bytes

Patched to fix with the following:

Replace line fbconnect.module:line 935

$account->data['fb_avatar'] = @$user_data['fb_avatar'];

with

if(isset($user_data['fb_avatar']) && isset($account->data)) {
  $account->data['fb_avatar'] = @$user_data['fb_avatar'];
} 

Patch is attached.

jcisio’s picture

Title: No user picture showing in poss » No user picture showing in post
Status: Needs review » Fixed

Thanks. Committed and pushed.

Status: Fixed » Closed (fixed)

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

Erik Frèrejean’s picture

The provided patch doesn't seem to fix this fully. The problem is caused because $account->data is a serialized array, but the module tries to alter data directly by accessing the data as if it is an array.
I've created an additional patch for this issue that will handle the serialization.

Erik Frèrejean’s picture

Status: Closed (fixed) » Needs review
Erik Frèrejean’s picture

Apperently the serialized string only gets passed in the first request after a cache clear. Updated the patch to only unserialize if there is actually a string in $account->data

  • jcisio committed 749d97e on authored by lhridley
    Issue #1917934 by lhridley | BParticle: Fixed No user picture showing in...