Projet

Général

Profil

Anomalie #999

Lors d'un import avec le statut de l'adhérent manquant, le 'galop d'essai' est OK mais l'import échoue

Ajouté par Eddy Minet il y a plus d'un an. Mis à jour il y a 6 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
Core
Version cible:
Début:
21/01/2017
Echéance:
% réalisé:

100%

Temps estimé:
Version utilisée:
Vote:

Description

Le champ 'id_status' de la table adherents devrait être 'required'


Demandes liées

Lié à Galette - Evolution #729: Détection des courriels en doulon lors d'un import en mode dry-runFermé2013-10-20

Révisions associées

Révision 2654bd21 (diff)
Ajouté par Johan Cwiklinski il y a plus d'un an

Check for status in dry-run import; fixes #999

Historique

#1 Mis à jour par Johan Cwiklinski il y a plus d'un an

  • Lié à Evolution #729: Détection des courriels en doulon lors d'un import en mode dry-run ajouté

#2 Mis à jour par Johan Cwiklinski il y a plus d'un an

  • Catégorie changé de Base de données à Core

S'il y a effectivement un souci ; rendre le champs requis ne solutionnera pas vraiment le problème ; et en causera d'autres (raison pour laquelle il n'est pas possible de définir ce champ comme requis depuis la personnalisation des champs).
Si le statut est manquant, un statut par défaut est attribué par Galette ; il n'est donc effectivement pas requis à ce moment là.

Par ailleurs, si le statut est renseigné, il faut qu'il corresponde à l'identifiant d'un statut existant en base, c'est donc un peu plus compliqué que ça à résoudre.

#3 Mis à jour par Eddy Minet il y a plus d'un an

Le problème est qu'actuellement :

- il est possible de mettre ce champ dans l'import
- il est possible de passer le galop d'essai sans mettre de valeur pour ce champ dans l'import
- lorsqu'on envoie l'import Galette n'assigne pas de statut par défaut et l'import plante avec une erreur indéfinie

Avec ma modif au moins le galon d'essai ne passe pas et indique une champ requis.

Si cette modif pose des problème ailleurs dans le code et que de plus l'utiliser dans l'import n'est pas cohérent alors dans ce cas il ne faudrait ne pas l'autoriser dans le fichier d'import ...
Car j'ai mis un moment avant de comprendre pourquoi l'import ne fonctionnait pas alors que le galop d'essai me disait que c'était bon.

#4 Mis à jour par Johan Cwiklinski il y a plus d'un an

- il est possible de mettre ce champ dans l'import

Oui, s'il est renseigné correctement partout, il n'y a pas de problèmes.

- il est possible de passer le galop d'essai sans mettre de valeur pour ce champ dans l'import

Ça, c'est un « bug ».

- lorsqu'on envoie l'import Galette n'assigne pas de statut par défaut et l'import plante avec une erreur indéfinie

Bah oui, normal, puisque l'import envoie une valeur...

Avec ma modif au moins le galon d'essai ne passe pas et indique une champ requis.

Oui, mais non. Ta modif impacte la totalité de Galette, pas juste l'import ; et donc, ça ne marche pas. Ça corrige un bug, et ça en crée d'autres... :)

Donc, oui, le mode dry-run devrait gueuler à ce niveau là ; mais la vraie correction est plus compliquée que ça. Si par exemple tu mets un id qui n'existe pas dans la base, je pense le problème sera tout pareil.

#5 Mis à jour par Johan Cwiklinski il y a plus d'un an

  • Version cible mis à 0.9

#6 Mis à jour par Johan Cwiklinski il y a plus d'un an

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

#7 Mis à jour par Johan Cwiklinski il y a plus d'un an

  • Statut changé de Résolu à Fermé

#8 Mis à jour par Stéphane Muteau il y a 6 mois

J'ai reproduit ce même problème en v0.9 ce jour : si id_statut vide dans csv, le galop d'essai est OK, l'import est KO.

Formats disponibles : Atom PDF