?
avatar brianteeman
brianteeman
16 Mar 2016

Steps to reproduce the issue

In global config Mail Settings set send mail to NO
Try and send a test email with the new Send Test Mail button

Expected result

JLIB_MAIL_FUNCTION_DISABLED="The mail() function has been disabled and the mail can't be sent."

Actual result

JLIB_MAIL_FUNCTION_OFFLINE="The mail function has been temporarily disabled on this site, please try again later."

avatar brianteeman brianteeman - open - 16 Mar 2016
avatar brianteeman brianteeman - change - 16 Mar 2016
Milestone Added:
avatar brianteeman brianteeman - change - 16 Mar 2016
Milestone Added:
avatar wilsonge wilsonge - change - 16 Mar 2016
Milestone Removed:
avatar andrepereiradasilva
andrepereiradasilva - comment - 16 Mar 2016

If you voluntarily disabled the site to not send mails. The mail function is offline.

JLIB_MAIL_FUNCTION_OFFLINE="The mail function has been temporarily disabled on this site, please try again later."

If your server can't send mails because can't use mail function then is disabled.

JLIB_MAIL_FUNCTION_DISABLED="The mail() function has been disabled and the mail can't be sent."

Maybe the text are not the better, but i think the behavior is correct.

This is processed in https://github.com/joomla/joomla-cms/blob/staging/libraries/joomla/mail/mail.php#L73-L111

avatar brianteeman
brianteeman - comment - 17 Mar 2016

If you voluntarily disabled the site to not send mails. The mail function is offline.
trying again later will have zero effect at all because you have chosen to disable the mail functonality


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Mar 2016

So, the only correction needed is the language variable.

JLIB_MAIL_FUNCTION_OFFLINE="The mail function has been temporarily disabled on this site, please try again later."

Maybe?

JLIB_MAIL_FUNCTION_OFFLINE="The mail function has been disabled by the administrator."

avatar shubhamnba2009
shubhamnba2009 - comment - 17 Mar 2016

What is the location of jlib_mail_function_offline language variable in language directory? Thanks in advance


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Mar 2016

/language/en-GB/en-GB.lib_joomla.ini
and
/administrator/language/en-GB/en-GB.lib_joomla.ini

avatar brianteeman
brianteeman - comment - 17 Mar 2016

NO the language string doesnt need changing - the code is calling the wrong message as stated in the first post


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Mar 2016

i don't agree with you on this one brian.

IMHO:

  • disabled: the system cannot send a mail, because the is no mail function in the system
  • offline: the super user decided to put the mail function offline, the same logic as site offline.
avatar shubhamnba2009
shubhamnba2009 - comment - 17 Mar 2016

Than we are only left with making correction in the language variable.
Or Brian u have any opinion on this?

avatar joo7
joo7 - comment - 17 Mar 2016

I'm with brianteeman

disabled: system cannot sent, because super user DISABLED function = voluntarily/wanted

offline: system cannot sent, because of other (technical) reasons (site local, no internet connection...) = unwanted

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Mar 2016

and when you set the site offline in global configuration ... " system cannot sent, because of other (technical) reasons (site local, no internet connection...) = unwanted"

what i mean is offline for one thing (mail server) should have the same treatment as offline for the other thing (site). consistency

avatar joo7
joo7 - comment - 17 Mar 2016

oh, sorry, true, we need consistency.
hm, some super user might not be aware when setting site offline, sent mail is disabled too...

avatar joo7
joo7 - comment - 20 Mar 2016

If we leave the language-strings unchanged for consistency and since we don't know, if the function is temporarily or permanent offline and/or temporarily or permanent disabled, maybe the text can be formulated like this?

JLIB_MAIL_FUNCTION_OFFLINE="The mail function is currently disabled by an administrator in the global configuration. Please try again later or contact the webmaster/administrator of this website."
(maybe drop " in the global configuration")

JLIB_MAIL_FUNCTION_DISABLED="The mail() function is currently not available. Please try again later or contact the webmaster/administrator of this website."

avatar Kubik-Rubik
Kubik-Rubik - comment - 21 Mar 2016

@brianteeman Thank you for the finding. Could you please check the proposed English text? This will go into 3.5.1.

