fix(autofixes): skip devanagari danda fix for latin variants

This also adds type annotation to autofixes.

Fixes #12517

docs/changes.rst

@@ -13,6 +13,7 @@ Not yet released.
 
 * :ref:`mt-deepl` now supports specifying translation context.
 * :ref:`mt-aws` now supports :ref:`glossary-mt`.
+* :ref:`autofix` for Devanagari danda now better handles latin script.
 
 **Bug fixes**
 

weblate/trans/autofixes/base.py

@@ -1,8 +1,14 @@
 # Copyright © Michal Čihař <[email protected]>
 #
 # SPDX-License-Identifier: GPL-3.0-or-later
+
 from __future__ import annotations
 
+from typing import TYPE_CHECKING
+
+if TYPE_CHECKING:
+    from weblate.trans.models import Unit
+
 
 class AutoFix:
     """Base class for AutoFixes."""
@@ -16,11 +22,13 @@ def get_identifier(self):
     def get_related_checks():
         return []
 
-    def fix_single_target(self, target, source, unit) -> tuple[str, bool]:
+    def fix_single_target(
+        self, target: str, source: str, unit: Unit
+    ) -> tuple[str, bool]:
         """Fix a single target, implement this method in subclasses."""
         raise NotImplementedError
 
-    def fix_target(self, target, unit):
+    def fix_target(self, target: str, unit: Unit) -> tuple[list[str], bool]:
         """Return a target translation array with a single fix applied."""
         source_strings = unit.get_source_plurals()
         if unit.translation.component.is_multivalue:

weblate/trans/autofixes/chars.py

@@ -2,7 +2,10 @@
 #
 # SPDX-License-Identifier: GPL-3.0-or-later
 
+from __future__ import annotations
+
 import re
+from typing import TYPE_CHECKING
 
 from django.utils.translation import gettext_lazy
 
@@ -15,6 +18,9 @@
 from weblate.formats.helpers import CONTROLCHARS_TRANS
 from weblate.trans.autofixes.base import AutoFix
 
+if TYPE_CHECKING:
+    from weblate.trans.models import Unit
+
 
 class ReplaceTrailingDotsWithEllipsis(AutoFix):
     """Replace trailing dots with an ellipsis."""
@@ -26,7 +32,9 @@ class ReplaceTrailingDotsWithEllipsis(AutoFix):
     def get_related_checks():
         return [EndEllipsisCheck()]
 
-    def fix_single_target(self, target, source, unit):
+    def fix_single_target(
+        self, target: str, source: str, unit: Unit
+    ) -> tuple[str, bool]:
         if source and source[-1] == "…" and target.endswith("..."):
             return f"{target[:-3]}…", True
         return target, False
@@ -42,7 +50,9 @@ class RemoveZeroSpace(AutoFix):
     def get_related_checks():
         return [ZeroWidthSpaceCheck()]
 
-    def fix_single_target(self, target, source, unit):
+    def fix_single_target(
+        self, target: str, source: str, unit: Unit
+    ) -> tuple[str, bool]:
         if unit.translation.language.base_code == "km":
             return target, False
         if "\u200b" not in source and "\u200b" in target:
@@ -56,7 +66,9 @@ class RemoveControlChars(AutoFix):
     fix_id = "control-chars"
     name = gettext_lazy("Control characters")
 
-    def fix_single_target(self, target, source, unit):
+    def fix_single_target(
+        self, target: str, source: str, unit: Unit
+    ) -> tuple[str, bool]:
         result = target.translate(CONTROLCHARS_TRANS)
         return result, result != target
 
@@ -67,9 +79,12 @@ class DevanagariDanda(AutoFix):
     fix_id = "devanadari-danda"
     name = gettext_lazy("Devanagari danda")
 
-    def fix_single_target(self, target, source, unit):
+    def fix_single_target(
+        self, target: str, source: str, unit: Unit
+    ) -> tuple[str, bool]:
         if (
             unit.translation.language.is_base(("hi", "bn", "or"))
+            and "_Latn" not in unit.translation.language.code
             and source.endswith(".")
             and target.endswith((".", "\u09f7", "|"))
         ):
@@ -87,7 +102,9 @@ class PunctuationSpacing(AutoFix):
     def get_related_checks():
         return [PunctuationSpacingCheck()]
 
-    def fix_single_target(self, target, source, unit):
+    def fix_single_target(
+        self, target: str, source: str, unit: Unit
+    ) -> tuple[str, bool]:
         if (
             unit.translation.language.is_base(("fr", "br"))
             and unit.translation.language.code != "fr_CA"