Fix KeyError: time on import of content with workflow without `time…

…` variable. Fixes #222
collective · Nov 30, 2023 · 6c4586f · 6c4586f
1 parent 58b2bc4
commit 6c4586f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -5,6 +5,9 @@ Changelog
 1.11 (unreleased)
 -----------------
 
+- Fix ``KeyError: time`` when importing content with a workflow that does not have the ``time`` variable.
+  [maurits]
+
 - Allow to use fix_html_in_content_fields without applying the default html_fixer.
   [pbauer]
 

diff --git a/src/collective/exportimport/import_content.py b/src/collective/exportimport/import_content.py
@@ -816,9 +816,10 @@ def import_workflow_history(self, obj, item):
         for key, value in workflow_history.items():
             # The time needs to be deserialized
             for history_item in value:
-                history_item["time"] = DateTime(
-                    dateutil.parser.parse(history_item["time"])
-                )
+                if "time" in history_item:
+                    history_item["time"] = DateTime(
+                        dateutil.parser.parse(history_item["time"])
+                    )
             result[key] = value
         if result:
             obj.workflow_history = PersistentMapping(result.items())