Development #30252
Permettre un sélecteur d'OU dans le formulaire de login
100%
Description
Pour certains déploiements nous aurons des comptes homonymes dans différentes OU, ce sélecteur permettra de les différencier.
Fichiers
Demandes liées
Révisions associées
backends: PEP8ness, style (#30252)
tests: add copyright header (#30252)
utils: add a lazy_label helper (#30252)
It allows giving a default value for a label with a getter function to
allow customization.
forms: add ou selector to login form (#30252)
support ou selector in backends and forms (fixes #30252)
Historique
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0004-utils-add-a-lazy_label-helper-30252.patch 0004-utils-add-a-lazy_label-helper-30252.patch ajouté
- Fichier 0006-support-ou-selector-in-backends-and-forms-fixes-3025.patch 0006-support-ou-selector-in-backends-and-forms-fixes-3025.patch ajouté
- Fichier 0001-forms-PEP8ness-style-30252.patch 0001-forms-PEP8ness-style-30252.patch ajouté
- Fichier 0002-backends-PEP8ness-style-30252.patch 0002-backends-PEP8ness-style-30252.patch ajouté
- Fichier 0005-forms-add-ou-selector-to-login-form-30252.patch 0005-forms-add-ou-selector-to-login-form-30252.patch ajouté
- Fichier 0003-tests-add-copyright-header-30252.patch 0003-tests-add-copyright-header-30252.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
- ajout d'un "utils" pour facilement avoir des chaînes traduites customisées depuis les settings
- ajout du champ au formulaire de login
- ajout du support dans ModelBackend et LdapBackend
Mis à jour par Paul Marillonnet il y a environ 5 ans
Rien à redire à première vue, à part peut-être def lazy_label(default, func):
où il me semble plus logique de mettre default
en second (puisque la fonction retourne un force_text de func() or default
, et non pas l'inverse).
Je teste maintenant.
Mis à jour par Paul Marillonnet il y a environ 5 ans
Paul Marillonnet a écrit :
Je teste maintenant.
Un premier test, tentative de connexion avec un usager créé à l'aide de la commande de manage "createsuperuser".
Cet usager privilégié n'est rattaché à aucune OU.
Avec l'option de sélecteur d'OU activée, on doit forcément choisir une OU dans la liste.
Donc impossibilité de se connecter pour cet usager "apatride".
Est-ce le comportement souhaité ?
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Paul Marillonnet a écrit :
Paul Marillonnet a écrit :
Je teste maintenant.
Un premier test, tentative de connexion avec un usager créé à l'aide de la commande de manage "createsuperuser".
Cet usager privilégié n'est rattaché à aucune OU.
Avec l'option de sélecteur d'OU activée, on doit forcément choisir une OU dans la liste.
Donc impossibilité de se connecter pour cet usager "apatride".
Est-ce le comportement souhaité ?
Ouaip c'est un peu un trou dans la raquette mais les utilisateurs créés via tous les autres moyens ont une OU, je te laisse ouvrir un ticket pour dire que createsuperuser devrait foutre le superuser dans l'OU par défaut.
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Lié à Support #30872: Les usagers créés par la commande createsuperuser n'appartiennent à aucune OU. ajouté
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Je continue à croire que les deux paramètres de la fonction lazy_label
seraient mieux une fois inversés.
Pour le reste c'est OK pour moi. J'ai créé le ticket pour gérer le rattachement à l'OU par défaut lors du createsuperuser
.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Echéance mis à 01 mars 2019
Ok à pousser après le 1er mars.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|fae901f5a2b1a6455191f66e67578fa49ef5b05f.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
forms: PEP8ness, style (#30252)