[API+Constructeur] Permettre de pouvoir passer plusieurs vérification #175

Closed
opened 7 months ago by maieul · 3 comments
maieul commented 7 months ago
Owner

CR discussion avec @rastapopoulos

Dans l'API

  • Il faut pouvoir passer un tableau de vérif
  • Garder la possibilité de ne passer qu'une verif (compat historique)
  • concretement if (isset($saisie['verifier']['type']) { $saisie['verifier'] = array($saisie['verifier']) }
  • puis foreach sur $saisie_verifier avec le même code qu'actuellement

Dans le constructeur

  • Si select2 est activé, proposer une saisie selection multiple oour les verifications avec la classe selec2, sinon prendre checkbox
  • Puis proposer les options de chaque verif, dans un fieldset par verif, avec afficher_si
  • ATTENTION : s'assurer d'avoir transformé la valeur deja en base en array, pour ne pas perdre les choses

Et aussi

  • Passer tjr la description de la saisie dans la fonction de verif par exemmple dans $options['_saisie']
  • Creer une verif "valeurs_acceptables"
CR discussion avec @rastapopoulos # Dans l'API * [x] Il faut pouvoir passer un tableau de vérif * [x] Garder la possibilité de ne passer qu'une verif (compat historique) * [x] concretement `if (isset($saisie['verifier']['type']) { $saisie['verifier'] = array($saisie['verifier']) }` * [x] puis foreach sur `$saisie_verifier` avec le même code qu'actuellement # Dans le constructeur * [x] <del>Si `select2` est activé</del>, proposer une saisie `selection` multiple oour les verifications avec la classe selec2<del>, sinon prendre checkbox</del> * [x] Puis proposer les options de chaque verif, dans un fieldset par verif, avec afficher_si * [x] ATTENTION : s'assurer d'avoir transformé la valeur deja en base en array, pour ne pas perdre les choses # Et aussi * [x] Passer tjr la description de la saisie dans la fonction de verif par exemmple dans ` $options['_saisie']` * [ ] Creer une verif "valeurs_acceptables"
maieul referenced this issue from a commit 7 months ago
Poster
Owner

fait en grand partie ce soir. Reste la verif "valeur_acceptable" à proposer.

@rastapopoulos je propose de forcer l'usage de select2 sur formidable + interface champ extra (les 2 seuls plugins à ma connaissance utilisent le constructeur de formulaire)

fait en grand partie ce soir. Reste la verif "valeur_acceptable" à proposer. @rastapopoulos je propose de forcer l'usage de select2 sur formidable + interface champ extra (les 2 seuls plugins à ma connaissance utilisent le constructeur de formulaire)

mmh pourquoi forcer le nécessite si dans le code on a case ou select suivant la présence ? mais peut-être…

(à terme peut-être bien que le constructeur, qui est une interface visible, pas API, devrait aller dans un plugin dédié, et du coup on mettrait les nécessites dans celui-ci, et Saisies ne serait vraiment que l'API code)

mmh pourquoi forcer le nécessite si dans le code on a case ou select suivant la présence ? mais peut-être… (à terme peut-être bien que le constructeur, qui est une interface visible, pas API, devrait aller dans un plugin dédié, et du coup on mettrait les nécessites dans celui-ci, et Saisies ne serait vraiment que l'API code)
Poster
Owner

mmh pourquoi forcer le nécessite si dans le code on a case ou select suivant la présence ? mais peut-être…

Ah oui, j'ai oublié de préciser un truc dans mon message : j'ai tenté avec checkbox. C'est tout à fait faisable. Mais ca rend juste la liste abonimablement long, avec juste les vérifs de base. Du coup j'ai finalement opté pour une selection dans tous les cas (simplifions le code). Je suis assez convaincu qu'à l'usage select2 est quasi indispensable.

> mmh pourquoi forcer le nécessite si dans le code on a case ou select suivant la présence ? mais peut-être… Ah oui, j'ai oublié de préciser un truc dans mon message : j'ai tenté avec checkbox. C'est tout à fait faisable. Mais ca rend juste la liste abonimablement long, avec juste les vérifs de base. Du coup j'ai finalement opté pour une selection dans tous les cas (simplifions le code). Je suis assez convaincu qu'à l'usage select2 est quasi indispensable.
maieul referenced this issue from a commit 7 months ago
maieul referenced this issue from a commit 7 months ago
maieul closed this issue 7 months ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.