ods: add time type backwards compatibility

jazzband · Jul 10, 2024 · b1c3b89 · b1c3b89
1 parent 0f6eb81
commit b1c3b89
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/tablib/formats/_ods.py b/src/tablib/formats/_ods.py
@@ -92,7 +92,11 @@ def convert_date(val):
                 return convert_date(date_value)
         if value_type == 'time':
             time_value = cell.getAttribute('timevalue')
-            return dt.datetime.strptime(time_value, "PT%HH%MM%SS").time()
+            try:
+                return dt.datetime.strptime(time_value, "PT%HH%MM%SS").time()
+            except ValueError:
+                # backwards compatibility for times exported with older tablib versions
+                return dt.datetime.strptime(time_value, "%H:%M:%S").time()
         if value_type == 'boolean':
             bool_value = cell.getAttribute('booleanvalue')
             return bool_value == 'true'

diff --git a/tests/files/book.ods b/tests/files/book.ods