Projet

Général

Profil

Bug #59402

form_role_xxx_name ne renvoie pas tous les rôles associés à une fonction

Ajouté par Stéphane Laget il y a plus de 2 ans. Mis à jour il y a 9 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
07 décembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

On peut maintenant associer plusieurs rôles dans une même fonction, mais la variable form_role_xxx_name ne renvoie que la 1ère fonction.

Un exemple ici :
https://formulaires.simplici.chambery.fr/backoffice/management/je-demande-une-subvention-pour-mon-association-copie-2/389/inspect


Fonction : Services référents => Subventions - Direction jeunesse, vie étudiante & relations internationales, Subventions - Direction événements culturels & congrès

et
{{ form_role_services_referents_name }} => Subventions - Direction jeunesse, vie étudiante & relations internationales

Révisions associées

Révision 08983f04 (diff)
Ajouté par Frédéric Péters il y a 9 mois

misc: add more variables for function multi attributions (#59402)

Historique

#1

Mis à jour par Stéphane Laget il y a plus de 2 ans

un ticket similaire : #57506

#2

Mis à jour par Stéphane Laget il y a plus de 2 ans

  • Sujet changé de form_role_xxx_name ne renvoit pas tous les rôles associés à une fonction à form_role_xxx_name ne renvoie pas tous les rôles associés à une fonction
#5

Mis à jour par Stéphane Laget il y a 11 mois

  • Tracker changé de Development à Bug
#7

Mis à jour par Thomas Noël il y a 11 mois

Le problème c'est qu'on ne peut pas modifier le type des champs form_role_xxx_uuid, form_role_xxx_name, form_role_xxx_emails et form_role_xxx_details, car ils sont déjà utilisés par ci par là.

On ne va pas toucher à ces variables, mais en ajouter d'autres... mais je n'ai pas vraiment d'idée :-/

Ca pourrait être juste form_role_xxx = une liste de dictionnaires avec des clés uuid, name, emails, details ?

Avec un tel objet, pour afficher les noms des rôles liés, il faudrait faire des choses du genre :

{% for role un form_role_foobar %}
* {{ role.name }}
{% endfor %}

Avis bienvenus.

#8

Mis à jour par Frédéric Péters il y a 11 mois

  • Assigné à mis à Frédéric Péters

De manière plus limitée, ça m'irait de simplement inclure plusieurs noms de rôle dans form_role_xxx_name, 1/ dans les deux tickets liés on est uniquement sur de l'affichage, 2/ l'assignation multiple reste peu commune.

Dans le cas où une fonction contient à la fois des rôles et des usagers on a d'ailleurs déjà form_role_xxx_name à "a, b, c"; d'un coup d'œil s'agirait de :

-        if any(x for x in role_ids if str(x).startswith('_user:')):
-            # there's some direct user attribution, only get names
+        if len(role_ids) > 1:
+            # there's multiple attributions, only get names

Il faut accepter dans ce cas qu'on perd uuid et details.

Pour les uuid ça pourrait être bricolé, on pourrait avoir un form_role_xxx_uuids (pluriel) avec la liste.

~~

Mais si on veut aller plus loin, je serais alors pour tout changer, prendre plus large et créer form_function_xxx, où on pourra correctement traiter davantage de questions, les attributions multiples, aussi bien vers des rôles que des usagers, mais également pouvoir fournir le nom de la fonction, ce qui me semble parfois avoir été demandé. (mais ça bouscule davantage).

#9

Mis à jour par Robot Gitea il y a 10 mois

  • Statut changé de Nouveau à En cours

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#10

Mis à jour par Frédéric Péters il y a 10 mois

  • Statut changé de En cours à Solution proposée
#11

Mis à jour par Robot Gitea il y a 10 mois

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

Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :

#12

Mis à jour par Robot Gitea il y a 9 mois

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

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#13

Mis à jour par Transition automatique il y a 9 mois

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

Mis à jour par Transition automatique il y a 7 mois

Automatic expiration

Formats disponibles : Atom PDF