se passer de saisie_est_tabulaire #71

Closed
opened 2 years ago by maieul · 2 comments
maieul commented 2 years ago
Owner

Dans la même ligne de simplification que #51, on devrait idéalement pouvoir se passer de saisies_est_tabulaire.

La raison d'être de ce pipeline : si j'ai un name truc[] (cas des checkbox !), si je ne coche rien le nav renvoi null et non pas array(). Or on a besoin pour les tests d'afficher_si/le stockage en base dans formidable de faire la distinction. Pour l'instant on s'en sort avec une déclaration manuelle, mais ce n'est pas extraordinaire. Mais je ne vois pas comment faire autrement...

Plugin qui l'utilise:

  • formidable dans traiter/enregistrement
  • saisies dans verifier_afficher_si

Plugin qui utilise le pipeline :

  • agenda

Et c'est tout

Dans la même ligne de simplification que #51, on devrait idéalement pouvoir se passer de saisies_est_tabulaire. La raison d'être de ce pipeline : si j'ai un name `truc[]` (cas des checkbox !), si je ne coche rien le nav renvoi `null` et non pas `array()`. Or on a besoin pour les tests d'afficher_si/le stockage en base dans formidable de faire la distinction. Pour l'instant on s'en sort avec une déclaration manuelle, mais ce n'est pas extraordinaire. Mais je ne vois pas comment faire autrement... Plugin qui l'utilise: - formidable dans traiter/enregistrement - saisies dans verifier_afficher_si Plugin qui utilise le pipeline : - agenda Et c'est tout
Poster
Owner

Oups, non 65f104c858 ne fix pas #71 mais #72...

Oups, non https://git.spip.net/spip-contrib-extensions/saisies/commit/65f104c8583b917d4b3600aee364b4d26d697305 ne fix pas #71 mais #72...
Poster
Owner

Bon, alors

  1. Je ne sais pas comment m'en passer, 11 mois après.
  2. De facto formidable utilise cette fonction :

donc on va dire que la foncton est d'usage autrement que pour les afficher_si, et donc on laisse tomber ce ticket.

Bon, alors 1. Je ne sais pas comment m'en passer, 11 mois après. 2. De facto formidable utilise cette fonction : - https://git.spip.net/spip-contrib-extensions/formidable/src/tag/v5.1.0/formidable_pipelines.php#L363 - https://git.spip.net/spip-contrib-extensions/formidable/src/tag/v5.1.0/traiter/enregistrement.php#L132 donc on va dire que la foncton est d'usage autrement que pour les afficher_si, et donc on laisse tomber ce ticket.
maieul added the
refusé
label 8 months ago
maieul closed this issue 8 months ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.