Projet

Général

Profil

Development #59348

Action "Rattachement d’un usager à une fiche" : compléter l'intitulé "Rattacher les fiches liées"

Ajouté par Marie Kuntz il y a plus de 2 ans. Mis à jour il y a presque 2 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Dans l'action "Rattachement d’un usager à une fiche", l'intitulé "Rattacher les fiches liées" n'est à mon avis pas clair, il faudrait compléter avec (par exemple) : "Rattacher les fiches liées au workflow"


Fichiers

Révisions associées

Révision 9e3c9ec3 (diff)
Ajouté par Paul Marillonnet il y a presque 2 ans

wf: clarify targetting labels for actions on linked forms/cards (#59348)

Historique

#1

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

c'est plus "Rattacher les fiches liées à la demande/fiche"

#2

Mis à jour par Marie Kuntz il y a plus de 2 ans

Oui ok, ça me va

#3

Mis à jour par Paul Marillonnet il y a plus de 2 ans

#4

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

On a déjà des "linked cards" ou "linked cards/forms" sur edit_carddata.py ou external_workflow.py et personne n'a pleuré. Il faut modifier aussi, en tout cas.

(Perso je vois pas à quoi "liées" pourrait faire référence autre que la demande présente, donc ça me semble pas vraiment utile, mais je suis chaffoin)

#5

Mis à jour par Marie Kuntz il y a plus de 2 ans

Thomas Noël a écrit :

(Perso je vois pas à quoi "liées" pourrait faire référence autre que la demande présente, donc ça me semble pas vraiment utile, mais je suis chaffoin)

C'est moi, je pige pas et si je pige pas, c'est que c'est pas clair.

#6

Mis à jour par Paul Marillonnet il y a plus de 2 ans

Marie Kuntz a écrit :

C'est moi, je pige pas et si je pige pas, c'est que c'est pas clair.

Quelque chose comme ça, ce serait plus clair ?

#7

Mis à jour par Pierre Cros il y a plus de 2 ans

Tant que t'es là dedans attention parce que le deuxième libellé est aujourd'hui utilisé sur l'action workflow externe et il est faux :
"Préciser l’identifiant de la demande/fiche sur laquelle l’action va s’exécuter"
En réalité on peut ici spécifier une liste de demandes/fiches (un objet liste django).

Je ne sais pas si c'est le cas aussi pour les autres actions (modifier une fiche, créer une fiche, rattacher une fiche à un utilisateur)

#8

Mis à jour par Pierre Cros il y a plus de 2 ans

Pierre Cros a écrit :

Je ne sais pas si c'est le cas aussi pour les autres actions (modifier une fiche, créer une fiche, rattacher une fiche à un utilisateur)

Je précise : je ne sais pas si ces actions fonctionnent en mettant un objet liste à la place d'un simple numéro.

#9

Mis à jour par Lauréline Guérin il y a plus de 2 ans

Je précise : je ne sais pas si ces actions fonctionnent en mettant un objet liste à la place d'un simple numéro.

Non, en mode manuel on ne peut préciser d'un id simple, ou une expression donnant un unique id, ou un formdata, ... mais un seul objet au final

#10

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

et donc il faudrait faire un nouveau ticket pour que cette action fonctionne comme les autres, et acceptent un objet liste ?

#11

Mis à jour par Paul Marillonnet il y a plus de 2 ans

Stéphane Laget a écrit :

et donc il faudrait faire un nouveau ticket pour que cette action fonctionne comme les autres, et acceptent un objet liste ?

#59396

#13

Mis à jour par Lauréline Guérin il y a plus de 2 ans

les autres non plus n'acceptent pas de liste

je relis le code:
en mode manuel on peut y mettre:
- un formdata (donc simple)
- une chaine de caractères qui, évaluée (template) donne un id (donc simple), exemples: '{{ form_var_string }}', '42', ...
- un "LazyFormDefObjectsManager", exemple: '{{cards|objects:"foo"}}', '{{cards|objects:"foo"|filter_by:"card_string"|filter_value:"Text 2"}}', ... (du coup multiple)

le mode manuel ou automatique est commun aux actions "workflow externe", "update carddata" et la nouvelle action d'assignation

donc oui, désolée pour le bruit, on peut avoir une liste, à condition d'avoir configuré un cards|objects (ou un forms|objects pour l'action workflow externe)

(l'action créer une fiche est à part, elle crée une fiche)

#14

Mis à jour par Pierre Cros il y a plus de 2 ans

Pour résumer la situation actuelle :

Acceptent un objet liste :
  • l'action "WF externe"
  • L'action "Modifier une fiche"
  • L'action "Rattacher un utilisateur à une fiche"
N'accepte pas de liste (ne concerne qu'une fiche) :
  • l'action "Créer une fiche"

Je pense qu'on peut rejeter #59396

#15

Mis à jour par Paul Marillonnet il y a plus de 2 ans

Lauréline Guerin a écrit :

donc oui, désolée pour le bruit, on peut avoir une liste, à condition d'avoir configuré un cards|objects (ou un forms|objects pour l'action workflow externe)

No worries, j’aurais pu aller voir le code pour m’en assurer. Merci pour l’éclaircissement.

#16

Mis à jour par Paul Marillonnet il y a plus de 2 ans

  • Statut changé de Solution proposée à En cours
  • Assigné à mis à Paul Marillonnet

Pierre Cros a écrit :

Pour résumer la situation actuelle :

Acceptent un objet liste :
  • l'action "WF externe"
  • L'action "Modifier une fiche"
  • L'action "Rattacher un utilisateur à une fiche"
N'accepte pas de liste (ne concerne qu'une fiche) :
  • l'action "Créer une fiche"

Je pense qu'on peut rejeter #59396

Ok, le patch de ce ticket doit être modifié en conséquence et je rejette #59396.

#18

Mis à jour par Frédéric Péters il y a presque 2 ans

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

Mis à jour par Frédéric Péters il y a presque 2 ans

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

Je l'ai poussé,

commit 9e3c9ec3a9e596db89fba2365f5d621cdbff491d
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Mon Dec 6 11:31:23 2021 +0100

    wf: clarify targetting labels for actions on linked forms/cards (#59348)
#20

Mis à jour par Transition automatique il y a presque 2 ans

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

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF