Project

General

Profile

Actions

Anomalie #1238

closed

Enregistrement d'événements sans effet

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

Status:
Fermé
Priority:
Normal
Target version:
Start date:
01/18/2019
Due date:
% Done:

100%

Estimated time:
Version utilisée:

Description

Galette 0.9.2 Plugins Événements 1.1.0
Menu Nouvel événement
Saisie des données obligatoires
Enregistrement
Pas de message d'erreur mais aussi pas de message "l'événement à été enregistré"
Retour à la liste de événements : Pas d'événement enregistré

C Hardy


Files


Related issues 2 (0 open2 closed)

Has duplicate Évènements - Anomalie #1248: Création d'une réservation impossible Rejeté01/30/2019

Actions
Precedes Évènements - Evolution #1245: Suprpession de la nouvelle sélection des adhérentsFerméJohan Cwiklinski01/21/201901/21/2019

Actions
Actions #1

Updated by Johan Cwiklinski over 5 years ago

  • Project changed from Plugins to Évènements
  • Version utilisée set to 1.1.0
Actions #2

Updated by Johan Cwiklinski over 5 years ago

  • Subject changed from Plugin Événements to Enregistrement d'événements sans effet
Actions #3

Updated by Johan Cwiklinski over 5 years ago

  • Status changed from Nouveau to In Progress
  • Assignee set to Johan Cwiklinski
Actions #4

Updated by Johan Cwiklinski over 5 years ago

  • Target version set to 1.1.1
Actions #5

Updated by Johan Cwiklinski over 5 years ago

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

Updated by Johan Cwiklinski over 5 years ago

  • Status changed from Résolu to Fermé
  • % Done changed from 100 to 0

C'est corrigé, ainsi qu'un autre problème qui pouvait survenir à à l'ajout de réservations. La version 1.1.1 est sortie, ces problèmes sont majeurs.

Actions #7

Updated by Christian Hardy over 5 years ago

Merci de ta réactivité et pour la correction rapide.
Un problème subsiste.

Galette 0.9.2 Événements 1.1.1 PHP 7.2
Nouvel Événement OK Évenement enregistré, PARFAIT
Nouvelle activité OK 4 activités enregistrées, PARFAIT
Rattachement de 4 activités à l'événement OK Tout est pris en compte parfait.
Nouvelle réservation : plantage de l'application . Ci joint rapport de plantage.

Cordialement
Christian

Actions #8

Updated by Johan Cwiklinski over 5 years ago

  • Status changed from Fermé to Nouveau
  • % Done changed from 0 to 50

Arf zut ! Je regarderai ça dçs que je pourrai :/

Actions #9

Updated by Johan Cwiklinski over 5 years ago

Pourrais-tu essayer avec :

