chore: use format_html_join_comma

This allows to localize the comma.

.pre-commit-config.yaml

@@ -113,6 +113,12 @@ repos:
     files: docs/changes\.rst
     entry: ^ *\* .*[^.]\n$
     language: pygrep
+  - id: py-join-comma
+    name: Localized comma in join
+    types: [python]
+    args: [--multiline]
+    entry: format_html_join\(\s*", "
+    language: pygrep
 - repo: https://github.com/sphinx-contrib/sphinx-lint
   rev: v1.0.0
   hooks:

weblate/checks/base.py

@@ -16,6 +16,7 @@
 from siphashc import siphash
 
 from weblate.utils.docs import get_doc_url
+from weblate.utils.html import format_html_join_comma
 from weblate.utils.xml import parse_xml
 
 if TYPE_CHECKING:
@@ -309,8 +310,7 @@ def format_value(self, value: str) -> StrOrPromise:
     def get_values_text(self, message: str, values: Iterable[str]) -> StrOrPromise:
         return format_html(
             message,
-            format_html_join(
-                ", ",
+            format_html_join_comma(
                 "{}",
                 ((self.format_value(value),) for value in sorted(values)),
             ),

weblate/checks/glossary.py

@@ -7,11 +7,12 @@
 import re
 from typing import TYPE_CHECKING
 
-from django.utils.html import escape, format_html, format_html_join
+from django.utils.html import escape, format_html
 from django.utils.translation import gettext, gettext_lazy
 
 from weblate.checks.base import SourceCheck, TargetCheck
 from weblate.utils.csv import PROHIBITED_INITIAL_CHARS
+from weblate.utils.html import format_html_join_comma
 
 if TYPE_CHECKING:
     from weblate.trans.models import Unit
@@ -80,7 +81,7 @@ def get_description(self, check_obj):
             escape(
                 gettext("Following terms are not translated according to glossary: {}")
             ),
-            format_html_join(", ", "{}", ((term,) for term in sorted(results))),
+            format_html_join_comma("{}", ((term,) for term in sorted(results))),
         )
 
 
@@ -104,5 +105,5 @@ def get_description(self, check_obj) -> str:
                     "The string starts with one or more of the following forbidden characters: {}"
                 )
             ),
-            format_html_join(", ", "{}", PROHIBITED_INITIAL_CHARS),
+            format_html_join_comma("{}", PROHIBITED_INITIAL_CHARS),
         )