Development #46642
sélectionner par défaut l'OU en cours dans la fenêtre d'import de rôles
0%
Description
Je vais sur /manage/roles/, je choisis une OU, je fais "importer", la boite de dialogue me propose de faire l'import dans l'OU par défaut, je préférerais que soit sélectionnée l'OU que j'avais choisie.
Fichiers
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus de 3 ans
- Sujet changé de sélectionner par défaut l'OU en cours dans la fenêtre d'export de rôles à sélectionner par défaut l'OU en cours dans la fenêtre d'import de rôles
J'allais dire “chez moi ça marche” mais en fait ce n'est pas de l'export dont il est question.
Mis à jour par Frédéric Péters il y a plus de 3 ans
(oui bien sûr il n'y a d'ailleurs pas de fenêtre pour l'export)
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Fichier 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Nicolas Roche
- Patch proposed changé de Non à Oui
J'ai dû aller chercher l'ou à la main, dans le contenu de la query string :
query_dict = QueryDict(self.request.environ.get('QUERY_STRING'))
mais je ne vois pas comment faire autrement.
Mis à jour par Frédéric Péters il y a plus de 3 ans
# django >= 1.10
on ne gère que ça.
J'ai dû aller chercher l'ou à la main, dans le contenu de la query string :
request.GET ?
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Fichier 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch ajouté
- Fichier 0002-manager-remove-django-1.10-compatibility-46642.patch 0002-manager-remove-django-1.10-compatibility-46642.patch ajouté
(remarques prises en compte)
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Mais FormView hérite déjà de FormMixin ? (https://docs.djangoproject.com/en/3.1/ref/class-based-views/generic-editing/#django.views.generic.edit.FormView)
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Fichier 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch ajouté
Oups, oui en effet.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Sur la forme, c'est mieux si result s'appelle initial, le if me semble inutile, et je préfère une affectation explicite plutôt que .update. Ça ferait genre
+ def get_initial(self):
+ initial = super().get_initial()
+ initial['ou'] = self.request.GET.get('search-ou')
+ return initial
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Fichier 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch ajouté
- Statut changé de Solution validée à Solution proposée
le if me semble inutile
En me rebasant sur #46448 je pense que je ne dois pas renvoyer un champ vide si le champs est caché.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
OK
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3ba066a70ab69704cadcddea3f9c01d851129987
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Mon Sep 21 15:20:03 2020 +0200
manager: use selected ou by default in import roles form (#46642)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: use selected ou by default in import roles form (#46642)