hub: do not call save twice in Task.save()

... to ensure that the db operation is atomic.

Fixes: c5d7f8e ("Fix calculating the number of task's subtasks in new Django versions")
Related: release-engineering#217 (comment)

kobo/hub/models.py

@@ -606,9 +606,8 @@ def __str__(self):
         return u"#%s [method: %s, state: %s, worker: %s]" % (self.id, self.method, self.get_state_display(), self.worker)
 
     def save(self, *args, **kwargs):
-        # save to db to precalculate subtask counts and obtain an ID (on insert) for stdout and traceback
-        super(self.__class__, self).save()
-        self.subtask_count = self.subtasks().count()
+        if self.id is not None:
+            self.subtask_count = self.subtasks().count()
         super(self.__class__, self).save()
         self.logs.save()
         if self.parent: