Update lock time only for owned locks

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

weblate/trans/models/translation.py

@@ -343,17 +343,21 @@ class Translation(models.Model, URLMixin, PercentMixin, LoggerMixin):
 
         self.save()
 
-    def update_lock(self, user):
+    def update_lock(self, user, create=True):
         '''
         Updates lock timestamp.
         '''
+        # Check if we can lock
+        if self.is_user_locked(user):
+            return
+
         # Update timestamp
-        if self.is_user_locked():
+        if self.lock_user == user:
             self.update_lock_time()
             return
 
         # Auto lock if we should
-        if appsettings.AUTO_LOCK:
+        if appsettings.AUTO_LOCK and create:
             self.create_lock(user)
             return
 

weblate/trans/views/lock.py

@@ -37,8 +37,7 @@ from weblate.trans.permissions import (
 def update_lock(request, project, subproject, lang):
     obj = get_translation(request, project, subproject, lang)
 
-    if not obj.is_user_locked(request.user):
-        obj.update_lock_time()
+    obj.update_lock(request.user, False)
 
     return HttpResponse('ok')