spip-contrib-extensions/formidable | 2 commits
Par Maïeul Rouquette, le 5 mai 2022 à 17h13min :
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_arobasesformidable_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().
Modifié
formidable_fonctions.php
inc/formidable.php
paquet.xml
==============================
Par Maïeul Rouquette, le 5 mai 2022 à 17h13min :
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_arobasesformidable_post_raccourcis_arobases
On supprime donc le pipeline formidable_affiche_resume_reponse.
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 (
$valeurset
$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 pipelineformidable_affiche_resume_reponse. Puisqu’on supprime ce pipeline, plus besoin de ces valeurs. - On regroupe tout les paramètres en troisième arguments, dans
$options, tout en assurant une rétrocompatibilité (pas d’autre usage de
formidable_raccourcis_arobases_2_valeurs_champs()sur la zone, mais j’ai
du code perso qui l’utilise, et il n’est pas impossible que d’autres
fassent de même. - On ajoute une option
'contexte'pour indiquer le contexte d’appel
deformidable_raccourcis_arobases_2_valeurs_champs, ce qui permet de
remplacer utilement le pipelineformidable_affiche_resume_reponse.
Modifié
formidable_fonctions.php
formulaires/formidable.php
inc/formidable.php
paquet.xml
traiter/email.php