Development #66184
Import CSV d'évènements, revoir l'indication de la ligne qui pose problème
0%
Description
Si un évènement du CSV est invalide, on indique qu'il y a une erreur à la ligne n, mais ça ne correspond pas aux numéros de ligne du fichier dès lors qu'un champ (par exemple la description) est découpé sur plusieurs lignes.
Ça paraît galère voire impossible d'indiquer la ligne réelle, donc il y a sûrement juste à changer le message pour parler de numéros d'évènements et non pas de numéros de lignes.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-manager-report-errors-in-CSV-import-using-event-inde.patch 0001-manager-report-errors-in-CSV-import-using-event-inde.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus d'un an
raise ValidationError(_('Invalid file format. (%s event)') % ordinal(i + 1))
En français ordinal(1) → '1<sup>er</sup>' et si je suis bien ça se trouve passé affiché sans |safe, donc ça n'est pas ok.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-manager-report-errors-in-CSV-import-using-event-inde.patch 0001-manager-report-errors-in-CSV-import-using-event-inde.patch ajouté
Très bien vu, corrigé avec un test en plus.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Je valide mais est-ce qu'on en aurait pas profité pour vérifier toutes les lignes d'un coup plutôt que de s'arrêter à la première qui foire ? Au lieu de faire des raise ça revient à accumuler dans une liste errors et à faire if errors: raise ValidationError(errors) en fin de boucle.
Le ticket CD06 pointe immédiatement au moins deux lignes qui posent souci.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 5d620ba5961cce0ed61c1e056e637950a17bbca4
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Sep 21 16:10:04 2022 +0200
manager: report errors in CSV import using event indexes (#66184)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
manager: report errors in CSV import using event indexes (#66184)