Bug #51918
closedNative date and datetime values do not consider timezone
100%
Description
The TCA configuration property "dbType" allows to store "date" or "datetime" values directly in the database (instead of using a timestamp value).
However, the timezone is not recognized correctly - besides that, the serverTimeZone is applied which does not make much sense at all, since it leaves out DST handling.
See #32081 for a full description on how times/dates are stored in TYPO3 and how they are converted.
The current scenario is:- being on GMT+0200 (Europe/Berlin with DST - which is "CEST")
- serverTimeZone is set to "1" in Install Tool
- using the time "2013-09-30 00:00:00"
- timestamp (UTC) representation is sent to DataHandler
- "1380499200"
- incoming fields are converted to represent the native date
- "2013-09-30 02:00:00" (because using date() instead of gmdate())
- "2013-09-30 00:00:00" would be correct here
- native date is converted to timestamp again (should be UTC still)
- "1380492000" (7200 seconds are missing, due to wrong timezone handling)
- "1380499200" would be correct (as submitted to DataHandler)
- serverTimeZone is added
- "1380495600" (3600 seconds are missing)
- "1380499200" would be correct (as submitted to DataHandler)
- timezone offset is substracted
- timestamp gets converted back to native date
- "1380488400" -> "2013-09-29 23:00:00"
- "1380492000" would be correct (timezone cleared value for local date() handling)
Updated by Philipp Gampe about 11 years ago
IMHO we should store dates+times as UTC and do the conversion on the fly.
Updated by Gerrit Code Review about 11 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23749
Updated by Oliver Hader about 11 years ago
Philipp Gampe wrote:
IMHO we should store dates+times as UTC and do the conversion on the fly.
Good idea, however timestamps have been stored for the local (default) timezone for ages already... For backwards-compatibility we stay with our holy cow.
Updated by Gerrit Code Review about 11 years ago
Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25367
Updated by Gerrit Code Review about 11 years ago
Patch set 1 for branch TYPO3_6-0 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25368
Updated by Oliver Hader about 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 74f47123d268a5962a7e228e563163984f8807b5.