[com_config] Fallback warning in send test mail (solves #9680)

[andrepereiradasilva]

Pull Request for Issue #9680.

Summary of Changes

Sometimes when sending a testmail the mail is sent even that you know the mail config is not correct.

This happens because JMail has some fallbacks (see, for instance, https://github.com/joomla/joomla-cms/blob/staging/libraries/joomla/mail/mail.php#L601-L627).

So this PR adds a fallback warning in global configuration "Send Test Mail" button result s users can notice that the mail is sent, but not with the configurations that the user chossed.

Before PR

After PR

Testing Instructions

1. Apply patch and go to global config
2. Clear browser cache
3. Test "Send Test Mail" with all possible mail configs you can remember.

I get the fallback message by using SMTP with SMTP Authentication and didn't type the SMTP Auth user or password (leave the fields empty). I this will not allow to connect to my SMTP server.

[zero-24]

I have tested this item ✅ successfully on a90a531

Works good. please remember on testing to clear your browser cache.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[andrepereiradasilva]

@brianteeman since you also add the original issue, can you test this one?

[andrepereiradasilva]

@brianteeman can i ask you to please test this one?

[brianteeman]

I would test it if I could recreate the scenario now but i cant

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[andrepereiradasilva]

ok @brianteeman thanks anyway.
anyone else can test this?

[AnishaTailored]

Hello,
I have tried with incorrect SMTP details (incorrect username and password) but I could not see the same message that you have shown in before PR section. Instead I am getting 'Test mail could not be sent.' and 'SMTP Error: Could not authenticate.'

I have tested this on Joomla! 3.6.0-alpha2-dev staging.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[gunjanpatel]

Unfortunately I am also not able to reproduce this issue before applying patch. I have followed instructions in the main issue. Like giving wrong port and wrong host but still not getting green message. Is there any other possible way to reproduce it?

Thanks.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[andrepereiradasilva]

Unfortunately I am also not able to reproduce this issue before applying patch. I have followed instructions in the main issue. Like giving wrong port and wrong host but still not getting green message. Is there any other possible way to reproduce it?

Set smtp authentication to yes, with user and password empty

[gunjanpatel]

Thanks @andrepereiradasilva Follow your comments, but still having the same and not able to reproduce it before applying patch. I am using latest staging branch with smtp authentication to yes, with user and password empty.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[andrepereiradasilva]

what error do you have with the last test case? Maybe you don't have php mailer working so there is not fallback.

[gunjanpatel]

I am getting the same error which @AnishaVora is getting.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[andrepereiradasilva]

so your SMTP fails and your fallback also fails.

To test this you have to php mail function working properly to have a fallback in case SMTP does not work.

Probably you don't have a correct sendmail path in your server.
Check if your php ini sendmail_path is correct: http://php.net/manual/en/mail.configuration.php#ini.sendmail-path

[AnishaTailored]

Hello,
I have tested this successfully with empty SMTP username and password (re-tested after installing sendmail). After applying the patch the following message is displayed.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[AnishaTailored]

I have tested this item ✅ successfully on a90a531

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[andrepereiradasilva]

nice thanks!

[andrepereiradasilva]

@brianteeman RTC?

[gunjanpatel]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[gunjanpatel]

I can also confirm the test successful as I was there while Anisha was testing this issue on her machine. 😄

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[wilsonge]

Conflicts please :)

[joomla-cms-bot]

This PR has received new commits.

CC: @AnishaVora, @zero-24

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[andrepereiradasilva]

conflicts fixed

[joomla-cms-bot]

This PR has received new commits.

CC: @AnishaVora, @zero-24

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10370.}

[andrepereiradasilva]

sorry for this late one, just one thing lost in the conflixcts fix
c41292d

[wilsonge]

It's ok :) Thanks for getting on it so fast!