himselfv

Merge this one too, please! (I don’t see it in dev)

> Hook: let filters reliably guess existing users (similar to how it’s done by email)
> http://boku.ru/files/wsl/commit-56b8b84

Here’s why it’s needed. I have a mixed OpenID/WPSL setup so say a user logs in with Google OpenID. This address is then linked with his account. Then he logins with the same address but through WP Social Login. I want to be able to figure out that it’s the same user.

For that, I wrote a plugin which hooks the “search for existing user”/”create new user”, and for certain providers either checks for associated OpenID or adds new OpenID association so that whatever way the user logs in, he ends up with the same account.

But it needs the above patch to work.

Updated links:

This one fixed the issue:
http://boku.ru/files/wsl/commit-e786223

Hook: run user data through filters before updating
http://boku.ru/files/wsl/commit-5b17842

Hook: let filters reliably guess existing users (similar to how it’s done by email)
http://boku.ru/files/wsl/commit-56b8b84

Fix: Do not call wsl_hook_process_login_alter_userdata twice, what if hooks do something permanent?
http://boku.ru/files/wsl/commit-5d73689

And these too please while we’re at it!)

http://boku.ru/temp/commit-5b17842
http://boku.ru/temp/commit-56b8b84
http://boku.ru/temp/commit-5d73689

(They add two hooks and fix _alter_userdata hook so it’s not called twice)

Also I have no idea what to do with login generation so please you deal with it 🙂
I’ll fix the non-checking-for-errors on wp_insert_user.

Apparently there IS a check in wsl_process_login_create_user, but the WP_Error value is still returned as $user_id after printing out error. And wsl_process_login_reauth which uses wsl_process_login_create_user doesn’t check whether $user_id it recieves is not WP_Error by chance.