?
Referenced as Pull Request for: # 9803
avatar anibalsanchez
anibalsanchez
6 Apr 2016

Steps to reproduce the issue

  1. Fill all SMTP fields in Mail Settings for a server that requires authentication
  2. But, don't enable SMTP Authentication

Expected result

Mails are not sent via SMTP that requires authentication. Because, SMTP Authentication is not enabled.

Actual result

Mails are sent via SMTP, even when SMTP Authentication is disabled.

System information (as much as possible)

Any browser

Additional comments

The radio buton is simply queried but jQuery does not take into account that it is a radio button.

Fix: sendtestmail-uncompressed.js

jQuery(document).ready(function ($)
{
$('#sendtestmail').click(function ()
{
var email_data = {
smtpauth : $('input[name="jform[smtpauth]"]:checked').val(),
smtpuser : $('input[name="jform[smtpuser]"]').val(),
smtppass : $('input[name="jform[smtppass]"]').val(),
smtphost : $('input[name="jform[smtphost]"]').val(),
smtpsecure: $('select[name="jform[smtpsecure]"]').val(),
smtpport : $('input[name="jform[smtpport]"]').val(),
mailfrom : $('input[name="jform[mailfrom]"]').val(),
fromname : $('input[name="jform[fromname]"]').val(),
mailer : $('select[name="jform[mailer]"]').val(),
mailonline: $('input[name="jform[mailonline]"]:checked').val()
};

avatar anibalsanchez anibalsanchez - open - 6 Apr 2016
avatar anibalsanchez anibalsanchez - reference | 727edab - 6 Apr 16
avatar anibalsanchez
anibalsanchez - comment - 6 Apr 2016

Hi,

I tested the fix the uncompressed file. What are the steps to generate the compressed version?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9768.

avatar zero-24
zero-24 - comment - 6 Apr 2016

@anibalsanchez

Nothing special. Just one of the varius online js compressors is the last tool i know ;)

avatar anibalsanchez
anibalsanchez - comment - 8 Apr 2016

Updated from #9685. This issue is solved simply adding :checked to smtpauth value read.

avatar brianteeman brianteeman - change - 8 Apr 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-04-08 09:09:30
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 8 Apr 2016

Closed as we have a PR for testing - #9803


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9768.

avatar brianteeman brianteeman - close - 8 Apr 2016

Add a Comment

Login with GitHub to post a comment