fix wrong calculation for attempts, update date_modified in AbstractT…

…imeStampedModel
kobotoolbox · Dec 18, 2024 · 2a50088 · 2a50088
1 parent d5181d7
commit 2a50088
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/kobo/apps/long_running_migrations/models.py b/kobo/apps/long_running_migrations/models.py
@@ -51,7 +51,7 @@ def execute(self):
             return
 
         self.status = LongRunningMigrationStatus.IN_PROGRESS
-        self.attempts += self.attempts
+        self.attempts += 1
         self.save(update_fields=['status', 'attempts'])
 
         try:

diff --git a/kpi/models/abstract_models.py b/kpi/models/abstract_models.py
@@ -25,4 +25,6 @@ def save(self, *args, **kwargs):
         update_fields = kwargs.get('update_fields', None)
         if update_fields is None or 'date_modified' not in update_fields:
             self.date_modified = timezone.now()
+            if update_fields:
+                update_fields.append('date_modified')
         super().save(*args, **kwargs)