id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
30342	Remove the LANGUAGES_BIDI<=LANGUAGES check.	Matthias Kestenholz	Matthias Kestenholz	"(Adding Nick Pope to Cc: as author of the commit referenced below)

Since https://github.com/django/django/commit/4400d8296d268f5a8523cd02ddc33b12219b2535 there is a system check which verifies that `LANGUAGES_BIDI` is a subset of `LANGUAGES`. This breaks almost all installations of Django using a custom `LANGUAGES` list which do not also override `LANGUAGES_BIDI` -- probably many installations.

All of them will either have to add a `LANGUAGES_BIDI` override or silence `translation.E005` when updating. If this is intentional then this change has to be mentioned in the release notes and documented somewhere.

However, I don't really see the need to verify that `LANGUAGES_BIDI` is a subset of `LANGUAGES` and propose that the easiest and also the best way to solve this is to remove the `translation.E005` check again.

Here's a test which currently fails but shouldn't in my opinion:

{{{
diff --git a/tests/check_framework/test_translation.py b/tests/check_framework/test_translation.py
index 9a34b65c06..cea844988d 100644
--- a/tests/check_framework/test_translation.py
+++ b/tests/check_framework/test_translation.py
@@ -92,3 +92,7 @@ class TranslationCheckTests(SimpleTestCase):
             self.assertEqual(check_language_settings_consistent(None), [
                 Error(msg, id='translation.E005'),
             ])
+
+    def test_languages_without_bidi(self):
+        with self.settings(LANGUAGE_CODE='en', LANGUAGES=[('en', 'English')]):
+            self.assertEqual(check_language_settings_consistent(None), [])
}}}"	Bug	closed	Core (System checks)	dev	Release blocker	fixed		pope1ni	Accepted	1	0	0	0	0	0