Project

General

Profile

Development #46642

sélectionner par défaut l'OU en cours dans la fenêtre d'import de rôles

Added by Frédéric Péters 4 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
15 Sep 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision 3ba066a7 (diff)
Added by Nicolas Roche 4 months ago

manager: use selected ou by default in import roles form (#46642)

History

#1

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.

#2

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)

#3

Updated by Nicolas Roche 4 months ago

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.

#4

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 ?

#8

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

#9

Updated by Nicolas Roche 4 months ago

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é.

#10

Updated by Valentin Deniaud 4 months ago

  • Status changed from Solution proposée to Solution validée

OK

#11

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)
#12

Updated by Frédéric Péters 4 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF