Projet

Général

Profil

Actions

Anomalie #1561

fermé

Création véhicule échoue si ajout d'une image à la création

Ajouté par Alfa-Club Des Volcans il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Début:
16/08/2021
Echéance:
% réalisé:

100%

Temps estimé:

Description

Lorsque l'on ajoute la photo au moment de la création d'un véhicule, cette création échoue, car l'enregistrement de la photo est tentée avant la création d'un véhicule -> la propriété "Id" de la picture ne peut pas être renseignée, et la requête SQL est invalide.

Bug reproductible sur demo.galette.eu

Mis à jour par Johan Cwiklinski il y a plus de 2 ans

  • Statut changé de Nouveau à In Progress

Proposition de correctif : https://github.com/galette/plugin-auto/compare/fix/1561

Je n'ai pas trop le temps de tester par moi-même ; mais ça devrait corriger. Le hic, c'est que les vérifications sur l'image ne seront pas correctement remontées en création (mais ça ne devrait pas bloquer la création du véhicule lui même).

Mis à jour par Alfa-Club Des Volcans il y a plus de 2 ans

ok, je regarde ce we

Mis à jour par Alfa-Club Des Volcans il y a plus de 2 ans

Je viens de faire le test, ça ne plante plus, mais l'image n'est pas enregistrée.
C'est vrai que lorsque j'étais tombé sur ce bug, j'avais regardé comment traiter, mais je m'étais heurté à la structure du code : je n'avais pas trouvé comment repousser l'enregistrement de l'image à après l'enregistrement du véhicule.

Pour l'instant, j'ai contourné dans le template, en conditionnant l'affichge du bloc image sur $car->id, mais ce n'est pas très satisfaisant pour l'utilisateur : il faut créer le véhicule, puis retourner en modification pour ajouter l'image.

Mis à jour par Johan Cwiklinski il y a plus de 2 ans

J'ai eu le loisir de tester un peu ce matin, et j'ai trouvé le problème. Le correctif arrive sous peu.

Mis à jour par Johan Cwiklinski il y a plus de 2 ans

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

Mis à jour par Alfa-Club Des Volcans il y a plus de 2 ans

Oui, parfait
merci

Mis à jour par Johan Cwiklinski il y a plus de 2 ans

  • Statut changé de Résolu à Fermé
  • Assigné à mis à Johan Cwiklinski
Actions

Formats disponibles : Atom PDF