django-bulk-hooks 0.1.104__py3-none-any.whl → 0.1.105__py3-none-any.whl

django_bulk_hooks/manager.py

@@ -54,8 +54,18 @@ class BulkHookManager(models.Manager):
         for obj in objs:
             base_obj = base_model()
             for field in base_model._meta.fields:
-                if field.name != 'id' and hasattr(obj, field.name):
-                    setattr(base_obj, field.name, getattr(obj, field.name))
+                # Skip ID field
+                if field.name == 'id':
+                    continue
+                
+                # Safely copy field values
+                try:
+                    if hasattr(obj, field.name):
+                        setattr(base_obj, field.name, getattr(obj, field.name))
+                except (AttributeError, ValueError):
+                    # Skip fields that can't be copied
+                    continue
+            
             base_objects.append(base_obj)
         
         return base_objects
@@ -72,10 +82,20 @@ class BulkHookManager(models.Manager):
             
             # Copy only fields specific to this model
             for field in model_cls._meta.fields:
-                if (field.name != 'id' and 
-                    field.model == model_cls and 
-                    hasattr(obj, field.name)):
-                    setattr(child_obj, field.name, getattr(obj, field.name))
+                # Skip ID field and fields that don't belong to this model
+                if field.name == 'id':
+                    continue
+                
+                # Check if this field belongs to the current model
+                # Use a safer way to check field ownership
+                try:
+                    if hasattr(field, 'model') and field.model == model_cls:
+                        # This field belongs to the current model
+                        if hasattr(obj, field.name):
+                            setattr(child_obj, field.name, getattr(obj, field.name))
+                except AttributeError:
+                    # Skip fields that don't have proper model reference
+                    continue
             
             child_objects.append(child_obj)
         
@@ -100,33 +120,42 @@ class BulkHookManager(models.Manager):
             objects_by_class[obj_class].append(obj)
         
         for obj_class, class_objects in objects_by_class.items():
-            # Check if this class has multi-table inheritance
-            parent_models = [p for p in obj_class._meta.get_parent_list() 
-                           if not p._meta.abstract]
-            
-            if not parent_models:
-                # No inheritance, use standard bulk_create
-                chunk_result = super(models.Manager, self).bulk_create(class_objects, **kwargs)
-                result.extend(chunk_result)
-                continue
-            
-            # Handle multi-table inheritance
-            # Step 1: Bulk create base objects without hooks
-            base_objects = self._extract_base_objects(class_objects, obj_class)
-            created_base = super(models.Manager, self).bulk_create(base_objects, **kwargs)
-            
-            # Step 2: Update original objects with base IDs
-            for obj, base_obj in zip(class_objects, created_base):
-                obj.pk = base_obj.pk
-                obj._state.adding = False
-            
-            # Step 3: Bulk create child objects without hooks
-            child_objects = self._extract_child_objects(class_objects, obj_class)
-            if child_objects:
-                # Use _base_manager to avoid recursion
-                obj_class._base_manager.bulk_create(child_objects, **kwargs)
-            
-            result.extend(class_objects)
+            try:
+                # Check if this class has multi-table inheritance
+                parent_models = [p for p in obj_class._meta.get_parent_list() 
+                               if not p._meta.abstract]
+                
+                if not parent_models:
+                    # No inheritance, use standard bulk_create
+                    chunk_result = super(models.Manager, self).bulk_create(class_objects, **kwargs)
+                    result.extend(chunk_result)
+                    continue
+                
+                # Handle multi-table inheritance
+                # Step 1: Bulk create base objects without hooks
+                base_objects = self._extract_base_objects(class_objects, obj_class)
+                created_base = super(models.Manager, self).bulk_create(base_objects, **kwargs)
+                
+                # Step 2: Update original objects with base IDs
+                for obj, base_obj in zip(class_objects, created_base):
+                    obj.pk = base_obj.pk
+                    obj._state.adding = False
+                
+                # Step 3: Bulk create child objects without hooks
+                child_objects = self._extract_child_objects(class_objects, obj_class)
+                if child_objects:
+                    # Use _base_manager to avoid recursion
+                    obj_class._base_manager.bulk_create(child_objects, **kwargs)
+                
+                result.extend(class_objects)
+                
+            except Exception as e:
+                # Add debugging information
+                import logging
+                logger = logging.getLogger(__name__)
+                logger.error(f"Error in _bulk_create_inherited for {obj_class}: {e}")
+                logger.error(f"Model fields: {[f.name for f in obj_class._meta.fields]}")
+                raise
         
         return result
 

{django_bulk_hooks-0.1.104.dist-info → django_bulk_hooks-0.1.105.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-bulk-hooks
-Version: 0.1.104
+Version: 0.1.105
 Summary: Hook-style hooks for Django bulk operations like bulk_create and bulk_update.
 License: MIT
 Keywords: django,bulk,hooks

{django_bulk_hooks-0.1.104.dist-info → django_bulk_hooks-0.1.105.dist-info}/RECORD

@@ -6,12 +6,12 @@ django_bulk_hooks/decorators.py,sha256=tckDcxtOzKCbgvS9QydgeIAWTFDEl-ch3_Q--ruEG
 django_bulk_hooks/engine.py,sha256=3HbgV12JRYIy9IlygHPxZiHnFXj7EwzLyTuJNQeVIoI,1402
 django_bulk_hooks/enums.py,sha256=Zo8_tJzuzZ2IKfVc7gZ-0tWPT8q1QhqZbAyoh9ZVJbs,381
 django_bulk_hooks/handler.py,sha256=xZt8iNdYF-ACz-MnKMY0co6scWINU5V5wC1lyDn844k,4854
-django_bulk_hooks/manager.py,sha256=5dBScf_xaXBHSCLU2sGMzmfLz-4FqvrkonpWQ-RomXg,11788
+django_bulk_hooks/manager.py,sha256=Q9Oss6kWCgyL-08t3XYplLSQSLILMAX7Qv9eg7-VgKA,13147
 django_bulk_hooks/models.py,sha256=7RG7GrOdHXFjGVPV4FPRZVNMIHHW-hMCi6hn9LH_hVI,3331
 django_bulk_hooks/priority.py,sha256=HG_2D35nga68lBCZmSXTcplXrjFoRgZFRDOy4ROKonY,376
 django_bulk_hooks/queryset.py,sha256=iet4z-9SKhnresA4FBQbxx9rdYnoaOWbw9LUlGftlP0,1466
 django_bulk_hooks/registry.py,sha256=-mQBizJ06nz_tajZBinViKx_uP2Tbc1tIpTEMv7lwKA,705
-django_bulk_hooks-0.1.104.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
-django_bulk_hooks-0.1.104.dist-info/METADATA,sha256=QAuTwfGoLwuS5lqxxu1bcKheRAKwNrKr5dDDu4btqWU,6951
-django_bulk_hooks-0.1.104.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_bulk_hooks-0.1.104.dist-info/RECORD,,
+django_bulk_hooks-0.1.105.dist-info/LICENSE,sha256=dguKIcbDGeZD-vXWdLyErPUALYOvtX_fO4Zjhq481uk,1088
+django_bulk_hooks-0.1.105.dist-info/METADATA,sha256=PVNvKCg4CciKQTGbe7p4L38tsF9kuum22H7zaPu8oNs,6951
+django_bulk_hooks-0.1.105.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_bulk_hooks-0.1.105.dist-info/RECORD,,