The first if() in first_login_user_login():

if ($account->data['first_login']['status'] && ($account->data['first_login']['status'] != FALSE || !isset($account->data['first_login']['status'])) && isset($account->uid)) {

seems to prevent first_login_set_data() from ever being run for a new user. I suggest changing it to:

if (isset($account->uid) && (!isset($account->data['first_login']['status']) || $account->data['first_login']['status'] != FALSE)) {

Comments

KarinaH’s picture

KarinaH commented

Hi Jeremy,

nice code snippet. I had the "Undefined index: first_login in first_login_user_login()..." in line 84 - which was exactly the line you suggestetd to change. I did it and it worked a charm for me.

Thanks for posting it!

afox’s picture

afox commented
Status: Active » Closed (fixed)

Thanks for the report. Updated the -dev branch for D7.

afox’s picture

afox commented
Issue summary: View changes

tweak layout