Bug #20511
BO: dans la vue des rôles, on devrait voir la listes des collectivités dans lesquels on va trouver des rôles
100%
Description
Si un utilisateur n'a de permission que sur un rôle, la liste sera vide alors qu'on devrait y voir l'OU de ce rôle.
Si une collectivité n'a aucun rôle elle y sera quand même alors que cela ne sert à rien.
Ce patch corrige ces deux points.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Fichier 0001-manager-show-all-role-s-OUs-visible-to-user-fixes-20.patch 0001-manager-show-all-role-s-OUs-visible-to-user-fixes-20.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 6 ans
Si une collectivité n'a aucun rôle elle y sera quand même alors que cela ne sert à rien.
Je ne suis pas sûr de ce point; ça me semble plus clair d'avoir toutes les collectivités. (et définitivement utile si on adopte pour le "Ajouter le rôle" le même fonctionnement "grisé" quand l'OU n'est pas sélectionnée).
Mis à jour par Mikaël Ates il y a plus de 6 ans
- Lié à Development #20182: Choix de la collectivité lors de l'ajout d'un rôle ajouté
Mis à jour par Mikaël Ates il y a plus de 6 ans
Je ne suis pas sûr de ce point; ça me semble plus clair d'avoir toutes les collectivités. (et définitivement utile si on adopte pour le "Ajouter le rôle" le même fonctionnement "grisé" quand l'OU n'est pas sélectionnée).
Pareil, je lie #20182.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Fichier 0001-manager-show-all-role-s-OUs-visible-to-user-fixes-20.patch 0001-manager-show-all-role-s-OUs-visible-to-user-fixes-20.patch ajouté
Frédéric Péters a écrit :
Si une collectivité n'a aucun rôle elle y sera quand même alors que cela ne sert à rien.
Je ne suis pas sûr de ce point; ça me semble plus clair d'avoir toutes les collectivités. (et définitivement utile si on adopte pour le "Ajouter le rôle" le même fonctionnement "grisé" quand l'OU n'est pas sélectionnée).
Tu as raison, j'ai modifié le code de OUSearchForm pour combiner la liste des OUs obtenue via les permissions par OUs avec la liste des OUs obtenue via le listing des objets réellement visible, de cette manière on a tout.
Mis à jour par Frédéric Péters il y a plus de 6 ans
s/event/even/ dans le commentaire mais à mon sens Django devrait assurer l'ordre du tri. s/distinc/distinct/ dans un autre commentaire.
ack avec ça.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|57588c4e5df0c2664064d03d2effc67d7b77d634.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
manager: show all role's OUs visible to user (fixes #20511)