Github: Fix for "always return todays date"

[hominid4]

Regarding Github commit 5032880, Fix for "always return todays date", should "$val = JFactory::getDate('now', $timeZone);" and "$val = JFactory::getDate('now');" be flipped?

I have a date element set to UTC "always return todays date" and it kept recording the local time instead of UTC and wondered if that was the culprit? I switched it on my end and is working correctly but wanted to double check.

Thanks!

 

[cheesegrits]

Can you test it the other way and let me know? So set up two dates stored as UTC and local, and make sure they both work as advertized?

If you can do that quickly, I'll fold the change into the 3.6 release I'm building atm (which is why I'm too busy to test it myself).

-- hugh

 

[hominid4]

Thanks Hugh, yes I switched it around and only have a couple entries come in so far but seems to be working correctly now. It was recording incorrectly as local time but is now recording correct as UTC time, per my setting.

And just checked as Local Time and it stored correctly as Local Time.


Thanks again.

 

[cheesegrits]

https://github.com/Fabrik/fabrik/commit/4078e2a1b2c2b8f05b325479d0bafc4291fd8dde

tx

-- hugh