Fixed #31141 -- Relaxed system check of translation settings for sublanguages.
Regression in 4400d8296d
.
Thanks Enrique Matías Sánchez for the report.
This commit is contained in:
parent
979f61abd3
commit
53d8646f79
|
@ -1,4 +1,5 @@
|
||||||
from django.conf import settings
|
from django.conf import settings
|
||||||
|
from django.utils.translation import get_supported_language_variant
|
||||||
from django.utils.translation.trans_real import language_code_re
|
from django.utils.translation.trans_real import language_code_re
|
||||||
|
|
||||||
from . import Error, Tags, register
|
from . import Error, Tags, register
|
||||||
|
@ -55,7 +56,9 @@ def check_setting_languages_bidi(app_configs, **kwargs):
|
||||||
@register(Tags.translation)
|
@register(Tags.translation)
|
||||||
def check_language_settings_consistent(app_configs, **kwargs):
|
def check_language_settings_consistent(app_configs, **kwargs):
|
||||||
"""Error if language settings are not consistent with each other."""
|
"""Error if language settings are not consistent with each other."""
|
||||||
available_tags = {i for i, _ in settings.LANGUAGES} | {'en-us'}
|
try:
|
||||||
if settings.LANGUAGE_CODE not in available_tags:
|
get_supported_language_variant(settings.LANGUAGE_CODE)
|
||||||
|
except LookupError:
|
||||||
return [E004]
|
return [E004]
|
||||||
return []
|
else:
|
||||||
|
return []
|
||||||
|
|
|
@ -16,3 +16,7 @@ Bugfixes
|
||||||
* Fixed a regression in Django 3.0 where ``QuerySet.values()`` and
|
* Fixed a regression in Django 3.0 where ``QuerySet.values()`` and
|
||||||
``values_list()`` crashed if a queryset contained an aggregation and
|
``values_list()`` crashed if a queryset contained an aggregation and
|
||||||
``Exists()`` annotation (:ticket:`31136`).
|
``Exists()`` annotation (:ticket:`31136`).
|
||||||
|
|
||||||
|
* Relaxed the system check added in Django 3.0 to reallow use of a sublanguage
|
||||||
|
in the :setting:`LANGUAGE_CODE` setting, when a base language is available in
|
||||||
|
Django but the sublanguage is not (:ticket:`31141`).
|
||||||
|
|
|
@ -3,7 +3,7 @@ from django.core.checks.translation import (
|
||||||
check_language_settings_consistent, check_setting_language_code,
|
check_language_settings_consistent, check_setting_language_code,
|
||||||
check_setting_languages, check_setting_languages_bidi,
|
check_setting_languages, check_setting_languages_bidi,
|
||||||
)
|
)
|
||||||
from django.test import SimpleTestCase
|
from django.test import SimpleTestCase, override_settings
|
||||||
|
|
||||||
|
|
||||||
class TranslationCheckTests(SimpleTestCase):
|
class TranslationCheckTests(SimpleTestCase):
|
||||||
|
@ -75,12 +75,36 @@ class TranslationCheckTests(SimpleTestCase):
|
||||||
Error(msg % tag, id='translation.E003'),
|
Error(msg % tag, id='translation.E003'),
|
||||||
])
|
])
|
||||||
|
|
||||||
|
@override_settings(USE_I18N=True, LANGUAGES=[('en', 'English')])
|
||||||
def test_inconsistent_language_settings(self):
|
def test_inconsistent_language_settings(self):
|
||||||
msg = (
|
msg = (
|
||||||
'You have provided a value for the LANGUAGE_CODE setting that is '
|
'You have provided a value for the LANGUAGE_CODE setting that is '
|
||||||
'not in the LANGUAGES setting.'
|
'not in the LANGUAGES setting.'
|
||||||
)
|
)
|
||||||
with self.settings(LANGUAGE_CODE='fr', LANGUAGES=[('en', 'English')]):
|
for tag in ['fr', 'fr-CA', 'fr-357']:
|
||||||
self.assertEqual(check_language_settings_consistent(None), [
|
with self.subTest(tag), self.settings(LANGUAGE_CODE=tag):
|
||||||
Error(msg, id='translation.E004'),
|
self.assertEqual(check_language_settings_consistent(None), [
|
||||||
])
|
Error(msg, id='translation.E004'),
|
||||||
|
])
|
||||||
|
|
||||||
|
@override_settings(
|
||||||
|
USE_I18N=True,
|
||||||
|
LANGUAGES=[
|
||||||
|
('de', 'German'),
|
||||||
|
('es', 'Spanish'),
|
||||||
|
('fr', 'French'),
|
||||||
|
('ca', 'Catalan'),
|
||||||
|
],
|
||||||
|
)
|
||||||
|
def test_valid_variant_consistent_language_settings(self):
|
||||||
|
tests = [
|
||||||
|
# language + region.
|
||||||
|
'fr-CA',
|
||||||
|
'es-419',
|
||||||
|
'de-at',
|
||||||
|
# language + region + variant.
|
||||||
|
'ca-ES-valencia',
|
||||||
|
]
|
||||||
|
for tag in tests:
|
||||||
|
with self.subTest(tag), self.settings(LANGUAGE_CODE=tag):
|
||||||
|
self.assertEqual(check_language_settings_consistent(None), [])
|
||||||
|
|
Loading…
Reference in New Issue