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.)

Comments

bledbetter’s picture

StatusFileSize
new1.47 KB

As 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.

bledbetter’s picture

StatusFileSize
new1.5 KB

Patch 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).

  • Commit ce9b05c on 7.x-1.x authored by bledbetter, committed by antiorario:
    Issue #2279259 by bledbetter: Added multiple_email integration.
    
antiorario’s picture

Version: 7.x-1.6 » 7.x-1.x-dev
Issue summary: View changes
Status: Active » Fixed

Thanks 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.

Status: Fixed » Closed (fixed)

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