[SPIP Zone] FORMIDABLE =?utf-8?Q?p=C3=A9t=C3=A9._C=E2=80=99est_?=pas formidable…

Hello,

à la faveur d’un upgrade on découvre (je crédite au passage Bennyb qui a dépeloté le bug) le commit

https://zone.spip.net/trac/spip-zone/browser/spip-zone/plugins/saisies/trunk/saisies/hidden.php?rev=112440

qui casse totalement l’usage licite et bien fichu

#FORMULAIRE_FORMIDABLE{#ID_FORMULAIRE,#ARRAY{hidden_1,#ID_AUTEUR}}

à savoir, il me semble, si je ne parle pas trop vite sans avoir réfléchi, si vous voulez bien me l'accorder, qu'il y a une erreur de conception :

on ne peut pas refuser une valeur qui est passée par défaut à l'appel du formulaire !

Je ne sais pas si ça se limite aux hidden : le passage de valeur initiale à l'appel du formulaire est un outil qui permet de dynamiser certain formulaires alors que la configuration est par essence fixe.

Il me semble donc que si on va par là, il faut soit ajouter un champ spécifique pour la vérification de valeurs acceptables, dans lequel on peut fournir tout plein de choses (une énumération, une regexp, rien…), soit a minima mettre une case à cocher pour skipper (ou activer ?) cette vérification

Il y a eu un groupe de discussion pour valider ça ? :smiley:

--
Cédric

oui,
effectivement c'est une erreur. la première solution me parait la plus
propre. La seconde est un bugfix rapide.
Le jeudi 11 avril 2019 à 15:36 +0200, Cerdic a écrit :

Hello,

à la faveur d’un upgrade on découvre (je crédite au passage Bennyb qui
a dépeloté le bug) le commit

https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/saisies/trunk/saisies/hidden.php?rev=112440

qui casse totalement l’usage licite et bien fichu

#FORMULAIRE_FORMIDABLE{#ID_FORMULAIRE,#ARRAY{hidden_1,#ID_AUTEUR}}

à savoir, il me semble, si je ne parle pas trop vite sans avoir
réfléchi, si vous voulez bien me l'accorder, qu'il y a une erreur de
conception :

on ne peut pas refuser une valeur qui est passée par défaut à l'appel
du formulaire !

Je ne sais pas si ça se limite aux hidden : le passage de valeur
initiale à l'appel du formulaire est un outil qui permet de dynamiser
certain formulaires alors que la configuration est par essence fixe.

Il me semble donc que si on va par là, il faut soit ajouter un champ
spécifique pour la vérification de valeurs acceptables, dans lequel on
peut fournir tout plein de choses (une énumération, une regexp,
rien…), soit a minima mettre une case à cocher pour skipper (ou
activer ?) cette vérification

Il y a eu un groupe de discussion pour valider ça ? :smiley:

--
Cédric

Solution rapide discutée :

1) Rétablir l'ancien comportement avec aucune vérif de légitimité des
valeurs par défaut.

2) Ajouter une option globale (maintenant qu'il y a l'interface pour ça)
permettant d'activer cette vérification sur l'ensemble du formulaire.

3) On verra dans un deuxième temps pour le permettre plus finement champ
par champ si besoin…

--
RastaPopoulos

Ça parait un compromis acceptable, merci !

--
Cédric
Le 11 avr. 2019 à 16:34 +0200, RastaPopoulos <rastapopoulos@spip.org>, a écrit :

Solution rapide discutée :

1) Rétablir l'ancien comportement avec aucune vérif de légitimité des
valeurs par défaut.

2) Ajouter une option globale (maintenant qu'il y a l'interface pour ça)
permettant d'activer cette vérification sur l'ensemble du formulaire.

3) On verra dans un deuxième temps pour le permettre plus finement champ
par champ si besoin…

--
RastaPopoulos

----
spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

Le 11/04/2019 à 16:44, Cerdic a écrit :

Ça parait un compromis acceptable, merci !

--

solution appliquée par deux commits, sur saisies et sur formidable.

Le 11/04/2019 à 17:43, Maïeul a écrit :

solution appliquée par deux commits, sur saisies et sur formidable.

Super merci !

Par contre c'est bizarre, tu as mis les chaines de langue dans Saisies,
alors qu'elles n'y sont pas utilisées. Et l'option ensuite est dans
Formidable forcément (comme celles de l'activation des étapes etc).

À priori il faut mettre les chaines comme les autres options globales,
càd dans Formidable, non ?
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/formidable/trunk/lang/formidable_fr.php?rev=114932#L60

--
RastaPopoulos

Le 11/04/2019 à 18:16, RastaPopoulos a écrit :

Le 11/04/2019 à 17:43, Maïeul a écrit :

solution appliquée par deux commits, sur saisies et sur formidable.

Super merci !

Par contre c'est bizarre, tu as mis les chaines de langue dans Saisies,
alors qu'elles n'y sont pas utilisées. Et l'option ensuite est dans
Formidable forcément (comme celles de l'activation des étapes etc).

À priori il faut mettre les chaines comme les autres options globales,
càd dans Formidable, non ?
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/formidable/trunk/lang/formidable_fr.php?rev=114932#L60

j'avais hésité, mais je me suis dit qu'en mettant dans saisies on pourrait mutualiser si d'autres plugins à constructeur de formulaire en ont besoin.