Bug #36832
import csv : certaines erreurs ne sont pas affichées
0%
Description
Par exemple si on a deux utilisateurs prénommés Bob, qu'on met comme clé first_name, la ligne apparaît verte dans l'interface alors qu'elle n'est pas traitée car plusieurs utilisateurs correspondent.
J'ai l'impression qu'il vaut mieux laisser row.is_valid
être le reflet de la validation du formulaire, et simplement ajouter un check supplémentaire de row.errors
.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-csv_import-display-all-row-errors-36832.patch 0001-csv_import-display-all-row-errors-36832.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-csv_import-display-all-row-errors-36832.patch 0001-csv_import-display-all-row-errors-36832.patch ajouté
J'ai l'impression qu'il vaut mieux laisser row.is_valid être le reflet de la validation du formulaire
Maintenant j'ai plutôt l'impression que non (ça sera aussi plus pratique comme ça pour un autre patch error related dans #35773).
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
C'est peut-être parce qu'en cas de problème d'unicité ça n'est découvert qu'à partir de la deuxième ligne qu'on trouve ? Effectivement ce serait mieux que row.is_valid contienne toute l'information sur la validité ou non d'une ligne.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Statut changé de Solution proposée à En cours
Oui, plus généralement c'est parce que l'erreur n'est découverte que pendant le traitement dans do_import_row
(un tel cas de découverte d'erreur tardive apparaît dans #35773). Par contre mon deuxième patch est nul et met en lumière que le test que j'avais ajouté l'est tout autant, je vais reprendre ça.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-csv_import-display-all-row-errors-36832.patch 0001-csv_import-display-all-row-errors-36832.patch ajouté
- Statut changé de En cours à Solution proposée
Voilà ça devrait être bon, dur dur les patches d'une ligne.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Bloque Development #35773: import csv, ajout d'un rôle ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 92742a2feee6b11db87b2c3525d6b61197f71568 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Thu Oct 10 11:26:29 2019 +0200 csv_import: display all row errors (#36832)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
csv_import: display all row errors (#36832)