Projet

Général

Profil

Bug #43269

Affichage des héritages de rôles - Multicollectivité

Ajouté par Stéphane Laget il y a presque 4 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
25 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Dans un contexte de multi-collectivités, il serait intéressant de voir la collectivité en préfixe du nom du rôle dans la partie "contient les permissions des rôles".

cf capture

Par exemple, le rôle sur la capture contient 7 fois le rôle "Agent", il s'agit de Bron:Agent, Oullins:Agent, Corbas:Agent, etc.

Ce serait pratique de le voir dès cet écran.


Fichiers

Révisions associées

Révision d1dda9c3 (diff)
Ajouté par Nicolas Roche il y a plus de 3 ans

manager: display parent roles OU on the role members page (#43269)

Historique

#1

Mis à jour par Nicolas Roche il y a plus de 3 ans

(un premier avis ?)

#2

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

Ajoute une petite copie d'écran, ça aidera.

is_several_parent_ous : non, il faut afficher l'OU dès qu'il y en a plusieurs sur Authentic et pas seulement dans la liste, donc quelque chose comme "multiple_ou" (je suis pas doué en anglais, y'a sans doute plus clair à écrire)

Et pour le code HTML tu peux faire plus court :

       {% if multiple_ou and parent.ou %}{{ parent.ou }} - {% endif %}{{ parent }}

#3

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

Si je comprends bien tu fais deux choses :

.order_by(F('ou').desc(nulls_first=True)))

Trier les rôles par OU. Ok pourquoi pas.

ctx['is_several_parent_ous'] = len(set(x.ou for x in self.object.parents(include_self=False)))

Quelque chose pour déterminer s'il y a plusieurs OU. C'est plutôt illisible je trouve. Je serais pour un simple OU.objects.count() > 1. Mais il y a peut-être des considérations de visiblité sur les OU à avoir et ça se trouve peut-être géré par ton code, no se.

(s/is_/has_/).

#4

Mis à jour par Nicolas Roche il y a plus de 3 ans

Il y a peut-être des considérations de visibilité sur les OU

Non, je n'ai pas pris en compte le cas des usagers qui ne voient qu'une seule OU.

#5

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

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

Validé poussé, j'y ai ajouté un tri secondaire sur le nom du rôle.

commit d1dda9c39239d0c9d9201bf426e149b6234c9bfb
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Fri Jul 31 17:08:36 2020 +0200

    manager: display parent roles OU on the role members page (#43269)
#6

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

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

Formats disponibles : Atom PDF