Bug #20512
BO: le lien de création des rôles s'affiche aux utilisateurs qui ne peuvent administrer qu'un rôle et certainement pas en ajouter
100%
Description
La permission d'administration d'un unique rôle se nomme admin comme la permission d'administrer tous les rôles (et donc d'en ajouter) ce qui fausse le résultat de request.user.has_any_perm('a2_rbac.add_role'). Ce patch n'est qu'un contournement de ce problème, une vraie correction nécessitera de créer une nouvelle permission plus fine ne désignant que le fait de gérer les membres d'un rôle, par exemple nommée manage-members.
Fichiers
Demandes liées
Révisions associées
Revert "manager: do not use has_any_perm() to get add permission on roles (fixes #20512)"
This reverts commit 1972076bfd4f69cf1bb277ce59b19a802b0a7a40.
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Lié à Development #20513: Ajouter une permission explicite pour gérer les membres d'un rôle ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Fichier 0001-manager-do-not-use-has_any_perm-to-get-add-permissio.patch 0001-manager-do-not-use-has_any_perm-to-get-add-permissio.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 6 ans
De mon côté je peux ici juste faire confiance, je ne mesure pas.
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|1972076bfd4f69cf1bb277ce59b19a802b0a7a40.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
manager: do not use has_any_perm() to get add permission on roles (fixes #20512)
This is a temporary fix, the real fix would be to create a real permission to
manage members of a roles so that role's admin roles would not have the admin
permission but the manage-members permission, so that for an user which can just
manager members of a role, request.user.has_any_perm('a2_rbac.add_role') would
return False, currently it returns True but it has no meaning.