Projet

Général

Profil

Development #39207

Fabrique de workflow: sur une action de saut, afficher le statut cible

Ajouté par Thomas Noël il y a environ 4 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
23 janvier 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Quand on construit un workflow, une action de saut est affichée sous la forme :

Saut manuel (« libellé du bouton » par Usager)

mais on ne voit pas le nom du statut cible, et quand il est très différent du libellé du bouton (voire contradictoire), c'est facilement perturbant quand on débogue.


Fichiers

Révisions associées

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

admin: detail target status on choice action (#39207)

Historique

#1

Mis à jour par Thomas Noël il y a environ 4 ans

Je propose d'ajouter le nom du statut cible, au risque que ça soit un peu long :

Saut manuel (« libellé du bouton », vers nom du statut cible, par Usager)
#3

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

  • Fichier 0001-admin-detail-target-status-on-choice-action-39207.patch ajouté
  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Nicolas Roche
  • Patch proposed changé de Non à Oui

J'ai un doute parce que personnellement j'ai tendance à nommer le saut avec le nom du statut à atteindre,
ce qui fera un libellé quelque peu redondant, ex: "Saut manuel (« Ressoumission », vers Ressoumission, par Usager).

#4

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

  • Fichier 0001-admin-detail-target-status-on-choice-action-39207.patch supprimé
#6

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

            if self.status:
                to_status = self.parent.parent.get_status(self.status)
            else:
                to_status = self.parent

Je pense que c'est faux, s'il n'y pas de status il faut plutôt afficher l'action comme n'étant pas configurée, parce que ça va passer dans :

    def submit_form(self, form, formdata, user, evo):
        if form.get_submit() == 'button%s' % self.id:
            wf_status = self.get_target_status(formdata)
            if wf_status:

et le self.get_target_status() n'aura rien retourné et donc il ne se passera rien.

#7

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

Si le statut cible n'est pas défini, j'affiche juste "Manual Jump (not completed)",
quand bien même un libellé serait défini.

#8

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

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit b49d4010f7dc31507c2857615d46dbc37e8af944
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Aug 13 14:44:14 2020 +0200

    admin: detail target status on choice action (#39207)
#10

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