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.
Files
Associated revisions
History
Updated by Paul Marillonnet 4 months ago
- Subject changed from sélectionner par défaut l'OU en cours dans la fenêtre d'export de rôles to 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.
Updated by Frédéric Péters 4 months ago
(oui bien sûr il n'y a d'ailleurs pas de fenêtre pour l'export)
Updated by Nicolas Roche 4 months ago
- File 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch added
- Status changed from Nouveau to Solution proposée
- Assignee set to Nicolas Roche
- Patch proposed changed from No to Yes
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.
Updated by Frédéric Péters 4 months ago
# 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 ?
Updated by Nicolas Roche 4 months ago
- File 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch added
- File 0002-manager-remove-django-1.10-compatibility-46642.patch 0002-manager-remove-django-1.10-compatibility-46642.patch added
(remarques prises en compte)
Updated by Valentin Deniaud 4 months ago
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)
Updated by Nicolas Roche 4 months ago
- File 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch added
Oups, oui en effet.
Updated by Valentin Deniaud 4 months ago
- Status changed from Solution proposée to 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
Updated by Nicolas Roche 4 months ago
- File 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch 0001-manager-use-selected-ou-by-default-in-import-roles-f.patch added
- Status changed from Solution validée to 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é.
Updated by Valentin Deniaud 4 months ago
- Status changed from Solution proposée to Solution validée
OK
Updated by Nicolas Roche 4 months ago
- Status changed from Solution validée to 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)
Updated by Frédéric Péters 4 months ago
- Status changed from Résolu (à déployer) to Solution déployée
manager: use selected ou by default in import roles form (#46642)