Changeset 8207 for main/waeup.kofa/trunk/src/waeup/kofa/widgets/datetimewidget.py

Timestamp:

18 Apr 2012, 12:46:52 (12 years ago)

Author:

uli

Message:

Try to be smart (evil?). Interpret entered datetimes w/o TZ as app-wide TZ data.

File:

• main/waeup.kofa/trunk/src/waeup/kofa/widgets/datetimewidget.py (modified) (3 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/widgets/datetimewidget.py

@@ -20 +20 @@
 import datetime
 import pytz
+from zope.component import queryUtility
+from zope.datetime import parseDatetimetz
+from zope.datetime import DateTimeError
+from zope.formlib.i18n import _
+from zope.formlib.interfaces import ConversionError
 from zope.formlib.textwidgets import DatetimeWidget
+from waeup.kofa.utils.helpers import to_timezone
+from waeup.kofa.interfaces import IKofaUtils
+
+class _DummyUtils(object):
+    tzinfo = pytz.utc
+
+# A fallback, if no IKofaUtils can be found
+_DUMMY_UTILS = _DummyUtils()
 
 class PytzDatetimeWidget(DatetimeWidget):
@@ -38 +51 @@
     go into the DB.
 
+    For datetimes without any timezone set, we interpret the input to
+    be meant as local app-time. I.e. if application TZ is
+    ``Africa/Lagos``, we assume that a string like '2012-02-01 12:13'
+    (apparently not providing TZ info) was meant as 12:13 h Lagos
+    time.
+
     From zope.datetime, however, we save the parser abilities to
     interpret even bizarre entered data as some datetime.
@@ -47 +66 @@
 
         """
-        value = super(PytzDatetimeWidget, self)._toFieldValue(string)
+        if string == self._missing:
+            return self.context.missing_value
+        else:
+            try:
+                # Different to original implementation we do not
+                # automatically request local server time if no TZ was
+                # set in `string`. In this case we want a datetime with
+                # tzinfo set to `None` instead.
+                value = parseDatetimetz(string, local=False)
+            except (DateTimeError, ValueError, IndexError), v:
+                raise ConversionError(_("Invalid datetime data"), v)
+
         if not isinstance(value, datetime.datetime):
             return value
         if value.tzinfo is None:
-            return pytz.utc.localize(value)
+            utils = queryUtility(IKofaUtils, default=_DUMMY_UTILS)
+            value = utils.tzinfo.localize(value)
         return value.astimezone(pytz.utc)