WIP dev/issues51 #72

Closed
maieul wants to merge 14 commits from dev/issues51 into master
maieul commented 3 months ago
Collaborator

@tcharlss @nicod_ @rastapopoulos une nouvelle implémentation du js d'afficher_si, à tester abondamment

Voir sur #51 le pourquoi du comment. Le but est de se passer d'avoir à utiliser le pipeline saisies_afficher_si_js_type, et de faciliter l'intégration de nouveau type de saisies conditionnante qui peuvent tantot avoir des radios, tantot des select, tantot des checkbox).

Concrètement dans 95% des cas, on utilise afficher_si_js_defaut qui renvoie dans data-afficher-si un truc du genre afficher_si({json}), charge à la fonction de afficher_si d'interprèter le json

J'en ai profité pour

  • nettoyer un peu ce qui est renvoyé par la fonction de parsing, pour enlever des données qui ne servent normalement qu'à la REGEXP (en théorie il faudrait que je nettoie la REGEXP, mais je n'ai pas trouvé pour l'heure de solution alternative à celle que j'ai);
  • rétablir la syntaxe @truc@ > -2, avec nombre négatif donc
  • mettre tout ce qui concerne les testes @fichiers@:TOTAL > 2 dans CVT-Upload (mais du coup il faudra que les gens mettent à jour les 2 en même temps) + debuger car cela ne marchait pas vraiment (du coup ma première parenthèse n'est pas tellement pertinente !)

Reste deux questions ouvertes :

  • l'une qui n'est pas liée au JS, mais à ce qui est envoyà par le nav, et donc au PHP : #71 > on en discute là bas si jamais vous avez une solution
  • l'une qui concerne le cas où une saisie remplie dynamiquement une autre saisie "cachée", comme la saisie evenement, qui peut basculer un hidden <nom_de_la_saisie>_liste_attente. Actuellement il faut déclarer cette pseudo-saisie via saisies_afficher_si_js_saisies_form. Mais ce pipeline n'a que deux usages réels :
    a. il permet dans formidable d'afficher cette pseudo-saisie dans le memento
    b. il permet de faire que si on test @une_saisie_inexistant@ == truc`, le data-afficher-si renvoyé soit vide (tout en générant un log), ce qui me parait plus clair que de generer un test sur un truc inexistant

On pourrait décider de supprimer le point b et de ne garder la pipeline qu'à titre indicatif informatif.

@tcharlss @nicod_ @rastapopoulos une nouvelle implémentation du js d'afficher_si, à tester abondamment Voir sur #51 le pourquoi du comment. Le but est de se passer d'avoir à utiliser le pipeline `saisies_afficher_si_js_type`, et de faciliter l'intégration de nouveau type de saisies conditionnante qui peuvent tantot avoir des radios, tantot des select, tantot des checkbox). Concrètement dans 95% des cas, on utilise `afficher_si_js_defaut` qui renvoie dans data-afficher-si un truc du genre `afficher_si({json})`, charge à la fonction de afficher_si d'interprèter le json J'en ai profité pour - nettoyer un peu ce qui est renvoyé par la fonction de parsing, pour enlever des données qui ne servent normalement qu'à la REGEXP (en théorie il faudrait que je nettoie la REGEXP, mais je n'ai pas trouvé pour l'heure de solution alternative à celle que j'ai); - rétablir la syntaxe `@truc@ > -2`, avec nombre négatif donc - mettre tout ce qui concerne les testes `@fichiers@:TOTAL > 2` dans CVT-Upload (mais du coup il faudra que les gens mettent à jour les 2 en même temps) + debuger car cela ne marchait pas vraiment (du coup ma première parenthèse n'est pas tellement pertinente !) Reste deux questions ouvertes : - l'une qui n'est pas liée au JS, mais à ce qui est envoyà par le nav, et donc au PHP : #71 > on en discute là bas si jamais vous avez une solution - l'une qui concerne le cas où une saisie remplie dynamiquement une autre saisie "cachée", comme la saisie evenement, qui peut basculer un hidden `<nom_de_la_saisie>_liste_attente`. Actuellement il faut déclarer cette pseudo-saisie via `saisies_afficher_si_js_saisies_form`. Mais ce pipeline n'a que deux usages réels : a. il permet dans formidable d'afficher cette pseudo-saisie dans le memento b. il permet de faire que si on test `@une_saisie_inexistant@ == `truc`, le data-afficher-si renvoyé soit vide (tout en générant un log), ce qui me parait plus clair que de generer un test sur un truc inexistant On pourrait décider de supprimer le point b et de ne garder la pipeline qu'à titre indicatif informatif.
Poster
Collaborator

Bon,
après 4c480b2a02, je crois qu'on est pas mal.

En tout cas j'ai testé sur un formulaire assez complexe, et cela me parait suffisament stable pour que je prenne le risque de soumettre cela sur nos formulaires en ligne à Planète Sciences. Wait and see.

Bon, après 4c480b2a02b437dc7d46eba4c23809d9e58f6582, je crois qu'on est pas mal. En tout cas j'ai testé sur un formulaire assez complexe, et cela me parait suffisament stable pour que je prenne le risque de soumettre cela sur nos formulaires en ligne à Planète Sciences. Wait and see.
Poster
Collaborator

Des retours ? pour info c'est déployé sur notre site associatif, et visiblement les affiches conditionnels sont bien passé

Des retours ? pour info c'est déployé sur notre site associatif, et visiblement les affiches conditionnels sont bien passé

non je ne l'utilise pas souvent, et j'ai juste un client qui a vraiment un gros Formidable + un profil avec champs extras, les deux avec beaucoup de afficher_si, mais pas touché du tout encore, donc pas testé

non je ne l'utilise pas souvent, et j'ai juste un client qui a vraiment un gros Formidable + un profil avec champs extras, les deux avec beaucoup de afficher_si, mais pas touché du tout encore, donc pas testé
maieul force-pushed dev/issues51 from 4c480b2a02 to 544046a2b2 3 months ago
Poster
Collaborator

Suite à e4903b5 j'ai rebasé la branche et fait un push force. Si vous avez pas commité entre entre temps (je ne pense pas) il suffit de faire un git pull.

Suite à e4903b5 j'ai rebasé la branche et fait un push force. Si vous avez pas commité entre entre temps (je ne pense pas) il suffit de faire un git pull.
Poster
Collaborator

du coup suite à d2b5c9eb38, on doit garder afficher_si_js_saisies_form, mais il faudrait renommer. On fera cela avant de merger+je ferais les changement sur les quelques dépôts qui l'utilisent.

du coup suite à d2b5c9eb3869151046c49914ad5e68585b081bbb, on doit garder afficher_si_js_saisies_form, mais il faudrait renommer. On fera cela avant de merger+je ferais les changement sur les quelques dépôts qui l'utilisent.
maieul force-pushed dev/issues51 from 544046a2b2 to f5efe33a4e 2 months ago
maieul force-pushed dev/issues51 from 9f70aedcbb to 6b8d437e13 2 months ago
maieul force-pushed dev/issues51 from 6b8d437e13 to 35335da649 2 months ago
maieul force-pushed dev/issues51 from 35335da649 to 70f9fe6094 2 months ago
Poster
Collaborator

C'est cloturé et mergé en ff avec un gros commit unitaire. Merci @nicod_ pour les tests !

C'est cloturé et mergé en ff avec un gros commit unitaire. Merci @nicod_ pour les tests !
maieul closed this pull request 2 months ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.