?
Referenced as Pull Request for: # 5594
avatar phalouvas
phalouvas
30 Dec 2014

Steps to reproduce the issue

From back end
In Users component options have:

  • allow user registration
  • send password to NO In plugins->user->joomla options disable emails notification during registration.

Expected result

When a user register not to get any notification email.

Actual result

The email notification is send not matter what.

System information (as much as possible)

Joomla 3.3.6

Additional comments

The problem found to be on file:
/joomla_path/components/com_users/models/registration.php
line 502. You must change this line with below:

$user_plugin = JPluginHelper::getPlugin('user', 'joomla');
$mail_to_user = json_decode($user_plugin->params)->mail_to_user;
if ($mail_to_user) {
$return = JFactory::getMailer()->sendMail($data['mailfrom'], $data['fromname'], $data['email'], $emailSubject, $emailBody);
} else {
$return = true;
}

I will try to put the fix in Git.

avatar phalouvas phalouvas - open - 30 Dec 2014
avatar phalouvas phalouvas - reference | 64fbd24 - 30 Dec 14
avatar PhilETaylor PhilETaylor - reference | 479ffb2 - 3 Jan 15
avatar PhilETaylor
PhilETaylor - comment - 3 Jan 2015

I was able to replicate this, and after testing your proposed solution, I mangled it to create foundation of a merge-able PR #5594

avatar brianteeman brianteeman - change - 3 Jan 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-01-03 00:11:06
Closed_By brianteeman
avatar brianteeman brianteeman - close - 3 Jan 2015
avatar zero-24 zero-24 - close - 3 Jan 2015
avatar brianteeman brianteeman - close - 3 Jan 2015
avatar zero-24 zero-24 - change - 7 Jul 2015
Labels Added: ?
avatar itsam
itsam - comment - 19 Nov 2015

The issue still remains on 3.4.5

avatar Bakual
Bakual - comment - 19 Nov 2015

@itsam Check the "User - Joomla!" plugin. The notification is controlled there.

avatar itsam
itsam - comment - 19 Nov 2015

@Bakual Thanks for the prompt reply. Yes I did and I believe no matter the choice the notification email is sent anyway!

avatar Bakual
Bakual - comment - 20 Nov 2015

If you believe it's still an issue, please open a new issue instead of writing on a closed one.
Then it can be properly tested and confirmed.

Add a Comment

Login with GitHub to post a comment