Développement #66187
Import CSV d'évènements, supporter des quotes dans une description déjà quotée
Status:
Fermé
Priority:
Normal
Assignee:
A. B.
Category:
-
Target version:
-
Start date:
13 June 2022
Due date:
% Done:
0%
Estimated time:
Patch proposed:
Yes
Planning:
No
Description
À priori il suffit de passer dialect.doublequote à True.
Files
Associated revisions
History
Updated by A. B. over 2 years ago
- File 0001-agendas-fix-import-event-crashing-with-nested-quotes.patch 0001-agendas-fix-import-event-crashing-with-nested-quotes.patch added
- Tracker changed from Support to Bug
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by A. B. over 2 years ago
Updated by A. B. over 2 years ago
Updated by A. B. over 2 years ago
Updated by Valentin Deniaud over 2 years ago
Donc le fix fonctionne, chouette !
Quelques remarques :- Il y a une modif au
__init__qui n'a pas l'air de concerner le bug en question - Le test doit plutôt suivre ceux qui ont déjà été écrits pour l'import CSV, ils sont dans tests/manager/test_event.py
- De manière générale on va plutôt préférer tester les choses en situation réelle, c'est à dire arriver sur la page -> upload d'un fichier -> vérifier la description de l'évènement créé en base, plutôt que d'instancier le formulaire à la mano
- Je vois que tu as repris la chaîne qui pose problème depuis le ticket client mais je trouverais ça mieux d'avoir un fichier minimal (je ne sais pas si on peut mettre moins de colonnes mais on pourrait au moins remplacer les chaînes en français pas des « there should be ""quotes"" » ou autre texte moins surprenant)
Updated by A. B. over 2 years ago
Merci pour tes retours, je passe sur un test complet avec le client HTTP django :)
Updated by Valentin Deniaud over 2 years ago
- Status changed from Solution proposée to Solution validée
Super, à pousser quand jenkins est content !
Updated by A. B. over 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit dcb8aed83a18d731dcd6e31cbb6823df596b3251
Author: Agate <aberriot@entrouvert.com>
Date: Tue Jul 5 11:25:43 2022 +0200
agendas: fix import event crashing with nested quotes (#66187)Updated by Transition automatique over 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
agendas: fix import event crashing with nested quotes (#66187)