JeromeDevome / GRR

GRR Officiel - Copyright Team DEVOME
https://grr.devome.com
GNU General Public License v2.0
81 stars 50 forks source link

Réservation en blocs #344

Closed ccheveaux closed 3 months ago

ccheveaux commented 4 months ago

Bonjour,

Nous rencontrons un bug sur la réservation en blocs depuis mon passage de la v3.4.3 à la v4.3.2 que je ne rencontrais pas avant.

A l'étape 2/4 de la réservation en bloc il est demandé le "Type" qui pour nous n'a jamais était des lettres mais plutôt des "Noms".

image

image

Le problème est présent sur une installation propre ainsi que sur le site de démonstration.

image

Capture d'écran 2024-03-05 160842

Sur le planning aucune réservation n'apparait. Aussi bien sur le Grr de prod que sur le Grr de test et sur le site de démonstration (il y a d'ailleurs une erreur SQL sur ce dernier). En regardant sur PHPMyAdmin dans la table "grr_entry" on remarque que la colonne "type" est non pas avec une lettre, mais avec le début du nom qui est semble t'il tronqué.

image

image

Si on le change manuellement (dans la BDD) par la lettre alors la réservation apparait bien dans le planning.

Si vous pouviez m'indiquer le fichier a modifier sur le serveur puis faire les corrections en conséquence pour une futur mise à jour svp ?

Merci pour votre aide :)

Infos GRR Numéro de version GRR fichier : : 4.3.2 - Release-v4.3.2 Numéro de version GRR BDD : : 0400004 Système d'exploitation : : Linux 5.4.0-172-generic #190-Ubuntu SMP Fri Feb 2 23:24:22 UTC 2024 x86_64 Version PHP : 7.4.3-4ubuntu2.20 Base de données : : mysql => 8.0.36-0ubuntu0.20.04.1 Time : 1709652103 Timezone : Europe/Paris

ynaessens commented 4 months ago

Bonjour, effectivement, une tentative de réservation en blocs sur GRR 4.3.2 échoue. Pour ce qui est des types de réservation, il y a une erreur dans le calcul du sélecteur : c'est bien le nom qui devrait apparaître et pas la lettre symbolique. Pour mémoire, les types sont codés par une ou deux lettres et c'est ce code qui apparaît dans la table grr_entry. Si vous voulez contribuer à corriger ce bug, je pense qu'il faut regarder du côté de admin_reservation_bloc .php pour le contrôleur ou .twig pour le template. Cordialement, YN

JeromeDevome commented 4 months ago

Bonjour, J'ai corrigé 3 erreurs dans le code via le commit 54264dbd

Votre analyse détaillé à permis d'identifier rapidement le problème.

Vous récupérer directement les deux fichiers (uniquement) du commit pour les mettre dans votre installation, il n'y a eu aucun changement entre la version 4.3.2 et ce commit.

ccheveaux commented 4 months ago

Merci beaucoup pour votre rapidité ! 👍 C'est parfaitement fonctionnel ! Bonne soirée

image

ynaessens commented 4 months ago

Bonjour, à mon avis il reste un problème avec les mails automatiques, j'ai une erreur déclenchée par la directive require_once 'include/pages.class.php'; depuis include\functions.inc.php on line 2939 Je pense que require_once 'pages.class.php' conviendrait mieux. Cordialement, YN