Hi Adam Thanks for your answer. I'll try to convince you (and others) that the error in question does not make sense, not even when downgraded to a warning.
There are system checks which provide great value such as django.contrib.admin checking whether templates, context processors etc. are configured correctly because it would be very confusing if the admin interface would not show messages after all. There are other checks such as this one which do not protect the code from going wrong but that simply impose additional requirements to Django users. I've reread the discussion in the pull request and haven't found a single reason why it is a good thing to start requiring that LANGUAGES_BIDI is specified explicitly. What is the rationale for requiring an explicit LANGUAGES_BIDI when all you want to do is restrict the overall list of languages for a single project (or the testsuite of many Django libraries for that matter)? After all, the set of bidirectional languages does not change just because some of them aren't active inside a given project. And there is no ambiguity when inheriting LANGUAGES_BIDI from django.conf.global_settings and only overriding LANGUAGES. Thanks, Matthias On Tue, Apr 9, 2019 at 1:28 PM Adam Johnson <m...@adamj.eu> wrote: > Hi Matthias, > > I can see why this is annoying. At least the resolution is easy by setting > LANGUAGES_BIDI = []. I do agree that it should be in the release notes, > although it's quite hard to miss a new system check's output when upgrading. > > I see it was only at the last review of the PR ( > https://github.com/django/django/pull/11060 ) that the check in question > was upgraded from warning W005 to E005. I can see why a warning would make > sense, but at the same time a warning is still a message to the user and > resolved the same way, either by setting LANGUAGES_BIDI or adding it to > SILENCED_SYSTEM_CHECKS. > > I think I'm against removing the check, but pro anything that can make it > easier for users such as yourself - perhaps something as simple as a > suggestion in the error message to set LANGUAGES_BIDI = [] if you're not > using RTL languages? > > Thanks, > > Adam > > On Tue, 9 Apr 2019 at 11:22, Matthias Kestenholz <m...@feinheit.ch> wrote: > >> Hello everyone, >> >> The resolution of https://code.djangoproject.com/ticket/30241 in >> https://github.com/django/django/commit/4400d8296d268f5a8523cd02ddc33b12219b2535 >> introduced a new and in my opinion backwards incompatible requirement >> through the system checks framework. >> >> Previously*, overriding LANGUAGES to restrict the list of available >> languages on a Django site worked. Now it's also necessary to override >> LANGUAGES_BIDI because there is a new system check (translation.E005) which >> verifies that there are no language codes in LANGUAGES_BIDI not in >> LANGUAGES as well. >> >> It is my strong opinion that enforcing this does not provide any benefits >> and has the downside of forcing almost everyone who overrides LANGUAGES to >> now add a new setting to their project to override LANGUAGES_BIDI as well >> with unclear benefits. >> >> Strictness is a good thing, but since LANGUAGES_BIDI is only ever used >> for membership checking (and never iterated over) I don't see the problem >> in keeping the default language codes from django/conf/global_settings.py >> in there even though some of those languages may not be available on a >> given installation anyway. >> >> The report in https://code.djangoproject.com/ticket/30342 was closed as >> wontfix that's why I'm writing to the list. Also, since system checks >> generally aren't mentioned in the release notes users will not be informed >> ahead of time about this change when reading the release notes. >> >> Anyone has any opinions to offer on this? >> >> Thanks, >> Matthias >> >> * At least as long as I'm using Django, probably for 11 years or >> something. >> >> -- >> You received this message because you are subscribed to the Google Groups >> "Django developers (Contributions to Django itself)" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to django-developers+unsubscr...@googlegroups.com. >> To post to this group, send email to django-developers@googlegroups.com. >> Visit this group at https://groups.google.com/group/django-developers. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/django-developers/CANvPqgAy_HytsvmnOpPtrCP_969QhoncJaPUe%3D6pN-VYWMVcjA%40mail.gmail.com >> <https://groups.google.com/d/msgid/django-developers/CANvPqgAy_HytsvmnOpPtrCP_969QhoncJaPUe%3D6pN-VYWMVcjA%40mail.gmail.com?utm_medium=email&utm_source=footer> >> . >> For more options, visit https://groups.google.com/d/optout. >> > > > -- > Adam > > -- > You received this message because you are subscribed to the Google Groups > "Django developers (Contributions to Django itself)" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to django-developers+unsubscr...@googlegroups.com. > To post to this group, send email to django-developers@googlegroups.com. > Visit this group at https://groups.google.com/group/django-developers. > To view this discussion on the web visit > https://groups.google.com/d/msgid/django-developers/CAMyDDM25B-y3D6iHuROz0rfDPzHp8qnHYhDw2inHNOawEDerGA%40mail.gmail.com > <https://groups.google.com/d/msgid/django-developers/CAMyDDM25B-y3D6iHuROz0rfDPzHp8qnHYhDw2inHNOawEDerGA%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > For more options, visit https://groups.google.com/d/optout. > -- www.feinheit.ch — berat...@feinheit.ch — +41 555 11 11 41 -- You received this message because you are subscribed to the Google Groups "Django developers (Contributions to Django itself)" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-developers+unsubscr...@googlegroups.com. To post to this group, send email to django-developers@googlegroups.com. Visit this group at https://groups.google.com/group/django-developers. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/CANvPqgCJov8680kMT1MVC4%2BGb94JHKBQWLGx38D1t6dKtoFYyQ%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
