Development #66187
Import CSV d'évènements, supporter des quotes dans une description déjà quotée
Début:
13 juin 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
À priori il suffit de passer dialect.doublequote à True.
Fichiers
Révisions associées
Historique
Mis à jour par A. Berriot il y a presque 2 ans
- Fichier 0001-agendas-fix-import-event-crashing-with-nested-quotes.patch 0001-agendas-fix-import-event-crashing-with-nested-quotes.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par Valentin Deniaud il y a presque 2 ans
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)
Mis à jour par A. Berriot il y a presque 2 ans
Merci pour tes retours, je passe sur un test complet avec le client HTTP django :)
Mis à jour par A. Berriot il y a presque 2 ans
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Super, à pousser quand jenkins est content !
Mis à jour par A. Berriot il y a presque 2 ans
- Statut changé de Solution validée à 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)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
agendas: fix import event crashing with nested quotes (#66187)