Projet

Général

Profil

Development #29323

ne pas utiliser settings.SETTINGS_MODULE pour obtenir le nom de l'application

Ajouté par Frédéric Péters il y a plus de 5 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
22 décembre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

De #29320,

Mais accéder à SETTINGS_MODULE reste une mauvaise idée vu que ça ne marche pas toujours (ça pourra péter dans des tests qui utiliseront override_settings()).

Fichiers

0001-multitenant-use-environment-to-get-application-name-.patch (2,48 ko) 0001-multitenant-use-environment-to-get-application-name-.patch Frédéric Péters, 22 décembre 2018 13:39

Révisions associées

Révision 20da13f8 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

multitenant: use environment to get application name (#29323)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 5 ans

#2

Mis à jour par Thomas Noël il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée

Impec. Éventuellement pour rendre le test un poil plus clair :

-os.environ['DJANGO_SETTINGS_MODULE'] = 'fake'
+os.environ['DJANGO_SETTINGS_MODULE'] = 'fake.settings'
#3

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)

Poussé avec cette modification.

commit 20da13f8428bb5cdcbdb28a87f0b9e71034c1ea0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Dec 21 18:39:25 2018 +0100

    multitenant: use environment to get application name (#29323)
#4

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF