Projet

Général

Profil

Bug #36832

import csv : certaines erreurs ne sont pas affichées

Ajouté par Valentin Deniaud il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
10 octobre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Bloque Authentic 2 - Development #35773: import csv, ajout d'un rôleFermé03 septembre 2019

Actions

Révisions associées

Révision e472246f (diff)
Ajouté par Valentin Deniaud il y a plus de 4 ans

csv_import: display all row errors (#36832)

Historique

#1

Mis à jour par Valentin Deniaud il y a plus de 4 ans

#2

Mis à jour par Valentin Deniaud il y a plus de 4 ans

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

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.

#4

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.

#5

Mis à jour par Valentin Deniaud il y a plus de 4 ans

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

#6

Mis à jour par Valentin Deniaud il y a plus de 4 ans

#7

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée
#8

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)

#9

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

Formats disponibles : Atom PDF