Fix randText

[aofc]

Description

Fix randText qui brise sous php8. La fonction utilise incorrectement evaluate en lui passant un array alors qu'elle attend un string.

Types of changes

• Bug fix (non-breaking change which fixes)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
  • This change is only breaking for integrators, not for external standards or end-users.
• Documentation improvement

PR checklist

• I have checked there is no other PR open for the same change.
• I have read the [La ligne directrice pour contribuer à ce projet / Contribution guidelines for this project).
• I grant the project the right to include and distribute the code under the GNU.
• I have added tests to cover my changes.
• I have verified that the code complies with the projects coding standards.
• [Required for new sniffs] I have added MD documentation for the sniff.

[Mips2648]

Bonjour,

Il y a effectivement un truc bizarre avec cette fonction randText car $result n'est en fait jamais utilisé.
Cependant je ne suis pas convaincu par la correction non plus;

je pense qu'il faudrait

• mieux décrire le problème rencontrer,
• identifier tous les cas d'usages

pour pouvoir fixer une solution ensuite

et dans tous les cas il faut faire le PR sur la branche 4.5.3 plutôt, pas sur la beta qui n'est plus utilisé

[aofc]

Ah c'est qu'il y a un message sur Github qui dit de le faire sur la branche Beta.  En tout cas la correction règle le problème, mais je vous laisse voir si on peut faire mieux Le dimanche 22 février 2026 à 09:47, Mips ***@***.***> a écrit :

…

Mips2648 left a comment (jeedom/core#3197) Bonjour, Il y a effectivement un truc bizarre avec cette fonction randText car $result n'est en fait jamais utilisé. Cependant je ne suis pas convaincu par la correction non plus; je pense qu'il faudrait - mieux décrire le problème rencontrer, - identifier tous les cas d'usages pour pouvoir fixer une solution ensuite et dans tous les cas il faut faire le PR sur la branche 4.5.3 plutôt, pas sur la beta qui n'est plus utilisé — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.