? ? Pending

User tests: Successful: Unsuccessful:

avatar julienV
julienV
30 Mar 2017

Summary of Changes

I added function to the JUser to get the user timezone, which is very useful for any developer working with date and time in Joomla.
It's a simple function, but it already simplifies some cumbersome library calls.

Testing Instructions

Fill and save any form field using calendar field with 'user_utc', like article publish date for example

Expected result

The date and time saved should be the set date and time...

avatar julienV julienV - open - 30 Mar 2017
avatar julienV julienV - change - 30 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Mar 2017
Category Libraries External Library
avatar phproberto phproberto - test_item - 30 Mar 2017 - Tested successfully
avatar phproberto
phproberto - comment - 30 Mar 2017

I have tested this item successfully on 99c9e47


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

avatar joomdonation joomdonation - test_item - 31 Mar 2017 - Tested successfully
avatar joomdonation
joomdonation - comment - 31 Mar 2017

I have tested this item successfully on 99c9e47

Code review + write code JFactory::getUser()->getTimeZone(); to check the result. It is working as expected.


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 31 Mar 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Mar 2017

I have tested this item successfully on 99c9e47

Changed a given Date-Time-Format to user_utc give after saving Article same Date-Time-Format before saving.


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 31 Mar 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Mar 2017

RTC after 3 successful testes.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Mar 2017

@joomdonation was quicker than me.

avatar joomdonation
joomdonation - comment - 31 Mar 2017

@franz-wohlkoenig No problem if there are more than two tests per PR :D

avatar julienV
julienV - comment - 31 Mar 2017

thanks ;)

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Mar 2017

@joomdonation if you test #15021 we have one more for RTC ;-)

avatar julienV julienV - change - 31 Mar 2017
Labels Added: ?
avatar rdeutz rdeutz - change - 31 Mar 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-03-31 12:56:04
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 31 Mar 2017
avatar rdeutz rdeutz - merge - 31 Mar 2017

Add a Comment

Login with GitHub to post a comment