Hello,
Je ne comprends pas bien le passage d’une version 3.7.2 à 4.0.0 sans faire en sorte de créer une branche v3 au préalable.
Je pense qu’il faut créer une branche v3 à partir du commit précédent surtout si on considère que le changement est majeur et peut impacter les formulaires existant.
Après peut-être que pour une modification majeure on pourrait avoir une petite discussion auparavant.
Mais elle a peut-être eu lieu et dans ce cas désolé pour la remarque.
Salut,
la discussion a eu lieu avec Rastapopoulos en informel, qui a approuvé
le principe. Mais effectivement pas en public donc c'est de ma faute.
Concernant la branche : j'ai peut être été un peu trop alarmiste. En
fait comme cela ne s'applique qu'a des cas explicitement définie, cela
ne devrait pas casser les choses (par ex : cela ne va pas bloquer les
saisies pour lesquel je n'ai pas fait de fonction de tests).
Honnetemnt je ne vois pas de raison qu'il y ait un blocage, mais j'ai
quand même préféré marquer le coup.
Mais sur le fond tu as raison : il faut créer une branche. Je m'en
charge.
Maïeul
Le jeudi 15 novembre 2018 à 16:05 +0100, Eric Lupinacci a écrit :
Hello,
Je ne comprends pas bien le passage d'une version 3.7.2 à 4.0.0 sans
faire en sorte de créer une branche v3 au préalable.
Je pense qu'il faut créer une branche v3 à partir du commit précédent
surtout si on considère que le changement est majeur et peut impacter
les formulaires existant.Après peut-être que pour une modification majeure on pourrait avoir
une petite discussion auparavant.
Mais elle a peut-être eu lieu et dans ce cas désolé pour la remarque.++
EricLe jeu. 15 nov. 2018 à 13:48, <spip-zone-commit@rezo.net> a écrit :
> Author: maieul@maieul.net
> Date: 2018-11-15 12:48:41 +0000 (Thu, 15 Nov 2018)
> New Revision: 112440
>
> Added:
> _plugins_/saisies/trunk/saisies/case.php
> _plugins_/saisies/trunk/saisies/checkbox.php
> _plugins_/saisies/trunk/saisies/date.php
> _plugins_/saisies/trunk/saisies/email.php
> _plugins_/saisies/trunk/saisies/hidden.php
> _plugins_/saisies/trunk/saisies/input.php
> _plugins_/saisies/trunk/saisies/oui_non.php
> _plugins_/saisies/trunk/saisies/radio.php
> _plugins_/saisies/trunk/saisies/selection.php
> _plugins_/saisies/trunk/saisies/selection_multiple.php
> _plugins_/saisies/trunk/saisies/textarea.php
> Modified:
> _plugins_/saisies/trunk/inc/saisies.php
> _plugins_/saisies/trunk/inc/saisies_lister.php
> _plugins_/saisies/trunk/lang/saisies_fr.php
> _plugins_/saisies/trunk/paquet.xml
> _plugins_/saisies/trunk/saisies/hidden.html
> Log:
> <!> gros changement
> Lors de saisies_verifier():
> - vérifier que la valeur postée correspond aux valeurs définies
> comme
> possibles lors de création de la saisie (pour éviter que des gens
> modifient les valeurs possibles via par ex un inspecteur html)
> - soit une des valeurs définies dans data pour les radio, cases,
> select, etc
> - soit une valeur fixée pour les hidden, mais aussi pour les input
> readonly etc.
> - concrètement appelle à une fonction
> saisies_verifier_valeurs_acceptables() qui appelle
> <type-de-la-saisie>_verifier_valeurs_acceptables()
> - cette dernière fonction est définie dans un fichier
> saisies/<type-de-la-saisie>.php
> - elle peut s'appuyer sur les fonctions
> - saisies_verifier_gel_saisie() qui retourne true si la saisie est
> en
> readonly ou bien en disabled avec post
> - saisies_trouver_data() qui renvoie l'option data ou datas de la
> saisie, formatée sous forme de tableau
>
> Pour le moment, on applique cela uniquement aux saisies:
> - avec un .yaml
> - qui ne concernent pas des objets en BDD
> Ce qui devrait couvrir une grosse partie des cas.
>
> Testé avec 19 formulaires Formidable.
> Comme toutefois cela pourrait potentiellement casser certains
> formulaires mal fichus,
> on passe en 4.0.0 par sécurité.
>
> On ajoute un peu de narcissime dans paquet.xml
>
>
> Details: Connexion · GitLab
>
> _______________________________________________
> Spip-zone-commit@rezo.net -
> https://listes.rezo.net/mailman/listinfo/spip-zone-commit
Le 15/11/2018 à 16:14, Maïeul Rouquette a écrit :
Honnetemnt je ne vois pas de raison qu'il y ait un blocage, mais j'ai
quand même préféré marquer le coup.Mais sur le fond tu as raison : il faut créer une branche. Je m'en
charge.
Moi j'ai pas trop pigé en quoi c'était un blocage majeur, genre vraiment
cassage d'API etc.
Si c'est juste "ya 2 personnes qui ont des saisies vraiment chelou et
que ça peut les avoir embêté", je n'appelle pas ça du tout un cassage de
compat.
Donc je vois pas trop l'intérêt d'un changement en 4.0. Et vu que c'est
récent, je serais plutôt partisan de revenir en arrière et changer juste
le Y.
--
RastaPopoulos
Le 15/11/2018 à 16:18, RastaPopoulos a écrit :
Le 15/11/2018 à 16:14, Maïeul Rouquette a écrit :
Honnetemnt je ne vois pas de raison qu'il y ait un blocage, mais j'ai
quand même préféré marquer le coup.Mais sur le fond tu as raison : il faut créer une branche. Je m'en
charge.Moi j'ai pas trop pigé en quoi c'était un blocage majeur, genre vraiment
cassage d'API etc.Si c'est juste "ya 2 personnes qui ont des saisies vraiment chelou et
que ça peut les avoir embêté", je n'appelle pas ça du tout un cassage de
compat.Donc je vois pas trop l'intérêt d'un changement en 4.0. Et vu que c'est
récent, je serais plutôt partisan de revenir en arrière et changer juste
le Y.
ok, faisons cela. Effectivement le changement justifie pas de faire un x, surtout que cela ne s'applique pas aux saisies perso.
Ok, c 'est cool.