It would be nice if passwordless could detect the presence of the multiple_email module, and allow users to use any of their configured e-mail addresses to log in. (This would benefit those who have separate work/home addresses that may not be able to access one of their e-mail accounts during the day.)
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | passwordless-multiple_email.patch | 1.5 KB | bledbetter |
| #1 | passwordless-multiple_email.patch | 1.47 KB | bledbetter |
Comments
Comment #1
bledbetter commentedAs promised, here is a patch that enables Passwordless to query the multiple_email module for additional user e-mail addresses, if and only if the multiple_email module is installed.
I've tested it on my system and it seems to work. Will need to regress it to make sure it continues to function if multiple_email is not present.
Comment #2
bledbetter commentedPatch updated - multiple_email returns FALSE if the user account cannot be found. Added line 221 to transform that into an empty Array() that will be consumed by the downstream process (resulting in the "... is not a registered e-mail address on this website" message).
Comment #4
antiorario commentedThanks for the patch! I've cleaned up the code a bit (coding standards and a couple of possible errors), and made the logic more compact. It's in dev, so you can test it before I create a new release.