Skip to content
Extraits de code Groupes Projets
  • Maïeul's avatar
    fbc15376
    Il existait un pipeline `formidable_affiche_resume_reponse`, utilisé · fbc15376
    Maïeul a rédigé
    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 supprime donc le pipeline `formidable_affiche_resume_reponse`.
    
    On simplifie par ailleurs la signature de la fonction
    `formidable_raccourcis_arobases_2_valeurs_champs()`.
    1. 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`. Puisqu'on supprime ce pipeline, plus besoin de ces valeurs.
    2. 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.
    3. On ajoute une option `'contexte'` pour indiquer le contexte d'appel
    de `formidable_raccourcis_arobases_2_valeurs_champs`, ce qui permet de
    remplacer utilement le pipeline `formidable_affiche_resume_reponse`.
    fbc15376
    Historique
    Il existait un pipeline `formidable_affiche_resume_reponse`, utilisé
    Maïeul a rédigé
    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 supprime donc le pipeline `formidable_affiche_resume_reponse`.
    
    On simplifie par ailleurs la signature de la fonction
    `formidable_raccourcis_arobases_2_valeurs_champs()`.
    1. 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`. Puisqu'on supprime ce pipeline, plus besoin de ces valeurs.
    2. 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.
    3. On ajoute une option `'contexte'` pour indiquer le contexte d'appel
    de `formidable_raccourcis_arobases_2_valeurs_champs`, ce qui permet de
    remplacer utilement le pipeline `formidable_affiche_resume_reponse`.