? Pending
Pull Request for # 9768

User tests: Successful: Unsuccessful:

avatar anibalsanchez
anibalsanchez
8 Apr 2016

Pull Request for Issue #9768.

Summary of Changes

Send Test Mail button does not correctly evaluate "SMTP Authentication" radio. Then, mails are sent via SMTP even when "SMTP Authentication" is disabled.

Fix: smtpauth : $('input[name="jform[smtpauth]"]:checked').val(),

Testing Instructions

  1. Fill all SMTP fields in Mail Settings for a server that requires authentication
  2. Test it, click on 'Send Test Mail' button to receive an email
  3. Disable "SMTP Authentication" radio
  4. Test it, click on 'Send Test Mail' button ... email must not be received
avatar anibalsanchez anibalsanchez - open - 8 Apr 2016
avatar anibalsanchez anibalsanchez - change - 8 Apr 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Apr 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 8 Apr 2016
Category Administration
avatar brianteeman brianteeman - change - 8 Apr 2016
Title
#9768 send test mail button  ajax call has always smtpauth = 1
send test mail button  ajax call has always smtpauth = 1
Rel_Number 0 9768
Relation Type Pull Request for
avatar brianteeman brianteeman - change - 8 Apr 2016
Title
#9768 send test mail button  ajax call has always smtpauth = 1
send test mail button  ajax call has always smtpauth = 1
avatar brianteeman
brianteeman - comment - 8 Apr 2016

Not sure its quite correct
If I set smtp and authentication yes and invalid username and password then i correctly get SMTP Error: Could not authenticate.
BUT
1. If i leave username and password empty i get the success message
2. If i enter a fake username and leave password empty I get the success message

avatar anibalsanchez
anibalsanchez - comment - 8 Apr 2016

@brianteeman The problem is found when your fill all SMTP credentials and keep "SMTP Authentication"=No... it applies the credentials even when it is disabled.

In this PR, I have modified the code to really check for the "SMTP Authentication" radio value.

An alternative solution could be to just remove the useless "SMTP Authentication" radio. If there are SMTP credentials, they are applied without checking for a radio.

avatar smz
smz - comment - 8 Apr 2016

An alternative solution could be to just remove the useless "SMTP Authentication" radio. If there are SMTP credentials, they are applied without checking for a radio.

This! (and, if you wish, password displayed on showon of the username...)

avatar smz
smz - comment - 8 Apr 2016

... but probably "showon not empty text field" is not possible, or is it?

avatar anibalsanchez
anibalsanchez - comment - 11 Apr 2016

The aim of this issue is to fix the user interface bug.

avatar andrepereiradasilva
andrepereiradasilva - comment - 12 Apr 2016

could you solve the conflicts now the other one as been merged?

avatar anibalsanchez
anibalsanchez - comment - 13 Apr 2016

Please, close this one as I have submitted an updated PR for testing - #9904

@andrepereiradasilva @smz

avatar brianteeman brianteeman - change - 13 Apr 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-04-13 21:47:36
Closed_By brianteeman
avatar brianteeman brianteeman - close - 13 Apr 2016

Add a Comment

Login with GitHub to post a comment