This may be specific to the project I'm on, but I think this is good practice anyway.

If other code changes the user settings form, such as adding very light (-100) items, the FBConnect items still end up at the top. Giving the FBConnect items explicit 'normal' weights - eg starting at zero and going up - fixes this for me.

CommentFileSizeAuthor
#2 1290612-2-set-explicit-weights.patch2.31 KBdarrylh
#1 1290612-set-explicit-weights.patch1.89 KBdarrylh

Comments

darrylh’s picture

darrylh commented
StatusFileSize
new 1290612-set-explicit-weights.patch1.89 KB
darrylh’s picture

darrylh commented
StatusFileSize
new 1290612-2-set-explicit-weights.patch2.31 KB

Better patch, one that actually applies (*sigh*)

doka’s picture

doka commented
Status: Active » Needs work

The correct link to enable user picture support is admin/config/people/accounts.

So change the line 354 in fbconnect.pages.inc from this one:

     $form['fb_avatar']['#description'] = t('Enable user picture support in !link', array(
      '!link' => l(t('User settings'), 'admin/user/settings'),
    ));

into that:

     $form['fb_avatar']['#description'] = t('Enable user picture support in !link', array(
      '!link' => l(t('User settings'), 'admin/config/people/accounts'),
    ));
giorgio79’s picture

giorgio79 commented
Status: Needs work » Closed (duplicate)

#1296370: Code Cleanup and Fixes