After installing alt_login, I've received the following warning:
Notice: Undefined index: name in alt_login_user_validate() (line 157 of .../alt_login/alt_login.module).
while visiting simplenews newsletter subsription form (at /user/me/edit/simplenews).

I've checked and see that $form_state['values'] doesn't return ['name'] value, so I've added the similar conditional set as in a line below (for $alt_login). The patch is attached...

CommentFileSizeAuthor
alt_login.diff557 bytesluti

Comments

hunmonk’s picture

hunmonk commented
Project: Alternative login ID & display names » Simplenews
Version: 7.x-1.0 » 7.x-1.x-dev

why is simplenews calling the validation for user_register_form? this seems wrong. also, 'name' is a base form element of the registration form, it seems reasonable to assume that it's present.

moving to simplenews for resolution.

simon georges’s picture

simon georges commented

Simplenews validate a user form, I don't think user_validate is only called at registration, but I think it's used during profile edition too. Let me look further into it.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Project: Simplenews » Alternative login ID & display names
Status: Active » Needs review

Simplenews exposes itself as a user edit category, alt login needs to check the category and only do it's stuff on the default account category.

The same will happen if you e.g. have profile.module enabled and set up profile categories with it.