diff --git a/_routes.php b/_routes.php
index f4a6b7a..ea51c3c 100644
--- a/_routes.php
+++ b/_routes.php
@@ -613,7 +613,7 @@ $this->get(
         if (isset($booking) && $booking->getMemberId() > 0) {
             if (!isset($members[$booking->getMemberId()])) {
                 $members = array_merge(
-                    [$trans->member => Adherent::getSName($this->zdb, $trans->member, true)],
+                    [$booking->getMemberId() => Adherent::getSName($this->zdb, $booking->getMemberId(), true)],
                     $members
                 );
             }

Actions #10

Updated by Christian Hardy over 5 years ago

J'ai modifié le fichier _route.php de la racine de Galette events plugin.
Toujours erreur de l'application qui dit "variable trans non définie"
Copie ci-jointe

Actions #11

Updated by Johan Cwiklinski over 5 years ago

Il y a deux occurrences de $trans à remplacer dans la ligne en question (et c'étaient les deux seules du fichier).

Actions #12

Updated by Christian Hardy over 5 years ago

Merci Johan.
C'est une erreur de ma part. Je suis néophyte à l'utilisation du tracker et j'ai mal fait l'intégration du code.
Plus de plantage à l'activation du menu Nouvelle réservation . La fiche est bien affichée.
En revanche des problèmes à l'enregistrement :
En passant par le Menu "Nouvelle réservation"
1er cas :
Sélection de l'événement
Sélection de l'adhérent (1er de la liste présentée)
Pas d'enregistrement de la réservation
Saisie des informations complémentaires
Résultat enregistrement au nom de l'adhérent N°1 ( complètement différent)
2ème cas
Sélection de l'adhérent (autre que le 1er de la liste présentée)
Sélection de l'événement (l'ordre des sélections est influent)
Résultat => enregistrement de la réservation
Saisie des information complémentaires
Résultat =>Enregistrement correct au bon nom avec les bonnes valeurs
3ème cas
Sélection de l'événement
Adhérent = adhérent connecté (présentation par défaut)
Pas d'enregistrement de la réservation.
Retour sur " N'oubliez pas d'enregistrer la réservation".
Saisie d'informations complémentaires
Enregistrement : pas de réservation enregistrée
4ème cas
Sélection de l'événement
Sélection du second dans la liste présentée
Plantage (voir fichier joint)

En passant par la liste des adhérents
Sélection d'un adhérent
puis clic sur le bouton "Nouvelle réservation"
Affichage de la fiche "Réservation" OK mais le nom affiché est celui du premier de la liste
Sélection du l'adhérent voulu
Sélection de l'événement
Enregistrement OK mais avec en retour le nom du premier de la liste alphabétique.
Du coup si on ne fait pas attention, on enregistre les données complémentaires sous le nom du premier de la liste.

En revanche pas de plantage si on choisit un adhérent autre que le premier.
Christian

Actions #13

Updated by Johan Cwiklinski over 5 years ago

Je n'ai pas eu le temps de tout tester, mais voici déjà un correctif :

diff --git a/templates/default/booking.tpl b/templates/default/booking.tpl
index 41f9142..35b33f5 100644
--- a/templates/default/booking.tpl
+++ b/templates/default/booking.tpl
@@ -32,7 +32,7 @@
                         <option value="0">{_T string="-- select a name --"}</option>
         {/if}
         {foreach $members.list as $k=>$v}
-                            <option value="{$k}"{if $transaction->member == $k} selected="selected"{/if}>{$v}</option>
+                            <option value="{$k}"{if $booking->getMemberId() == $k} selected="selected"{/if}>{$v}</option>
         {/foreach}
                     </select>
     {else}
Actions #14

Updated by Christian Hardy over 5 years ago

Je reprends les différents cas ci dessus avec le résukltat

En passant par le Menu "Nouvelle réservation"

1er cas :
Sélection de l'événement
Sélection de l'adhérent (1er de la liste présentée)
Pas d'enregistrement de la réservation
Saisie des informations complémentaires
Résultat enregistrement au nom de l'adhérent N°1 ( complètement différent)
MEME RESULTAT
2ème cas
Sélection de l'adhérent (autre que le 1er de la liste présentée)
Sélection de l'événement (l'ordre des sélections est influent)
Résultat => enregistrement de la réservation
Saisie des information complémentaires
Résultat =>Enregistrement correct au bon nom avec les bonnes valeurs
RESULTAT ENREGISTREMENT AVEC L'ADHERENT PORTANT LE MËME NUMERO. Si je choisis le 6ème adhérent de la liste, la réservation est enregistrée sous le nom de l'adhérent N°6
3ème cas
Sélection de l'événement
Adhérent = adhérent connecté (présentation par défaut)
Pas d'enregistrement de la réservation.
Retour sur " N'oubliez pas d'enregistrer la réservation".
Saisie d'informations complémentaires
Enregistrement : pas de réservation enregistrée
PAS CHANGEMENT ; Pas d'enregistrement de la réservation
4ème cas
Sélection de l'événement
Sélection du second, troisième, quatrième cinquième, septième dans la liste présentée mais pas 6 et 8
Plantage (voir fichier joint)
A mon avis, c'est du au fait que les adhérents 2,3,4,5,7 ont été supprimé de la base. Donc l'appli ne peut inséré un nouvel enregistrement

En passant par la liste des adhérents
1er cas
Dans le liste des adhérents, sélection d'un adhérent
puis sur la fiche adhérent, clic sur le bouton "Nouvelle réservation"
Affichage de la fiche "Réservation" OK avec le bon nom d'adhérent
Sélection de l'événement
Enregistrement OK Pour moi c'est tout bon et y compris les modifications si on revient sur la réservation

2éme cas
Apparu fortuitement car j'avais trié la liste des adhérents par ordre de numéro d'adhérent
Dans le liste des adhérents triée par numéro d'adhérent, sélection d'un adhérent (n°13 par exemple)
puis sur la fiche adhérent, clic sur le bouton "Nouvelle réservation"
Affichage de la fiche "Réservation" avec le nom de l'adhérent du rang 13 de la liste alphabétique des adhérents
Sélection de l'événement
Enregistrement est fait au nom d'adhérent sélectionné en premier (n°13 donc) et ce que l'on change ou le nom de l'adhérent dans la liste de selection affichée sur la fiche réservation
Sans ce problème d'affichage de nom TRÈS PERTURBANT, LA RESERVATION EST BONNE
Si on revient sur la réservation, le nom affiché est celui du 13 ème adhérent dans la iste alphabétique des adhérents et non celui du n°13. L'enregistrement se fait au nom du N° 13 mais c'est très perturbant pour un utilisateur lambda.

Actions #15

Updated by Johan Cwiklinski over 5 years ago

Bon, c'est un nid à problèmes cette histoire ; je vais virer la sélection de l'adhérent via la liste, et remettre l'ancienne popup. C'est moins pratique, mais ça ne pose pas de problèmes.

Actions #16

Updated by Christian Hardy over 5 years ago

Dommage car cela marche très bien pour les cotisations. C'est bien le même principe.
Christian

Actions #17

Updated by Johan Cwiklinski over 5 years ago

Oui, c'est le même principe, mais ça n'a pas posé autant de problèmes sur les cotisations. Je n'ai pas vraiment beaucoup de temps libre, et je préfère privilégier la stabilité ; pour le coup, ce sera bien plus simple pour moi.

Ça ne me plaît qu'à moitié dans Galette finalement, et sinon j'attendrai d'avoir du temps avant de me pencher de nouveau sur cette fonctionnalité.

Actions #18

Updated by Johan Cwiklinski over 5 years ago

  • Precedes Evolution #1245: Suprpession de la nouvelle sélection des adhérents added
Actions #19

Updated by Johan Cwiklinski over 5 years ago

  • Status changed from Nouveau to Fermé
  • % Done changed from 50 to 100
Actions #20

Updated by Johan Cwiklinski over 5 years ago

J'ai trouvé et corrigé au moins un autre problème ; mais je manque de temps pour tester.

Il faudrait récupérer https://github.com/galette/plugin-events/archive/hotfix/1.1.2.zip et voir si c'est bon avec ou toujours pas.

Actions #21

Updated by Christian Hardy over 5 years ago

Bonsoir Johan
Je viens de tester rapidement entre deux réunions
Beaucoup d'améliorations et à première vue c'est beaucoup mieux
Trois améliorations à faire relevées
1- Par le menu Nouvelle réservation, le nom affiché en premier est celui de l'utilisateur connecté, est-il possible de laisser à blanc, car quand on choist l'évènement un enregistrement est automatiquement créé en ce nom ?

2 Dans la page Nouvelle réservation, il est possible de sélectionner un utilisateur en rapidement (vraiment rapidement !!!!) une suite de caractères représentatifs. Il faut être très rapide sinon c'est la première lettre saisie qui sert de sélecteur et donc pas de résultat satisfaisant. Peux-t-on ralentir le process de sélection.

3- Sélection de l’événement, sélection de l'adhérent, enregistrement fait OK. Quel que soit le menu d'appel, on complété les information, si on coche la case Payé sans mettre le montant et le type de paiement, rien n'est enregistré.
Visiblement,il y a un problème à ce niveau avec des subtilités que je n'ai pas eu le temps de tester car je repars en réunion.

Merci ne tout cas car nous pouvons avec quelques astuces et manip nous en sortir
Christian

Actions #22

Updated by Christian Hardy over 5 years ago

Bonjour Johan
J'ai pris un peu plus de temps pour tester ce matin.
Je confirme que, tant par la liste des adhérents que par le menu Nouvelle réservation l'enregistrement des réservations est correct.
Les 6 anomalies signalées plus haut sont corrigées.

En revanche je confirme les 3 demandes d'hier et en particulier, pour rendre utilisable le menu "Nouvelle réservation" par un utilisateur "agé" dont la dextérité est affaiblie, il serait bien de ralentir le process de sélection des noms par une suite de caractères.

Pour le bug 3, je confirme que si l'on coche la case "Payé" sans mettre de montant, les informations complémentaires saisies ne sont pas enregistrées.

Christian

Actions #23

Updated by Alain Paris over 5 years ago

Bonjour,

il semble que l'enregistrement d'une réservation par l'adhérent lui même ne fonctionne pas non plus.

Alain

Actions #24

Updated by Johan Cwiklinski about 5 years ago

  • Has duplicate Anomalie #1248: Création d'une réservation impossible added
Actions #25

Updated by Johan Cwiklinski over 4 years ago

Christian Hardy a écrit :

1- Par le menu Nouvelle réservation, le nom affiché en premier est celui de l'utilisateur connecté, est-il possible de laisser à blanc, car quand on choist l'évènement un enregistrement est automatiquement créé en ce nom ?

Réglé hier ;)

2 Dans la page Nouvelle réservation, il est possible de sélectionner un utilisateur en rapidement (vraiment rapidement !!!!) une suite de caractères représentatifs. Il faut être très rapide sinon c'est la première lettre saisie qui sert de sélecteur et donc pas de résultat satisfaisant. Peux-t-on ralentir le process de sélection.

Heu... Alors là, j'ai pas tout compris :D Et je n'ai pas forcément la main sur tout avec les bibliothèques JS. L'intégration de Selectize permet des choses intéressantes, mais il y a des soucis en plus :/

3- Sélection de l’événement, sélection de l'adhérent, enregistrement fait OK. Quel que soit le menu d'appel, on complété les information, si on coche la case Payé sans mettre le montant et le type de paiement, rien n'est enregistré.

J'ai testé hier avec les dernières versions des branches develop de Galette et du plugin ; je n'ai pas reproduit. Soit le cas est plus spécifique, soit j'ai loupé un truc, soit c'est corrigé ;-)

Actions

Also available in: Atom PDF