Project

General

Profile

Actions

Anomalie #1200

closed

Problème d'enregistrement des activités.

Added by Christian Hardy over 5 years ago. Updated over 5 years ago.

Status:
Fermé
Priority:
Normal
Target version:
Start date:
09/17/2018
Due date:
% Done:

100%

Estimated time:
Version utilisée:

Description

En préparation de notre AG 2019 notre trésorier a créé un événement correspondant, puis 4 activités.

Pas de problème pour associer événement et activités correspondantes.

Création d'une réservation

Validation des 4 activités, on indique payé, on met le montant, le type de paiement , le nom de la banque, le numéro de chèque, on enregistre.
Vérification de la réservation : Seul la dernière activité est marquée comme retenue, le reste est enregistré correctement.
Validation des 3 activités, enregistrement.
Vérification de la réservation : Les modifications ne sont pas enregistrées.
Même problème pour les rubriques "Payé" et " type de paiement", après un premier enregistrement impossible de modifier la valeur.
Pour les rubriques : "montant" "nom de la banque", "numéro de chèque" pas de problème, les changements sont bien enregistrés.

Cordialement
C Hardy

Actions #1

Updated by Johan Cwiklinski over 5 years ago

  • Project changed from Galette to Évènements
  • Subject changed from Plugin évènements : Problème d'enregistrement des activités. to Problème d'enregistrement des activités.
  • Status changed from Nouveau to In Progress
  • Assignee set to Johan Cwiklinski
  • Version utilisée changed from 0.9.1.2 to 1.0.1
Je confirme de mon côté :
  • la modification des activités retenues d'une réservation est hasardeuse,
  • il n'est pas possible de décocher "payé" (le cocher fonctionne).

J'ai pu modifier les autres champs sans problèmes. Il est fort possible que les problèmes que j'ai reproduits produisent des effets de bord, il faudra voir une fois corrigés :)

Actions #2

Updated by Johan Cwiklinski over 5 years ago

Celui là était un poil plus compliqué ; j'ai un correctif qui demande à être testé. Le plus simple, c'est de récupérer le nouveau fichier /plugin-events/lib/GaletteEvents/Booking.php dans son intégralité et de vérifier si les soucis d'enregistrement d'activités sont bien résolus.

D'après ce que j'ai pu constater, cela fonctionnait si l'on modifiait une seule activité, mais pas s'il y en avait plusieurs.

Pour le problème de la case à cocher, c'est plus simple, voici le correctif :

diff --git a/lib/GaletteEvents/Booking.php b/lib/GaletteEvents/Booking.php
index d6c5479..018aa74 100644
--- a/lib/GaletteEvents/Booking.php
+++ b/lib/GaletteEvents/Booking.php
@@ -247,7 +247,10 @@ public function check($values)

         if (isset($values['paid'])) {
             $this->paid = true;
+        } else {
+            $this->paid = false;
         }
+
         if (isset($values['amount']) && !empty($values['amount'])) {
             $this->amount = $values['amount'];
         }

Actions #3

Updated by Christian Hardy over 5 years ago

Bonjour

J'ai utilisé le nouveau Bookinglist.php. Chez moi l'enregistrement de modifications multiples se passe bien .
J'ai créé 5 activités et les ai modifiées chacune leur tour, puis plusieurs à la fois : Toutes les modifications sont enregistrées.

En revanche, le correctif proposé pour la case à cocher ne fonctionne pas. Impossible aussi de changer le montant.
Ne manquerait-il pas un { quelque part ?

Christian

Actions #4

Updated by Johan Cwiklinski over 5 years ago

Christian Hardy a écrit :

En revanche, le correctif proposé pour la case à cocher ne fonctionne pas. Impossible aussi de changer le montant.
Ne manquerait-il pas un { quelque part ?

S'il manquait un {, on aurait une page blanche ; donc, non à moins d'une faute au recopiage. Le plus simple, c'est sûrement de récupérer le fichier Booking.php avec toutes les modifications :
https://raw.githubusercontent.com/galette/plugin-events/hotfix/1.0.2/lib/GaletteEvents/Booking.php

J'ai testé de mon côté, la case "payé" fonctionne comme escompté, et le montant s'enregistre bien.
S'il subsiste un problème, c'est soit que le montant entré est incorrect, soit que c'est dû aux autres données modifiées en même temps ; dans tous les cas il me faudrait les informations pour reproduire de manière certaine.

Après, il se peut que le fichier avec l'ensemble des correctifs résolve le soucis aussi :)

Actions #5

Updated by Christian Hardy over 5 years ago

Bonjour

J'ai récupéré le fichier Booking.php.
Plus de problème pour enregistrer le paiement ou le supprimer. La case à cocher marche bien.

Petit problème résiduel :
1- Création d'une réservation avec paiement et enregistrement d'un montant
2- Suppression du paiement, Impossible de supprimer le montant payé.

C'est bien résiduel, car il suffit de recocher "PAYÉ" et de mettre le montant à zéro, d'enregistrer la modification et de revenir pour décocher la case PAYE.
Christian

Actions #6

Updated by Johan Cwiklinski over 5 years ago

  • Target version set to 1.0.2
Actions #7

Updated by Johan Cwiklinski over 5 years ago

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

Updated by Johan Cwiklinski over 5 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF