Add a few sanity checks when loading poses, for issue #58

src/mpfb/ui/applypose/operators/loadpartial.py

@@ -37,6 +37,10 @@ def execute(self, context):
         from mpfb.ui.applypose.applyposepanel import POSES_PROPERTIES
         name = POSES_PROPERTIES.get_value("available_partials", entity_reference=context.scene)
 
+        if not name:
+            self.report({'ERROR'}, "Must select a valid pose name")
+            return {'FINISHED'}
+
         rig_type = RigService.identify_rig(armature_object)
         if "default" in rig_type:
             rig_type = "default"
@@ -48,6 +52,10 @@ def execute(self, context):
         absolute_file_path = bpy.path.abspath(os.path.join(pose_root, name + ".json"))
         _LOG.debug("absolute_file_path", absolute_file_path)
 
+        if not os.path.exists(absolute_file_path):
+            self.report({'ERROR'}, "The selected pose '" + name + "' for rig type '" + rigtype + mode + "' does not exist as file. You should probably report this as a bug.")
+            return {'FINISHED'}
+
         pose = dict()
 
         with open(absolute_file_path, "r") as json_file:

src/mpfb/ui/applypose/operators/loadpose.py

@@ -37,6 +37,12 @@ def execute(self, context):
         from mpfb.ui.applypose.applyposepanel import POSES_PROPERTIES
         name = POSES_PROPERTIES.get_value("available_poses", entity_reference=context.scene)
 
+        name = str(name).strip()
+
+        if not name:
+            self.report({'ERROR'}, "Must select a valid pose name")
+            return {'FINISHED'}
+
         rig_type = RigService.identify_rig(armature_object)
         if "default" in rig_type:
             rig_type = "default"
@@ -52,6 +58,10 @@ def execute(self, context):
         absolute_file_path = bpy.path.abspath(os.path.join(pose_root, name + ".json"))
         _LOG.debug("absolute_file_path", absolute_file_path)
 
+        if not os.path.exists(absolute_file_path):
+            self.report({'ERROR'}, "The selected pose '" + name + "' for rig type '" + rigtype + mode + "' does not exist as file. You should probably report this as a bug.")
+            return {'FINISHED'}
+
         pose = dict()
 
         with open(absolute_file_path, "r") as json_file: