Bug #8878
erreurs suite à #8862
0%
Description
Dans #8867 j'ai attaché un patch qui n'est pas directement le sujet du ticket, voici donc un ticket dédié, avec d'autres corrections du même style.
Fichiers
Révisions associées
tests: add a bunch of tests on fields (#8878)
Historique
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-form-fix-handling-of-various-optional-fields-8878.patch 0001-form-fix-handling-of-various-optional-fields-8878.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-Revert-forms-store-empty-non-required-strings-as-emp.patch 0001-Revert-forms-store-empty-non-required-strings-as-emp.patch ajouté
- Fichier 0002-Revert-form-correctly-ignore-empty-values-in-WidgetL.patch 0002-Revert-form-correctly-ignore-empty-values-in-WidgetL.patch ajouté
- Fichier 0003-forms-don-t-restore-empty-fields-for-drafts-8878.patch 0003-forms-don-t-restore-empty-fields-for-drafts-8878.patch ajouté
- Fichier 0004-tests-add-a-bunch-of-tests-on-fields-8878.patch 0004-tests-add-a-bunch-of-tests-on-fields-8878.patch ajouté
Machine arrière, je reviens là-dessus et sur #8862 et je proposerais d'annuler ça, il y a encore trop de conséquences inconnues.
0001 et 0002 annulent les commits passés.
0003 modifie la restauration des brouillons pour uniquement garder les clés pour lesquelles une valeur est définie, c'est de là qu'était venue la "nécessité" de gérer différemment None et ''.
0004 ajoutent les tests qui étaient dans 0001 et 0002.
Reste l'effet de bord qui était de donner des chaines vides dans la sortie json, plutôt que null, qu'on aimerait voir revenir; ça sera un autre ticket, spécifique.
Mis à jour par Thomas Noël il y a plus de 8 ans
Frédéric Péters a écrit :
0001 et 0002 annulent les commits passés.
0003 modifie la restauration des brouillons pour uniquement garder les clés pour lesquelles une valeur est définie
0004 ajoutent les tests qui étaient dans 0001 et 0002.
Ack.
Reste l'effet de bord qui était de donner des chaines vides dans la sortie json,
plutôt que null, qu'on aimerait voir revenir; ça sera un autre ticket, spécifique.
Bof, et même... c'est pas si important. Ca ne m'a jamais vraiment perturbé.
Mis à jour par Thomas Noël il y a plus de 8 ans
quoique sur le 0003, j'ajouterai bien un "# restore draft" et "# don't consider empty values" juste pour rappeler ce qu'on fait à cet endroit du code...?
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de En cours à Résolu (à déployer)
Voià, avec commentaire ajouté.
commit 480bf287e4ca45737b55198eb01b14b278be7636 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Nov 5 13:11:30 2015 +0100 tests: add a bunch of tests on fields (#8878) commit 66a4068db6ebc71d09a546b0e5a181a5737501b6 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Nov 5 13:04:09 2015 +0100 forms: don't restore empty fields for drafts (#8878) commit f99f0fc313ead424a61eb507e25927f56fd55a5b Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Nov 5 12:52:45 2015 +0100 Revert "form: correctly ignore empty values in WidgetList (#8866)" This reverts commit 776c367f31f618cb7f68fa3ba56379e1f52a1520. commit 7f6d4a354dab1a61c04f1040e1c89527d9dadebf Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Nov 5 12:52:44 2015 +0100 Revert "forms: store empty non required strings as empty strings (#8862)" This reverts commit fe7f2a1ee28d7dcdfe5700de2cdc584d227a8bd6.
forms: don't restore empty fields for drafts (#8878)