Don't deactivate translations by default in management commands

[Chris Chambers]

I see the rationale in django.management.core.base, but this is unexpected behaviour for ./manage.py shell, for example - particularly if you're running with a LANGUAGE_CODE like 'en-gb' as there's no visual feedback/documentation to illustrate that l10n/i18n are off. Granted, this won't make affect many people, but it can cause all kinds of debugging woes.

I'd like to suggest the simple fix of either:

1. forcing translation.activate(settings.LANGUAGE_CODE) during the ./manage.py shell command.
2. Adding some visual feedback/documentation if this turns out to be impractical for whatever reason.

[Chris Chambers]

See also: #11648, which might require the same fix (albeit for a different reason) if the underlying issue isn't/can't be corrected (#9340).

[Claude Paroz]

Add switch to commands to keep active language

[Claude Paroz]

Attached patch with a possible approach to the problem.

[lpiatek]

Comment on https://code.djangoproject.com/attachment/ticket/17379/17379-1.diff patch:

• keep_active_language is not required, can_import_settings is only used for what we want to turn off. So maybe we do not need new flag?
• printing sholdn't be done on stderr, but on stdout?
• we changed information to: "Active language set to 'en-us'.\n"

Comment on whole:

• there should be in BaseCommand Class functions like: _pre_execute(self), _post_execute(self) when such things like translations would be run, but this kind of change requires 'Design Decision', so we ommit it for now

#djangosprint-krakow

[Claude Paroz]

Totally agree about changes related to stderr/stdout.

However I don't see how can_import_settings and keep_active_language are related. For me, those are two different unrelated flags, or am I missing something?

    ``can_import_settings``
        A boolean indicating whether the command needs to be able to
        import Django settings; if ``True``, ``execute()`` will verify
        that this is possible before proceeding. Default value is
        ``True``.

In some cases, we probably will have commands with can_import_settings = True, but where we don't want to switch to en-us.

[lpiatek]

decouple can_import_settings from force_en_us_locale

[gabooo]

Fix i18n makemessages command

[Ramiro Morales]

I'd like to ask to the contributors working on this ticket what are the practical consequences (as in concrete use cases) for the 'shell'and 'makemessages' of being run always with the en-us locale.

It is not clear to me why both of them are particular among the rest of the commands and IMHO having that information would be helpful when it comes to solve/re-triage this ticket.

[neaf]

The ticket requested shell to respect locale setting. Since it's an interface to your app there's no reason not to do that. You might want to see how your locale or other functionality that depends on i18n behaves.

can_import_settings was supposed to check if settings are available before execution of a command according to documentation (​https://docs.djangoproject.com/en/dev/howto/custom-management-commands/#BaseCommand.can_import_settings). It was doing that only by coincidence… it was doing that by forcing en_us locale via translation module which relies on settings.

makemessages used that coincidental behavior to avoid forced locale and respect language set in your settings file.

The patch decouples can_import_settings from the locale forcing behavior to allow its proper usage as described in documentation.

makemessages implementation didn't change. The only difference is it now uses force_en_us_locale instead of can_import_settings to achieve the same thing.

[Ramiro Morales]

Thanks for the explanation.

I'd like to:

• Review if raising StandardError when force_en_us_locale = True and can_import_settings = False is the right thing to do
• Decide if compilemessages (the only other one command that sets can_import_settings = False) does that because it needs to load settings or as a side effect of needing to (de)activate some locale(s), just as makemessages did.

Also needed is a change (addition) to the documentation.

[lpiatek]

Raising StandardError when force_en_us_locale = True and can_import_settings = False:

• for me it's wrong cause force_en_us_locale according to patch is set to True by default, so any command which now has can_import_settings = False would throw mentioned Exception. We just can't assume we need

Possible options:

• set force_en_us_locale to False by default, and enable it where necessary (?), don't really know where and for what it is used (is it really necessary), we only have commen:

        # Switch to English, because django-admin.py creates database content
        # like permissions, and those shouldn't contain any translations.
        # But only do this if we can assume we have a working settings file,
        # because django.utils.translation requires settings.

• don't see any other/better better

[Ramiro Morales]

I've opened #19730 to track the unfolding of the meaning of the can_import_settings option.

I'm also changing the title of this one from "Silent translation of all ./manage.py commands to en-us unexpected/undocumented" "Don't force output of the 'shell' management command to en-us" because the fact that by default commands use the en_US locale isn't undocumented (see "Management commands and locales" note in ​https://docs.djangoproject.com/en/1.4/howto/custom-management-commands/) and hence it shouldn't be unexpected.

Let's discuss in this ticket if it's possible to the 'shell' command to use the locale configured by the developer for the project if USE_I18N is active.

[Ramiro Morales]

#19737 proposes deprecating the shell command. If accepted, that would turn this ticket irrelevant.

[Claude Paroz]

I really think that this isn't about the shell command only. We should only switch to a specific locale when it is required by the command. I worked on a branch (proof of concept, unfinished) demonstrating this here: ​https://github.com/claudep/django/compare/simplify_commands
I'm not totally convinced about the decorator pattern used, but you get the idea.

[Claude Paroz]

I updated my branch and made it a ​PR.

[Claude Paroz <claude@…>]

In d65b0f7:

Fixed #17379 -- Removed management commands deactivation of the locale.