User tests: Successful: Unsuccessful:
Pull Request for Issue #9768.
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(),
Status | New | ⇒ | Pending |
Labels |
Added:
?
|
Category | ⇒ | Administration |
Title |
|
||||||
Rel_Number | 0 | ⇒ | 9768 | ||||
Relation Type | ⇒ | Pull Request for |
Title |
|
@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.
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...)
... but probably "showon not empty text field" is not possible, or is it?
The aim of this issue is to fix the user interface bug.
could you solve the conflicts now the other one as been merged?
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-04-13 21:47:36 |
Closed_By | ⇒ | brianteeman |
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