Techsystech
/
server-tools
mirror of https://github.com/OCA/server-tools.git
3
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

jsonifier: fix date handling 

We should always keep UTC and let the consumer deal w/ it.
This change is backward compatible but allows to turn off
the old behavior w/ the ctx flag `jsonifier__date_user_tz=False`.
pull/2418/head
Simone Orsi 2022-05-03 17:45:53 +02:00 committed by Sébastien BEAU
parent 58582895e2
commit 4b4de6471e
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
jsonifier/models/models.py
View File

@ -49,6 +49,7 @@ class Base(models.AbstractModel):
elif field.type == "datetime": elif field.type == "datetime":
# Ensures value is a datetime # Ensures value is a datetime
value = fields.Datetime.to_datetime(value) value = fields.Datetime.to_datetime(value)
if self.env.context.get("jsonifier__date_user_tz"):
# Get the timestamp converted to the client's timezone. # Get the timestamp converted to the client's timezone.
# This call also add the tzinfo into the datetime object # This call also add the tzinfo into the datetime object
value = fields.Datetime.context_timestamp(self, value) value = fields.Datetime.context_timestamp(self, value)
@ -162,9 +163,16 @@ class Base(models.AbstractModel):
results = [{} for record in self] results = [{} for record in self]
parsers = {False: parser["fields"]} if "fields" in parser else parser["langs"] parsers = {False: parser["fields"]} if "fields" in parser else parser["langs"]
records = self
if "jsonifier__date_user_tz" not in self.env.context:
# TODO: backward compat flag for v < 15.0
# Dates must always be UTC and the client/consumer of this data
# should deal w/ the format as preferred.
# Drop this flag for v15!
records = records.with_context(jsonifier__date_user_tz=True)
for lang in parsers: for lang in parsers:
translate = lang or parser.get("language_agnostic") translate = lang or parser.get("language_agnostic")
records = self.with_context(lang=lang) if translate else self records = records.with_context(lang=lang) if translate else records
for record, json in zip(records, results): for record, json in zip(records, results):
self._jsonify_record(parsers[lang], record, json) self._jsonify_record(parsers[lang], record, json)