Print error on logs if Specification loading fails (#2770)

spine-tools · May 15, 2024 · 2e3ca53 · 2e3ca53
2 parents 49bc7dc + 0d3660d
commit 2e3ca53
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/spinetoolbox/helpers.py b/spinetoolbox/helpers.py
@@ -1207,7 +1207,10 @@ def load_specification_from_file(spec_path, local_data_dict, spec_factories, app
     if spec_dict is None:
         return None
     spec_dict["definition_file_path"] = spec_path
-    spec = specification_from_dict(spec_dict, local_data_dict, spec_factories, app_settings, logger)
+    try:
+        spec = specification_from_dict(spec_dict, local_data_dict, spec_factories, app_settings, logger)
+    except KeyError:
+        spec = None
     if spec is not None:
         spec.definition_file_path = spec_path
     return spec

diff --git a/spinetoolbox/ui_main.py b/spinetoolbox/ui_main.py
@@ -1063,6 +1063,7 @@ def import_specification(self):
             def_file, local_data, self._item_specification_factories, self._qsettings, self
         )
         if not specification:
+            self.msg_error.emit("Failed to load specification.")
             return
         self.undo_stack.push(AddSpecificationCommand(self._project, specification, save_to_disk=False))