Projet

Général

Profil

Development #35209

Login, dans le sélecteur d'ou conserver une trace des dernières OU choisies

Ajouté par Benjamin Dauvergne il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
02 août 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Pas toute, disons les 5 dernières, en ajoutant un retrait "Mes collectivités préférées".


Fichiers

Révisions associées

Révision 0fd98047 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 4 ans

remember 5 last selected ous on login form (#35209)

Historique

#2

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Assigné à mis à Benjamin Dauvergne
#3

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

#4

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

Correction d'un non déterminisme sur le pk de l'ou par défaut.

#5

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

Dans src/authentic2/forms/authentication.py sur AuthenticationForm::__init__ je trouve que le :

if preferred_ous and 'ou' in self.fields:
   ...

gagnerait à être mis à côté du "del self.fields['ou']", genre :

-    if not app_settings.A2_LOGIN_FORM_OU_SELECTOR:
-        del self.fields['ou']
+    if app_settings.A2_LOGIN_FORM_OU_SELECTOR:
+        if preferred_ous:
+            choices = self.fields['ou'].choices
+            ...
+    else:
+        del self.fields['ou']

... je dis surtout ça parce que ça m'aurait ouvert les yeux : je n'avais pas vu le "del" en première lecture, je comprenais mal.

#6

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

Déplacement de la redéfinition de choices dans le if du setting de sélecteur d'OU.

#8

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

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

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 0fd98047eef0b035bb0a7357eeb6a770aad3ff3d
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Aug 2 13:53:46 2019 +0200

    remember 5 last selected ous on login form (#35209)
#10

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

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

Formats disponibles : Atom PDF