There is a bug when editing the account. If you are using modules that modify URL (e.g. me aliases).

In file logintoboggan.module not quite correctly loaded user account:

      $account = user_load(array('uid' => arg(1)));

If you enable the module "me aliases", then the arg(1) == 'me' and have error.

Applying patch to fix.

CommentFileSizeAuthor
#2 logintoboggan-803422.patch890 bytes1sp
logintoboggan-6.x-1.7-uid.patch631 bytesazovsky

Comments

hunmonk’s picture

hunmonk commented
Title: array_keys() [function.array-keys]: The first argument should be an array » use uid from form instead of arg() to determine uid
Priority: Critical » Normal
Status: Closed (won't fix) » Needs review
1sp’s picture

1sp commented
StatusFileSize
new logintoboggan-803422.patch890 bytes

The $form['_account']['#value'] does contains the desired user object. so no need of explicit user_load.

Attaching an alternate patch.

hunmonk’s picture

hunmonk commented
Status: Needs review » Postponed (maintainer needs more info)

is there a $form['#account'] in the form array there? if so, i believe that's the one we'd want to use.

1sp’s picture

1sp commented
Status: Postponed (maintainer needs more info) » Needs review

I couldn't find $form['#account'] in the form array, there is only $form['_account'] as stated earlier in http://drupal.org/node/803422#comment-2993182

hunmonk’s picture

hunmonk commented
Status: Needs review » Fixed

no need to cast _account to an object -- it's always created as one by core.

patch tests out well. committed to 5.x-1.x-dev and 6.x-1.x-dev -- 7.x does not have this issue.

Status: Fixed » Closed (fixed)
Issue tags: -bug, -array_keys, -me aliases, -arg

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

Issue tags: +, +array_keys, +me aliases, +arg

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.