Multiple registration mail if "Visitors, but administrator approval is required" [#2265459]

On a fresh install, i created 2 profile types (one and two) and 2 registration steps (first and second). Profile "one" is assigned to step "first" and profile "two" is assigned to step "second". When the registration is complete, i receive 2 mails (duplicates) of the registration process. The account creation setting is on "Visitors, but administrator approval is required". If i switch it to "Visitors", i receive only one mail.

How can i fix the mail duplication?

Comment	File	Size	Author
#2	multiple_registration-2265459-2.patch	1.79 KB	purushotam.rai

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

iaminawe CreditAttribution: iaminawe commented 29 December 2014 at 19:37

I can confirm this problem - any help would be greatly appreciated. Its the last bug I have to squish before launching this site.

Thanks in advance for any advice, patches or suggestions

Comment #2

purushotam.rai CreditAttribution: purushotam.rai at QED42 commented 22 January 2016 at 22:59

File	Size
multiple_registration-2265459-2.patch	1.79 KB

Hi guys,

I tried debugging the cause of this issue and after some efforts i found that after solving #2504791: Step's overwrites submit handlers from other modules (including LoginToboggan) issue, the submit handler functions have been arranged properly and hence we just now do not need to trigger mail explicitly from step module so removing the alters performed and mail trigger action from the step module solved the problem.

I have tested this patch and it works fine for me. It would be very great if someone else also tests it asap.

Would be releasing new stable version of Step Module soon.

Thanks,
Purushotam

Comment #3

purushotam.rai CreditAttribution: purushotam.rai at QED42 commented 22 January 2016 at 23:00

Status:

Active

» Needs review

Comment #4

sumitmadan CreditAttribution: sumitmadan at QED42 commented 23 January 2016 at 14:43

Status:

Needs review

» Reviewed & tested by the community

Looks good to me. :)

Comment #5

23 January 2016 at 17:15

purushotam.rai committed fbefef5 on 7.x-1.x

Issue #2265459 by purushotam.rai: Multiple registration mail if "...

Comment #6

purushotam.rai CreditAttribution: purushotam.rai at QED42 commented 23 January 2016 at 17:22

Status:

Reviewed & tested by the community

» Fixed

Code pushed at 7.x-1.x Dev Branch. If no issues found further would be releasing 7.x-2.x stable release very soon as this is the last bug of the module as of now.