Projet

Général

Profil

Actions

Anomalie #1238

fermé

Enregistrement d'événements sans effet

Ajouté par Christian Hardy il y a environ 5 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
Début:
18/01/2019
Echéance:
% réalisé:

100%

Temps estimé:
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


Fichiers


Demandes liées 2 (0 ouverte2 fermées)

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

Actions
Précède Évènements - Evolution #1245: Suprpession de la nouvelle sélection des adhérentsFerméJohan Cwiklinski21/01/201921/01/2019

Actions

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Projet changé de Plugins à Évènements
  • Version utilisée mis à 1.1.0

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Sujet changé de Plugin Événements à Enregistrement d'événements sans effet

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Statut changé de Nouveau à In Progress
  • Assigné à mis à Johan Cwiklinski

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Version cible mis à 1.1.1

Mis à jour par Johan Cwiklinski il y a environ 5 ans

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

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Statut changé de Résolu à Fermé
  • % réalisé changé de 100 à 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.

Mis à jour par Christian Hardy il y a environ 5 ans

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

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Statut changé de Fermé à Nouveau
  • % réalisé changé de 0 à 50

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

Mis à jour par Johan Cwiklinski il y a environ 5 ans

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
                 );
             }

Mis à jour par Christian Hardy il y a environ 5 ans

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

Mis à jour par Johan Cwiklinski il y a environ 5 ans

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

Mis à jour par Christian Hardy il y a environ 5 ans

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

Mis à jour par Johan Cwiklinski il y a environ 5 ans

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}

Mis à jour par Christian Hardy il y a environ 5 ans

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.

Mis à jour par Johan Cwiklinski il y a environ 5 ans

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.

Mis à jour par Christian Hardy il y a environ 5 ans

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

Mis à jour par Johan Cwiklinski il y a environ 5 ans

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é.

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Précède Evolution #1245: Suprpession de la nouvelle sélection des adhérents ajouté

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Statut changé de Nouveau à Fermé
  • % réalisé changé de 50 à 100

Mis à jour par Johan Cwiklinski il y a environ 5 ans

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.

Mis à jour par Christian Hardy il y a environ 5 ans

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

Mis à jour par Christian Hardy il y a environ 5 ans

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

Mis à jour par Alain Paris il y a environ 5 ans

Bonjour,

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

Alain

Mis à jour par Johan Cwiklinski il y a environ 5 ans

  • Dupliqué par Anomalie #1248: Création d'une réservation impossible ajouté

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

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

Formats disponibles : Atom PDF