Projet

Général

Profil

Development #38621

ne pas considérer l'info "erreur appel webservice" présente mais cachée dans l'historique, pour l'affichage "groupé" des statuts

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

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Une demande qui se trouve avec une succession de statuts identiques et vides affichera uniquement le premier passage + une info "(dernière vérification: date/heure)".

Mais si ce n'est pas vide, que ça a evolution.parts = [JournalWsCallErrorPart(...)] parce qu'il y a eu échec d'un appel webservice, il y aura une entrée par "evolution" dans l'historique, y compris en front où le JournalWsCallErrorPart() ne s'affiche pas.


Fichiers


Demandes liées

Lié à w.c.s. - Development #54497: log de debug des actions de workflowFermé01 juin 2021

Actions

Révisions associées

Révision e8926cd0 (diff)
Ajouté par Frédéric Péters il y a presque 3 ans

forms: ignore hidden evolution parts when joining statuses in history (#38621)

Historique

#2

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

Ajout d'une méthode is_hidden, définie pour le JournalWsCallErrorPart.

Ça demande un petit changement dans FormData, au niveau du stockage on n'enregistre pas de ligne d'historique sur un statut vide qui n'a pas changé (#22236), et l'évaluation de "vide" se fait via display_parts(), et comme désormais display_parts() retourne uniquement les bouts vraiment visibles, je change ça pour vérifier totalement .parts à la place. (dans la pratique ça n'a pas de conséquence, comme les JournalWsCallErrorPart étaient de toute façon "visibles", il y avait de toute façon une ligne ajoutée à l'historique). (on pourrait se poser la question d'également réutiliser la ligne "evolution" existante dans cette situation, ça ferait grossir la colonne "parts" plutôt qu'ajouter des lignes)

#3

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

Adaptation au test qui regardait si .view() retournait quelque chose, pour utiliser la nouvelle méthode is_hidden() à la place.

#4

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

#5

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

Rebase pour incorporer le is_hidden en attribut de la nouvelle classe de base.

Ce développement devient utile au-delà des appels webservice, pour qu'ensuite les infos de debug (de #54497) ne viennent pas perturbé l'affichage "groupé" des statuts.

#6

Mis à jour par Emmanuel Cazenave il y a presque 3 ans

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit e8926cd070aed9c02cf2afb92d872fa957be4f59
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Dec 18 14:26:39 2019 +0100

    forms: ignore hidden evolution parts when joining statuses in history (#38621)
#8

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

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

Formats disponibles : Atom PDF