Registration Module sends Activation E-Mail even if Send activation e-mail is NOT checked when Re-Registering

[mvscheidt]

Steps to Recreate the Issue in the Demo:

1. Create Registration Module with the following Settings
   
2. Add Registration Module somewhere on https://demo.contao.org/en/user.html
3. Go to https://demo.contao.org/en/user.html and register with an E-Mail.
4. You will land on the page https://demo.contao.org/en/user.html again without any special message, no Mail will be send.
5. Now try to Re-register again with the same E-Mail.
6. You will land on the page https://demo.contao.org/en/user.html again and the module has output a message "The activation mail has been re-sent to your e-mail address."

---

6. should not happen in my opinion and seems like a bug. Reason is because if I do not want to send any activation E-Mails this means I want to activate the users manually.

I read the Code and I think all that needs to be done is to change this

		// Check for a follow-up registration (see #7992)
		if (\Input::post('email', true) != '' && ($objMember = \MemberModel::findUnactivatedByEmail(\Input::post('email', true))) !== null)
		{
			$this->resendActivationMail($objMember);

			return;
		}

to this

		// Check for a follow-up registration (see #7992)
		if ($this->reg_activate && \Input::post('email', true) != '' && ($objMember = \MemberModel::findUnactivatedByEmail(\Input::post('email', true))) !== null)
		{
			$this->resendActivationMail($objMember);

			return;
		}

However I haven't tested what happens then, so maybe some additional changes are required.

Maybe in this case a message like Administrator has not activated your account yet would be appropriate, but not sure what would be best here seeing how this might require new translations to be made.

[leofeyer]

I think your fix is correct.

[leofeyer]

Fixed in 8c65ec8. Thank you @mvscheidt.