Decouple locking from request

Signed-off-by: Michal Čihař <michal@cihar.com>

weblate/trans/models/translation.py

@@ -343,7 +343,7 @@ class Translation(models.Model, URLMixin, PercentMixin, LoggerMixin):
 
         self.save()
 
-    def update_lock(self, request):
+    def update_lock(self, user):
         '''
         Updates lock timestamp.
         '''
@@ -354,7 +354,7 @@ class Translation(models.Model, URLMixin, PercentMixin, LoggerMixin):
 
         # Auto lock if we should
         if appsettings.AUTO_LOCK:
-            self.create_lock(request.user)
+            self.create_lock(user)
             return
 
     def _reverse_url_name(self):

weblate/trans/models/unit.py

@@ -549,13 +549,13 @@ class Unit(models.Model, LoggerMixin):
 
         Optional user parameters defines authorship of a change.
         """
-        # Update lock timestamp
-        self.translation.update_lock(request)
-
         # For case when authorship specified, use user from request
         if user is None:
             user = request.user
 
+        # Update lock timestamp
+        self.translation.update_lock(user)
+
         # Store to backend
         try:
             (saved, pounit) = self.translation.update_unit(self, request, user)