Galette » Plugins » Évènements

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

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

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}

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.

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

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

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.

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

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

Bonjour,

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

Alain

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