Project

General

Profile

Anomalie #1561

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

Added by Alfa-Club Des Volcans 2 months ago. Updated 26 days ago.

Status:
Résolu
Priority:
Normal
Assignee:
-
Start date:
08/16/2021
Due date:
% Done:

100%

Estimated time:

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

#1

Updated by Johan Cwiklinski about 1 month ago

  • Status changed from Nouveau to 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).

#2

Updated by Alfa-Club Des Volcans about 1 month ago

ok, je regarde ce we

#3

Updated by Alfa-Club Des Volcans 29 days ago

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.

#4

Updated by Johan Cwiklinski 28 days ago

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

#5

Updated by Johan Cwiklinski 28 days ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
#6

Updated by Alfa-Club Des Volcans 26 days ago

Oui, parfait
merci

Also available in: Atom PDF