- 🇺🇸United States dbouman
I tested this patch and while it did prevent a second email, the issue was that when an admin creates an account and selects to notify the user, the email that gets sent out is not the correct email. It should be sending out the core email "Welcome (new user created by administrator)", but instead it is sending out the email generated by this module which at least in my case because we have email verification turned on contains a broken link.
I've attached the 2.x patch that worked for me, it sends out the user_registrationpassword_mail_notify if the account was not created by an administrator. If the account was created by an administrator it let's core figure out whether or not to send the core welcome email.
- 🇺🇸United States sidgrafix
Just ran into this..
+1 for patch 6, I agree with dbouman.
- First commit to issue fork.
- Merge request !13Fixed wrong email issue related to account creation by admin → (Open) created by shivam_tiwari
- 🇮🇳India shivam_tiwari
I worked on it. At the time of creating an user by admin account, got error related to this issue: https://www.drupal.org/project/user_registrationpassword/issues/3467248 🐛 TypeError: Return value must be of type array, bool returned Active
After fixing that, checked here and it worked fine. Added my changes on MR 13, because previous MR code is outdated. Moving this issue to Needs Review - 🇮🇳India divyansh.gupta Jaipur
I have reviewed the issue and the MR-13 successfully applied for me and now it is working as expected, also the changes looks good to me thus moving it to RTBC+!