fix: no grounded joint -> progressDialog closes & no trackback

Author: azaleacolburn

exporter/SynthesisFusionAddin/src/ErrorHandling.py

@@ -129,10 +129,19 @@ def handle_err_top(func: Callable[..., Result[None]]) -> Callable[..., None]:
 
     def wrapper(*args, **kwargs):  # type: ignore
         result = func(*args, **kwargs)
+
         if result.is_err():
             message, severity = result.unwrap_err()
             if severity == ErrorSeverity.Fatal:
+                logger.debug("Testing1")
+                logger.log(ErrorSeverity.Error.value, "Testing1")
                 app = adsk.core.Application.get()
                 app.userInterface.messageBox(f"Fatal Error Encountered {message}")
+                logger.debug(f"Failed Function name: {func.__name__}" )
+                logger.debug("Testing")
+                if func.__name__ == "__init__":
+                    dialog = arg[5]
+                    if dialog is not None:
+                        dialog.hide()
 
     return wrapper

exporter/SynthesisFusionAddin/src/Parser/SynthesisParser/JointHierarchy.py

@@ -1,4 +1,5 @@
 import enum
+from os import error
 import sys
 from logging import ERROR
 from typing import Any, Iterator, cast
@@ -221,7 +222,7 @@ def __init__(self, design: adsk.fusion.Design) -> None:
         self.grounded = searchForGrounded(design.rootComponent)
 
         if self.grounded is None:
-            message = "These is no grounded component in this assembly, aborting kinematic export."
+            message = "There is not a grounded component in this assembly, aborting kinematic export."
             gm.ui.messageBox(message)
             _____: Err[None] = Err(message, ErrorSeverity.Fatal)
             raise RuntimeError()
@@ -522,13 +523,11 @@ def buildJointPartHierarchy(
 
         return Ok(None)
 
-    # I'm fairly certain bubbling this back up is the way to go
-    except Warning:
-        return Err(
-            "Instantiation of the JointParser failed, likely due to a lack of a grounded component in the assembly",
-            ErrorSeverity.Fatal,
-        )
-
+    # I'm fairly certain bubbling this back up is the way to go <- Actually it's ugly but doesn't really matter and there isn't a better option
+    except:
+        progressDialog.progressDialog.hide()
+        # We don't want two errors to pop up, we exit to prevent dealing with two error messages for the same issue and with re-exporting an empty file
+        sys.exit(1)
 
 def populateJoint(simNode: SimulationNode, joints: joint_pb2.Joints, progressDialog: PDMessage) -> Result[None]:
     if progressDialog.wasCancelled():

exporter/SynthesisFusionAddin/src/Parser/SynthesisParser/Parser.py

@@ -68,6 +68,7 @@ def export(self) -> None:
 
         # Physical Props here when ready
 
+        global progressDialog
         progressDialog = app.userInterface.createProgressDialog()
         progressDialog.cancelButtonText = "Cancel"
         progressDialog.isBackgroundTranslucent = False
@@ -89,6 +90,7 @@ def export(self) -> None:
             design.appearances.count,  # this is very high for some reason
             progressDialog,
         )
+                    
 
         Materials.mapAllAppearances(
             design.appearances,

exporter/SynthesisFusionAddin/src/UI/ConfigCommand.py

@@ -361,7 +361,10 @@ def export(exporterOptions: moduleExporterOptions.ExporterOptions) -> None:
     exporterOptions.version = docVersion
     exporterOptions.materials = 0
 
-    Parser.Parser(exporterOptions).export()
+    try:
+        Parser.Parser(exporterOptions).export()
+    except:
+        return
     exporterOptions.writeToDesign()
 
     if exporterOptions.openSynthesisUponExport:
@@ -431,7 +434,10 @@ def notify(self, _: adsk.core.CommandEventArgs) -> None:
         try:
             Parser.Parser(exporterOptions).export()
         except:
-            pass
+            jointConfigTab.reset()
+            gamepieceConfigTab.reset()
+
+            return
         exporterOptions.writeToDesign()
         jointConfigTab.reset()
         gamepieceConfigTab.reset()