avatar Kubik-Rubik Kubik-Rubik - change - 21 Mar 2016
Milestone Added:
avatar Kubik-Rubik Kubik-Rubik - assigned - 21 Mar 16
avatar brianteeman
brianteeman - comment - 21 Mar 2016

Changing the text is not the solution - as stated in the first post the code is giving the wrong error message

avatar mbabker
mbabker - comment - 21 Mar 2016

I'm inclined to agree with the others here that the text just needs to be cleaned up. Remove the "please try again later" part of the offline string. The other choice is in JMail::Send() use the disabled string and phase out the offline string completely.

avatar joo7
joo7 - comment - 21 Mar 2016

Ok, if "Send Mail" is set to "No" using "Send Test Mail" will result in:

JLIB_MAIL_FUNCTION_OFFLINE="The mail function is currently disabled by an administrator in the global configuration."

avatar joo7
joo7 - comment - 21 Mar 2016

In case this might be confusing since a user using the "Send Test Mail" button has to be an administrator and is in the global configuration already:

JLIB_MAIL_FUNCTION_OFFLINE="The 'Send Mail' function is currently disabled here in the 'Mail Settings' by an administrator."

avatar brianteeman brianteeman - change - 22 Mar 2016
Milestone Removed:
avatar brianteeman brianteeman - change - 22 Mar 2016
Milestone Added:
avatar brianteeman brianteeman - change - 22 Mar 2016
Milestone Added:
avatar brianteeman brianteeman - change - 22 Mar 2016
Milestone Removed:
avatar sovainfo
sovainfo - comment - 28 Apr 2016

Suggest to solve the real issue here of poor UI design. The button should not be available when Send Mail is set to No!

avatar brianteeman
brianteeman - comment - 28 Apr 2016

How about instead of suggesting something you submit a pull request

On 28 April 2016 at 15:50, sovainfo notifications@github.com wrote:

Suggest to solve the real issue here of poor UI design. The button should
not be available when Send Mail is set to No!


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#9446 (comment)

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar joo7
joo7 - comment - 28 Apr 2016

@sovainfo good point, but is it UI if your mails stopped working, you go backend in global config and your test-button is gone? how would he know some other admin set Send Mail to No voluntarily?

avatar sovainfo
sovainfo - comment - 28 Apr 2016

@brianteeman Considering the differences of opinion on this issue not going to waste time on that.
@joo7 Same question for someone changing SMTP to PHPMail. With multiple Super Loosers there is a big chance things are not as you expected. Sounds obvious to me, when troubleshooting! First thing to check when mails stopped working is to see whether sending mails is enabled! If your first reaction is to test it, the inability to do that is leading you to the thing you should have done first.

avatar joo7
joo7 - comment - 28 Apr 2016

@sovainfo right, things obvious to an experienced developer might not be obvious to beginners - isn't that what UX/UI is about? Who do we want to adress? I think we should give as much help and info to all. We should rather risk to repeat more than obvious info than do devtalk in a notice - from a viewpoint of a beginner :) He might not know where to look first. Do we want him to depend on developers? I don't think we should leave the "multiple Super Loosers" alone...

avatar sovainfo
sovainfo - comment - 28 Apr 2016

Rest my case! Glad I didn't waste my time!

avatar joo7
joo7 - comment - 28 Apr 2016

@sovainfo Sorry, I still think you make a good point. Maybe grey button with tip: "'Test Mail' is disabled because 'Send Mail' is set to 'No'"?

avatar sovainfo
sovainfo - comment - 28 Apr 2016

@joo7 No problem, we are both entitled to our own opinions. I'll leave it up to Kubik-Rubik to decide and implement whatever is decided upon. You are the only one that gave his opinion on my proposal. Considering we disagree, I'll leave it at that.

avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar brianteeman brianteeman - change - 9 May 2016
Milestone Removed:
avatar brianteeman brianteeman - change - 10 May 2016
Status New Confirmed
avatar brianteeman brianteeman - change - 3 Aug 2016
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2016-08-03 14:45:10
Closed_By brianteeman
avatar brianteeman brianteeman - close - 3 Aug 2016
avatar brianteeman
brianteeman - comment - 3 Aug 2016

Closed as we have a PR #11416


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

Add a Comment

Login with GitHub to post a comment