Development #64723
généraliser l'utilisation de append_item
0%
Description
Dans les tests principalement (mais aussi la construction des workflows par défaut) on a le plus souvent :
action = Action() action.id = 'xxx' […] # paramétrage action.parent = status status.items.append(action)
Il existe déjà une méthode append_item() qui fait le boulot combiné 1/ recherche de l'action via son slug, 2/ assignation d'un identifiant automatiquement, 3/ assignation parent, 4/ ajout à la liste.
Fichiers
Demandes liées
Révisions associées
workflows: rename append_item to add_action and use it everywhere (#64723)
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-workflows-use-.append_item-to-create-default-workflo.patch 0001-workflows-use-.append_item-to-create-default-workflo.patch ajouté
- Fichier 0002-workflows-rename-append_item-to-add_action-and-use-i.patch 0002-workflows-rename-append_item-to-add_action-and-use-i.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
0001 c'était ma motivation initiale sur les workflows par défaut, avec pour l'occasion la mise en commun du code qui était répété entre statut et action globale (dans un nouveau "SerieOfActionsMixin"), + l'ajout d'un paramètre id nécessaire pour les workflows par défaut (en pratique il n'y a rien qui devrait dépendre de ça mais autant ne pas prendre le risque).
0002 c'est l'affaire monstrueuse quand je décide d'également adapter les tests; dans ceux-ci il y a quelques endroits où l'action ajoutée doit être posée en premier, donc je renomme append_item en add_action et j'ajoute un paramètre prepend=False. (35 files changed, 491 insertions(+), 1627 deletions(-)).
Mis à jour par Frédéric Péters il y a environ 2 ans
- Lié à Development #64722: déplacer toutes les actions de workflows dans des modules sous wcs/wf/ ajouté
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
0001 : j'aurais bien retiré le "Mixin" à la fin du nom "SerieOfActionsMixin", mais allez, soit.
0002 : ce monstre... je vais avouer ne pas avoir tout relu dans tests/. Pour le reste, le "add_action" est effectivement plus joli.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4470d1e76cf21a5c151144b9ff3d3759bb00f8ce
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun May 1 17:07:31 2022 +0200
workflows: rename append_item to add_action and use it everywhere (#64723)
commit 4c62cdaa555d54e76df4ed1cc744282a16e80e4b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun May 1 12:56:07 2022 +0200
workflows: use .append_item to create default workflows (#64723)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
workflows: use .append_item to create default workflows (#64723)