Bug #40771
rendre convert_value_from_str plus laxiste sur un champ date
0%
Description
Actuellement convert_value_from_str crache brutalement sur un champ date si la donnée à convertir n'est pas dans un format de date connu.
Je pense qu'on devrait être plus laxiste et juste renvoyer None dans ce cas.
Exemple de problème que cela pose #40770 , qui est vu comme une regression suite à #40190
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 4 ans
Je veux bien qu'on discute des impacts, pour ma part je ne vois pas de soucis. On a déjà un tel comportement sur MapField. En fait actuellement seul DateField::convert_value_from_str peut lever une exception.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier 0001-fields-ignore-invalid-date-values-40771.patch 0001-fields-ignore-invalid-date-values-40771.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Genre ça ?
Mis à jour par Thomas Noël il y a environ 4 ans
- Fichier 0001-fields-ignore-invalid-date-values-40771.patch 0001-fields-ignore-invalid-date-values-40771.patch ajouté
Ouaip, avec la modif du test qui va avec.
Mis à jour par Frédéric Péters il y a environ 4 ans
Faut aussi mettre à jour test_backoffice_cards_import_data_from_csv, sans doute en retirant la partie qui testait l'erreur possible, vu que tout passera désormais.
Mis à jour par Frédéric Péters il y a environ 4 ans
Mis à jour par Thomas Noël il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
(à pousser dès que les tests sont passés)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 18ace9518125cbae9e38d9eb92f4372eecef77c8 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Mar 17 07:49:18 2020 +0100 fields: ignore invalid date values (#40771)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
fields: ignore invalid date values (#40771)