Développement #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.
Files
Associated revisions
History
Updated by Valentin Deniaud about 2 years ago
- File 0001-manager-report-errors-in-CSV-import-using-event-inde.patch 0001-manager-report-errors-in-CSV-import-using-event-inde.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Frédéric Péters about 2 years ago
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.
Updated by Valentin Deniaud about 2 years ago
- File 0001-manager-report-errors-in-CSV-import-using-event-inde.patch 0001-manager-report-errors-in-CSV-import-using-event-inde.patch added
Très bien vu, corrigé avec un test en plus.
Updated by Benjamin Dauvergne about 2 years ago
- Status changed from Solution proposée to 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.
Updated by Valentin Deniaud about 2 years ago
- Status changed from Solution validée to 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)
Updated by Transition automatique about 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
manager: report errors in CSV import using event indexes (#66184)