Do not increase counter in case of no change

weblate/trans/models.py

@@ -2553,7 +2553,7 @@ class Unit(models.Model):
         except FileLockException:
             logger.error('failed to lock backend for %s!', self)
             messages.error(request, _('Failed to store message in the backend, lock timeout occurred!'))
-            return
+            return False
 
         # Handle situation when backend did not find the message
         if pounit is None:
@@ -2561,12 +2561,12 @@ class Unit(models.Model):
             messages.error(request, _('Message not found in backend storage, it is probably corrupted.'))
             # Try reloading from backend
             self.translation.update_from_blob(True)
-            return
+            return False
 
         # Return if there was no change
         if not saved and propagate:
             self.propagate(request)
-            return
+            return False
 
         # Update translated flag
         self.translated = is_translated(pounit)

weblate/trans/views.py

@@ -1362,31 +1362,33 @@ def translate(request, project, subproject, lang):
                     # Update unit and save it
                     unit.target = join_plural(form.cleaned_data['target'])
                     unit.fuzzy = form.cleaned_data['fuzzy']
-                    unit.save_backend(request)
+                    saved = unit.save_backend(request)
 
-                    # Update stats
-                    profile.translated += 1
-                    profile.save()
-                    # Get new set of checks
-                    newchecks = set(
-                        unit.active_checks().values_list('check', flat=True)
-                    )
-                    # Did we introduce any new failures?
-                    if newchecks > oldchecks:
-                        # Show message to user
-                        messages.error(
-                            request,
-                            _('Some checks have failed on your translation!')
+                    if saved:
+                        # Update userstats
+                        profile.translated += 1
+                        profile.save()
+
+                        # Get new set of checks
+                        newchecks = set(
+                            unit.active_checks().values_list('check', flat=True)
                         )
-                        # Stay on same entry
-                        return HttpResponseRedirect(
-                            '%s?type=%s&pos=%d&dir=stay%s' % (
-                                obj.get_translate_url(),
-                                rqtype,
-                                pos,
-                                search_url
+                        # Did we introduce any new failures?
+                        if newchecks > oldchecks:
+                            # Show message to user
+                            messages.error(
+                                request,
+                                _('Some checks have failed on your translation!')
+                            )
+                            # Stay on same entry
+                            return HttpResponseRedirect(
+                                '%s?type=%s&pos=%d&dir=stay%s' % (
+                                    obj.get_translate_url(),
+                                    rqtype,
+                                    pos,
+                                    search_url
+                                )
                             )
-                        )
 
                 # Redirect to next entry
                 return HttpResponseRedirect('%s?type=%s&pos=%d%s' % (