Add option to only validate new registrations and address changes, exclude existing addresses (do not block user profile save)

Raising to Major, I definitely agree this shouldn't be happening by default.

I'd like existing sites to retain the "wrong" behaviour though rather than changing it for them, perhaps an update hook? I'll make sure the release note has a big note in it.

I won't get to this tonight, or possibly for a while. MRs welcome.

Any feedback on these new config settings?

LOL wow, uploaded the wrong screenshot, this is the right one:

I agree this is major. Asking @Grevil to work on this. We can't use this module until it's fixed, it blocks existing users from editing their accounts. They can not even change their passwords.

@Anybody I made an MR, please try grabbing a patch from that? It only needs tests to be committed but feedback on the UI would be gratefully accepted. If @Grevil could sort out tests we'll have a release in no time.

I think the UI is good! The only point against "checkboxes", would be that we disable the module functionality through unchecking both of these. But I guess, there is a use case hidden in that feature as well! :P

So yea, LGTM! And code looks good as well! I'll quickly add schema and install file entries, create an update hook, and add / refactor (existing) tests!

Alright, all finished! Please review!

Wonderful, thank you.

I've just changed the update hook to set both to true for existing sites so they retain the functionality they already have.

Release made, hope that unblocks you :)

While I was writing it up I realised we needed a follow-up for validation on email change: [#3441666)

Great stuff! Glad I could help! 👍

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