Fixed obj. loading duplicate bug

Author: justo46

bseq/importer.py

@@ -191,8 +191,23 @@ def create_meshio_obj(filepath):
 
     # Do I need this for multi-loading?
     if filepath.endswith(".obj"):
+        # Save all current objects
+        objs = set(bpy.context.scene.objects)
+        # Reload the object
         bpy.ops.import_scene.obj(filepath=filepath)
-        obj = bpy.context.selected_objects[0]
+        # Substract all previous items from the current items and print their names
+        imported_objs = set(bpy.context.scene.objects) - objs
+        print(", ".join(o.name for o in imported_objs))
+
+        # Check if the imported object worked correctly
+        if len(imported_objs) == 1:
+            obj = imported_objs.pop()
+        else:
+            show_message_box("Error when reading: " + filepath + ",\n" + traceback.format_exc(),
+                                "obj. Loading Error",
+                                icon="ERROR")
+            return
+        
         obj.name = os.path.basename(filepath)
         return
     #  create the object
@@ -213,16 +228,34 @@ def create_obj(fileseq, root_path, transform_matrix=Matrix([[1, 0, 0, 0], [0, 1,
     #.obj sequences have to be handled differently
     is_obj_seq = filepath.endswith(".obj")
     if is_obj_seq and bpy.context.scene.BSEQ.use_blender_obj_import:
+
+        # Save all current objects
+        objs = set(bpy.context.scene.objects)
+        # Reload the object
         bpy.ops.import_scene.obj(filepath=filepath)
-        enabled = True
+        # Substract all previous items from the current items and print their names
+        imported_objs = set(bpy.context.scene.objects) - objs
+        print(", ".join(o.name for o in imported_objs))
 
-        tmp_obj = bpy.context.selected_objects[-1]
+        # Check if the imported object worked correctly
+        if len(imported_objs) == 1:
+            tmp_obj = imported_objs.pop()
+        else:
+            show_message_box("Error when reading: " + filepath + ",\n" + traceback.format_exc(),
+                                "obj. Loading Error",
+                                icon="ERROR")
+            return
 
         name = fileseq.basename() + "@" + fileseq.extension()
-        object = bpy.data.objects.new(name, tmp_obj.data)
 
-        tmp_obj.select_set(True)
-        bpy.ops.object.delete()
+        # Create object with empty mesh
+        object = bpy.data.objects.new(name, bpy.data.meshes.new(name))
+        object.data = tmp_obj.data
+        
+        # Delete tmp_obj with data
+        bpy.data.objects.remove(tmp_obj, do_unlink=True)
+
+        enabled = True
 
     else:
         meshio_mesh = None
@@ -294,13 +327,37 @@ def update_obj(scene, depsgraph=None):
         if pattern.endswith(".obj") and scene.BSEQ.use_blender_obj_import:
             filepath = fs[current_frame % len(fs)]
 
+            print("File path update: " + filepath)
+
+            # Save all current objects
+            objs = set(scene.objects)
             # Reload the object
             bpy.ops.import_scene.obj(filepath=filepath)
-            tmp_obj = bpy.context.selected_objects[-1]
+            # Substract all previous items from the current items and print their names
+            imported_objs = set(scene.objects) - objs
+            print(", ".join(o.name for o in imported_objs))
+
+            # Check if the imported object worked correctly
+            if len(imported_objs) == 1:
+                new_tmp_obj = imported_objs.pop()
+            else:
+                show_message_box("Error when reading: " + filepath + ",\n" + traceback.format_exc(),
+                                 "obj. Loading Error",
+                                 icon="ERROR")
+                continue
+
+            # Copy the data except for material
+            if obj.data.materials:
+                # assign to 1st material slot
+                new_tmp_obj.data.materials[0] = obj.data.materials[0]
+            else:
+                # no slots
+                new_tmp_obj.data.materials.append(obj.data.materials[0])
+
+            obj.data = new_tmp_obj.data
 
-            obj.data = tmp_obj.data
-            tmp_obj.select_set(True)
-            bpy.ops.object.delete()
+            # Delete the temporary object with the data
+            bpy.data.objects.remove(new_tmp_obj, do_unlink=True)
 
             apply_transformation(meshio_mesh, obj, depsgraph)
 

bseq/operators.py

@@ -346,13 +346,33 @@ class WM_OT_batchSequences(bpy.types.Operator, ImportHelper):
     bl_label = "Import Sequences"
     bl_options = {'PRESET', 'UNDO'}
 
-    # filter_glob: bpy.types.StringProperty(
-    #     default="*.txt",
-    #     options={'HIDDEN'},
-    #     maxlen=255,  # Max internal buffer length, longer would be clamped.
-    # )
+    def update_filter_glob(self, context):
+        bpy.ops.wm.seq_import_batch('INVOKE_DEFAULT')
+
+    filter_string: bpy.props.StringProperty(
+        default="*.obj",
+        options={'HIDDEN'},
+        update=update_filter_glob,
+    )
+
+    filename_ext=''
+    filter_glob: bpy.props.StringProperty(
+        default='*.obj',
+        options={'HIDDEN', 'LIBRARY_EDITABLE'},
+    )
 
     files: bpy.props.CollectionProperty(type=bpy.types.PropertyGroup)
+    
+    def invoke(self, context, event):
+        scene = context.scene
+        if scene.BSEQ.filter_string:
+            self.filter_glob = scene.BSEQ.filter_string
+        else:
+            self.filter_glob = "*"
+
+        context.window_manager.fileselect_add(self)
+
+        return {'RUNNING_MODAL'}
 
     def execute(self, context):
         scene = context.scene
@@ -363,6 +383,7 @@ def execute(self, context):
             show_message_box("When using relative path, please save file before using it", icon="ERROR")
             return {"CANCELLED"}
 
+        self.filter_glob = '*'
 
         folder = Path(self.filepath)
         used_seqs = set()
@@ -379,7 +400,7 @@ def execute(self, context):
                 create_obj(matching_seqs[0], importer_prop.root_path, transform_matrix=transform_matrix)
                 used_seqs.add(matching_seqs[0])
         return {'FINISHED'}
-    
+
     def draw(self, context):
         pass
 
@@ -403,11 +424,10 @@ def draw(self, context):
         layout.use_property_split = True
         layout.use_property_decorate = False  # No animation.
 
-        sfile = context.space_data
-        operator = sfile.active_operator
+        # sfile = context.space_data
+        # operator = sfile.active_operator
 
-        layout.prop(operator, 'type')
-        layout.prop(operator, 'use_setting')
+        layout.prop(importer_prop, 'filter_string')
 
         layout.alignment = 'LEFT'
         layout.prop(importer_prop, "relative", text="Relative Path")

bseq/panels.py

@@ -22,19 +22,27 @@ def filter_items(self, context, data, property):
 
     def draw_item(self, context, layout, data, item, icon, active_data, active_propname):
         if item:
-            split = layout.split(factor=0.7)
+            split = layout.split(factor=0.5)
             col1 = split.column()
             col2 = split.column()
-            split2 = col2.split(factor=0.5)
+            split2 = col2.split(factor=0.25)
             col2 = split2.column()
             col3 = split2.column()
+            split3 = col3.split(factor=0.33)
+            col3 = split3.column()
+            col4 = split3.column()
+            col4.alignment = 'CENTER'
+            start_frame = item.BSEQ.start_end_frame[0]
+            end_frame = item.BSEQ.start_end_frame[1]
             col1.prop(item, "name", text='', emboss=False)
             if item.BSEQ.enabled:
                 col2.prop(item.BSEQ, "enabled", text="", icon="PLAY")
                 col3.prop(item.BSEQ, "frame", text="")
+                col4.label(text=str(start_frame) + '-' + str(end_frame))
             else:
                 col2.prop(item.BSEQ, "enabled", text ="", icon="PAUSE")
                 col3.label(text="", icon="BLANK1")
+                col4.label(text=str(start_frame) + '-' + str(end_frame))
         else:
             # actually, I guess this line of code won't be executed?
             layout.label(text="", translate=False, icon_value=icon)
@@ -213,7 +221,8 @@ def draw(self, context):
 
         col1.label(text="Import Settings")
 
-        # col2.prop(importer_prop, "use_blender_obj_import", text="Blender .obj Importer")
+        col2.prop(importer_prop, "filter_string", text="Filter String")
+
         col2.prop(importer_prop, "relative", text="Relative Path")
 
         if importer_prop.relative:

bseq/properties.py

@@ -107,6 +107,11 @@ class BSEQ_scene_property(bpy.types.PropertyGroup):
                                                    description="Use Blender's built-in .obj import function (or meshio's .obj import function)",
                                                    default=True,
                                                    )
+    
+    filter_string: bpy.props.StringProperty(name='Filter String',
+                                            description='Filter string for file sequences',
+                                            default='',
+                                            )
 
 class BSEQ_obj_property(bpy.types.PropertyGroup):
     init: bpy.props.BoolProperty(default=False)