? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
20 Oct 2018

Steps to reproduce the issue

Closes #22711

Enable the sending of Action Log entries to a Super Admin by Mail in Joomla 3.9
Set your Joomla Global Timezone to be something the other side of the world
Generate a load of emails

Expected result

Any reference to the date/time an event occured should be translated to the Joomla Global Configuration Timezone/offset

Actual result

The date/time right now is UTC/GMT, but a Joomla Super Admin the other side of the world (with a server another third of the way around the world) is confused and assumes its local time, so would be best when giving the date/time to tell the recipient WHAT timezone that timestamp relates to.

screenshot 2018-10-19 at 00 54 31

avatar PhilETaylor PhilETaylor - open - 20 Oct 2018
avatar PhilETaylor PhilETaylor - change - 20 Oct 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Oct 2018
Category Administration
avatar PhilETaylor PhilETaylor - change - 20 Oct 2018
Title
Apply timezone to datetime in com_actionlogs emails
[com_actionlogs][3.9]Apply timezone to datetime in com_actionlogs emails
avatar PhilETaylor PhilETaylor - edited - 20 Oct 2018
avatar laoneo
laoneo - comment - 21 Oct 2018

This is actually sent in the timezone of the current logged in user. If the recipient is in a different timezone then the date is wrong.

avatar mbabker mbabker - change - 24 Oct 2018
Labels Added: ?
avatar mbabker
mbabker - comment - 24 Oct 2018

This is actually sent in the timezone of the current logged in user. If the recipient is in a different timezone then the date is wrong.

Changed this to:

  • Hardcode the time format (so the email doesn't have a different format based on the active user)
  • Hardcode the date/time string to UTC

Good to go here, please test

avatar mbabker mbabker - change - 24 Oct 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-10-24 21:58:09
Closed_By mbabker
avatar mbabker mbabker - close - 24 Oct 2018
avatar mbabker mbabker - merge - 24 Oct 2018

Add a Comment

Login with GitHub to post a comment