JeromeDevome / GRR

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

4.2.0.RC1: error_end_date : ajout de réservation #299

Closed samfuzz closed 9 months ago

samfuzz commented 9 months ago

lors de l'ajout d'une réservation j'ai l'erreur suivante : (error_end_date)

log: _Warning: Undefined array key 1 in /var/www/html/grr14/edit_entry_handler.php on line 312 _Warning: Undefined variable $endtime in /var/www/html/grr14/edit_entryhandler.php on line 362 _Warning: Undefined variable $starttime in /var/www/html/grr14/edit_entry_handler.php on line 362__

contexte, il s'agit d'un upgrade depuis la 1.9.7e

avec la 4.1.1 ce problème apparait lors de la première réservation et ne se reproduit pas pour les suivantes avec une installation initiale de la 4.2.0RC1 : pas d'erreur

Infos GRR Numéro de version GRR fichier : : 4.2.0RC1 - Release-v4.2.0-RC.1 Numéro de version GRR BDD : : 0400003 Système d'exploitation : : Linux *****-GRR4 6.1.0-12-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.52-1 (2023-09-07) x86_64 Version PHP : 8.2.7 Base de données : : mysql => 10.11.3-MariaDB-1 Time : 1695893283 Timezone : Europe/Paris Problème suite à une mise à jour : depuis 1.9.7e

PHP Version 8.2.7

samfuzz commented 9 months ago

remarques 1- cette erreur apparait lorsque dans les paramètres de la ressource l'option :"la date/heure de fin de réservation" est choisie pas d'erreur avec l'option "durée de la réservation"

2- erreur présente aussi avec php 7.3 (downgrade de php 8.2 en PHP 7.3 sous debian bookworm avec les dépots php-sury)

JeromeDevome commented 9 months ago

Bonjour, Merci pour votre retour et toute ces précisions. J'arrive à reproduire le problème.

Le problème vient de la variable $GET["end"] ou les deux points ":" sont convertis en hexa. Je regarde pour corriger cela rapidement.

JeromeDevome commented 9 months ago

Cela est corrigé dans le commit 47cebc7 . Seul la modification de edit_entry_handler.php corrige votre problème. Pour les langues , je me suis aperçu d'un manquant non critique.

Si vous pouvez confirmer la solution. Merci.

samfuzz commented 9 months ago

je viens de tester rapidement, je n'ai plus l'erreur,

merci

JeromeDevome commented 9 months ago

Merci pour votre retour