spip-contrib-extensions/formidable | 2 commits
Par Maïeul Rouquette, le 22 février 2025 à 23h11min :
remove: appel à un pipeline inutile et piégeux
- L’ordre des traitements par défaut correspond à l’ordre stockés en base, hors
lors de la configuration des traitements, on appel déjà le pipeline - La structure du tableau passé ne correspond pas aux autres appels au
pipeline. Normalement c’est les traitements + la description des
options + la description du traitement, là c’est juste les
traitements + les valeurs des options. Donc ca peut
potentiellement faire planter des pipelines qui ajouterait des
options à un traitement - Le seul cas où cela pouvait éventuellement influencer c’est lorsqu’on
avait ajouté ces lignes il y a x années … cela permettait que les
anciennes configs soient automatiquement ajustées et de passer
l’envoi d’email après l’enregistrement. Deux solutions
a. On fait un script de migration pour les rares cas des
formulaires dont les traitements n’auraient pas été édités depuis.
b. Tant pis, on considère que cela ne vaut pas la peine de perdre
de l’énergie à un script de migration, vu que la seule plus value
de ce truc c’était de permettre d’avoir un lien vers la réponse
côté privé dans l’email.
J’ai tendance à opter pour b.
Modifié
CHANGELOG.md
formulaires/formidable.php
==============================
Par Maïeul Rouquette, le 22 février 2025 à 23h07min :
remove: code inutile
Modifié
formulaires/formidable.php
Détails : remove: code inutile (80d3eaea) · Validations · spip-contrib-extensions / formidable · GitLab