Please activate JavaScript in your browser.
?
Success
Success
continuous-integration/travis-ci/pr
The Travis CI build passed
Details
Success
JTracker/HumanTestResults
Human Test Results: 2 Successful 0 Failed.
Details
User tests:
Successful:
Unsuccessful:
Pull Request for Issue #9768 / #9803
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(),
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.
Steps to reproduce the issue
Fill all SMTP fields in Mail Settings for a SMTP server that requires authentication
But, don't enable SMTP Authentication
Send Test Mail button sends emails even when the radio is disabled
Testing Instructions
Fill all SMTP fields in Mail Settings for a SMTP server that requires authentication
Test it, click on 'Send Test Mail' button to receive an email
Disable "SMTP Authentication" radio
Test it, click on 'Send Test Mail' button ... email must not be received and error is shown about the failed authentication
anibalsanchez
-
open
-
13 Apr 2016
anibalsanchez
-
change
-
13 Apr 2016
joomla-cms-bot
-
change
-
13 Apr 2016
andrepereiradasilva
-
test_item
-
14 Apr 2016
-
Tested successfully
henkrijneveld
-
test_item
-
18 Apr 2016
-
Tested successfully
brianteeman
-
change
-
19 Apr 2016
Status
Pending
⇒
Ready to Commit
joomla-cms-bot
-
change
-
19 Apr 2016
brianteeman
-
change
-
19 Apr 2016
brianteeman
-
change
-
19 Apr 2016
brianteeman
-
change
-
19 Apr 2016
Category
⇒
Administration
rdeutz
-
merge
-
19 Apr 2016
rdeutz
-
close
-
19 Apr 2016
Status
Ready to Commit
⇒
Fixed in Code Base
Closed_Date
0000-00-00 00:00:00
⇒
2016-04-19 11:27:52
Closed_By
⇒
rdeutz
Labels
rdeutz
-
close
-
19 Apr 2016
rdeutz
-
merge
-
19 Apr 2016
joomla-cms-bot
-
close
-
19 Apr 2016
joomla-cms-bot
-
change
-
19 Apr 2016
Add a Comment
Login with GitHub to post a comment
Ok. I tested this.
Without patch i get. Mail is sent (i think is sent with php mail fallback - see #9680) and i get:
With patch i get. Mail is not sent and i get:
So is this a success?