Force API translation with URL language defined

.github/workflows/ci.yml

@@ -12,6 +12,10 @@ on:
 env:
   DEBIAN_FRONTEND: noninteractive
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
   flake8:
     name: Check python linting

.gitignore

@@ -12,3 +12,4 @@ var/
 /georiviere/settings/custom.py
 /install/georiviere/
 /install.zip
+.idea/

docs/changelog.rst

@@ -5,6 +5,10 @@ CHANGELOG
 1.3.0+dev    (XXXX-XX-XX)
 ---------------------
 
+**Bug fix**
+
+- Force translation defined in API url /api/portal/<lang> (fix #222)
+
 
 1.3.0    (2023-11-17)
 -------------------------

georiviere/middleware.py

@@ -0,0 +1,29 @@
+import re
+
+from django.utils import translation
+from django.utils.translation.trans_real import get_supported_language_variant
+
+language_code_prefix_re = re.compile(r'^/api/portal/([\w-]+)(/|$)')
+
+
+def get_language_from_path(path):
+    regex_match = language_code_prefix_re.match(path)
+    if not regex_match:
+        return None
+    lang_code = regex_match.group(1)
+    try:
+        return get_supported_language_variant(lang_code)
+    except LookupError:
+        return None
+
+
+class APILocaleMiddleware:
+    def __init__(self, get_response):
+        self.get_response = get_response
+
+    def __call__(self, request):
+        language = get_language_from_path(request.path_info)
+        if language:
+            translation.activate(language)
+            request.LANGUAGE_CODE = translation.get_language()
+        return self.get_response(request)

georiviere/portal/tests/test_views/test_river.py

@@ -43,3 +43,12 @@ def test_stream_list_json_structure(self):
         self.assertEqual(len(response.json()), 1)
         self.assertSetEqual(set(response.json()[0].keys()), {'attachments', 'flow', 'descent', 'length',
                                                              'name', 'id', 'description', 'geometryCenter'})
+
+    def test_stream_ranslation_according_url(self):
+        for lang in ['en', 'fr']:
+            url = reverse('api_portal:streams-detail',
+                          kwargs={'portal_pk': self.portal.pk, 'pk': self.stream.pk, 'lang': lang, 'format': 'json'})
+            response = self.client.get(url)
+            self.assertEqual(response.status_code, 200)
+            data = response.json()
+            self.assertEqual(data['flow'], 'To be defined' if lang == 'en' else 'À définir')

georiviere/settings/__init__.py

@@ -190,18 +190,19 @@ def construct_relative_path_mock(current_template_name, relative_name):
 PAPERCLIP_ENABLE_LINK = True
 PAPERCLIP_ENABLE_VIDEO = True
 
-CRISPY_ALLOWED_TEMPLATE_PACKS = ('bootstrap', 'bootstrap3', 'bootstrap4')
+CRISPY_ALLOWED_TEMPLATE_PACKS = ('bootstrap4', )
 CRISPY_TEMPLATE_PACK = 'bootstrap4'
 
 MIDDLEWARE = [
     'django.middleware.security.SecurityMiddleware',
     'django.contrib.sessions.middleware.SessionMiddleware',
     'django.middleware.common.CommonMiddleware',
+    'django.middleware.locale.LocaleMiddleware',
+    'georiviere.middleware.APILocaleMiddleware',
     'django.middleware.csrf.CsrfViewMiddleware',
     'django.contrib.auth.middleware.AuthenticationMiddleware',
     'django.contrib.messages.middleware.MessageMiddleware',
     'django.middleware.clickjacking.XFrameOptionsMiddleware',
-    'django.middleware.locale.LocaleMiddleware',
     'mapentity.middleware.AutoLoginMiddleware',
 ]