Skip to content

Pipeline supplémentaire et leger nettoyage

Maïeul requested to merge gh-fdf4c590/105/unknown/refs/pull/105/head into master

Message de commit

Il existait un pipeline formidable_affiche_resume_reponse, utilisé uniquement par le plugin formidable_quizz, actuellement non maintenu et non publié.

Ce pipeline n'était pas générique, car le remplacement des @@ ne concerne pas que l'affichage résumé des réponses, mais aussi en différent endroit (message de retour par exemple).

On créé deux pipelines plus générique :

  • formidable_pre_raccourcis_arobases
  • formidable_post_raccourcis_arobases

On simplifie par ailleurs la signature de la fonction formidable_raccourcis_arobases_2_valeurs_champs() en supprimant les deux derniers arguments passés par références ($valeurs et $valeurs_libellees).

La seule raison de ce passage par référence, qui n'était utilisé sur toute la zone que dans UN appel à la fonction, était précisement de passer les valeurs libellées en arguments du pipeline formidable_affiche_resume_reponse. Or, pour l'heure, le seul usage de ce pipeline n'utilise précisement pas ces valeurs.

Les personnes qui auraient besoin de ces valeurs peuvent utiliser calculer_voir_reponse().

Ping

@rastapopoulos parce que le pipeline formidable_affiche_resume_reponse vient probablement de toi

Complément

Cas d'usage : #102 (closed), mis en œuvre dans https://git.spip.net/spip-contrib-extensions/formidable_retour_etendu

Merge request reports