Merge pull request jazzband#611 from ProtixIT/delete-keep_parents

Fix `SoftDeletableModel.delete()` forwarding positional args to superclass

Author: foarsitter

CHANGES.rst

@@ -12,6 +12,7 @@ To be released
 - Make `contribute_to_class()` in `StatusField`, `MonitorField` and `SplitField`
   forward additional arguments to Django
 - `SplitField` no longer accepts `no_excerpt_field` as a keyword argument
+- Make `soft` argument to `SoftDeletableModel.delete()` keyword-only
 
 4.4.0 (2024-02-10)
 ------------------

model_utils/models.py

@@ -146,7 +146,7 @@ class Meta:
     available_objects = SoftDeletableManager()
     all_objects = models.Manager()
 
-    def delete(self, using=None, soft=True, *args, **kwargs):
+    def delete(self, using=None, *args, soft=True, **kwargs):
         """
         Soft delete object (set its ``is_removed`` field to True).
         Actually delete object if setting ``soft`` to False.
@@ -155,7 +155,7 @@ def delete(self, using=None, soft=True, *args, **kwargs):
             self.is_removed = True
             self.save(using=using)
         else:
-            return super().delete(using=using, *args, **kwargs)
+            return super().delete(using, *args, **kwargs)
 
 
 class UUIDModel(models.Model):