#71 se passer de saisie_est_tabulaire

Open
opened 2 months ago by maieul · 1 comments
maieul commented 2 months ago
Collaborator

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
maieul commented 1 month ago
Poster
Collaborator

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...
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.