PHP Warning (http.error) sur la page Backup

[TiTidom]

Bonjour,

BugReport

En épluchant les logs pour un plugin pour tout autre chose, j’ai remarqué dans le http.error un message PHP Warning

0000|[09-Jan-2024 10:34:50 Europe/Brussels] PHP Warning:  substr() expects parameter 3 to be int, string given in /var/www/html/desktop/php/backup.php on line 66

Il s’agit de cette ligne (pour corriger le soucis, il faut prendre le paramètre 3 et le mettre en paramètre 1

Avant :

<?php if (substr(0, 1, config::byKey('backup::path')) != '/') { ?>

Après :

<?php if (substr(config::byKey('backup::path'), 0, 1) != '/') { ?>

Page concernée : backup
Version Core : 4.3.21

Etape pour reproduire le Bug :

Se rendre sur la page des backups : Réglages / Système / Sauvegardes
Ensuite aller dans les logs « http.error »

Ce bug n’apparait pas en version 4.4.1, où cela a semble-t-il déjà été corrigé

Bonne journée,
TiTidom.

[Salvialf]

Salut,

L’installation en 4.3.21 sur laquelle l’erreur remonte est sous Debian 12 ? Si c’est le cas alors c’est normal, Jeedom n’est pleinement compatible debian12 qu’à partir de la 4.4.1.

[TiTidom]

Bonjour @Salvialf ,

Non c’est en Debian 11. (à la fois pour le jeedom en 4.3.21 et sur le jeedom 4.4.1)

Bonne journée,
TiTidom.

[Salvialf]

Oui autant pour moi, je pensais que l’erreur arrivait sous PHP8 mais après avoir testé elle est effectivement aussi présente sous PHP7.4.

Je regarde pour corriger en stable.

[TiTidom]

Super, merci

[Salvialf]

Par contre je suis un peu embêté par le fait de ne pas reproduire l’erreur sur une machine en 4.3.21

Du coup je creuse un peu +…

[TiTidom]

Bizarre, je viens de revérifier (tu m’as mis le doute), j’ai bien cette erreur sur tous mes jeedom en version stable (tous en 4.3.21)

Tous avec l’erreur dans le fichier /var/www/html/desktop/php/backup.php (line 66) sur la commande substr
Et en corrigeant la fameuse ligne 66, ensuite je n’ai plus l’erreur.

[Bad]

Tu es bien en log DEBUG par défaut ?

J’avais vu aussi, mais il me semble que ces messages ne remontent pas sinon.

[Salvialf]

Merci, effectivement là j’ai l’erreur

@TiTidom La correction est appliquée sur la branche stable sans nouveau numéro de version il faudra donc mettre à jour le core pour qu’elle soit déployée

[TiTidom]

C’est tout bon, Jeedom mis à jour et plus l’erreur dans les logs. Parfait.

Merci et bonne journée,
TiTidom.