Project

General

Profile

Bug #36832

import csv : certaines erreurs ne sont pas affichées

Added by Valentin Deniaud 12 days ago. Updated 6 days ago.

Status:
Résolu (à déployer)
Priority:
Normal
Category:
-
Target version:
-
Start date:
10 Oct 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-csv_import-display-all-row-errors-36832.patch View (6.01 KB) Valentin Deniaud, 10 Oct 2019 11:30 AM

0001-csv_import-display-all-row-errors-36832.patch View (5.42 KB) Valentin Deniaud, 14 Oct 2019 12:02 PM

0001-csv_import-display-all-row-errors-36832.patch View (5.44 KB) Valentin Deniaud, 14 Oct 2019 01:58 PM


Related issues

Blocks Authentic 2 - Development #35773: import csv, ajout d'un rôle En cours 03 Sep 2019

Associated revisions

Revision e472246f (diff)
Added by Valentin Deniaud 6 days ago

csv_import: display all row errors (#36832)

History

#1 Updated by Valentin Deniaud 12 days ago

#2 Updated by Valentin Deniaud 8 days ago

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).

#3 Updated by Benjamin Dauvergne 8 days ago

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.

#4 Updated by Valentin Deniaud 8 days ago

  • Status changed from Solution proposée to 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.

#5 Updated by Valentin Deniaud 8 days ago

Voilà ça devrait être bon, dur dur les patches d'une ligne.

#6 Updated by Valentin Deniaud 6 days ago

#7 Updated by Benjamin Dauvergne 6 days ago

  • Status changed from Solution proposée to Solution validée

#8 Updated by Valentin Deniaud 6 days ago

  • Status changed from Solution validée to 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)

Also available in: Atom PDF