Fix multiple PHPStan issues level 2

[kwizer15]

Description

Cette PR corrige plusieurs avertissements PHPStan niveau 2 dans le code Jeedom pour améliorer la qualité du code et la sécurité des types.

Principales corrections apportées :

• Remplacement de static:: par self:: dans les classes DB et cache pour les propriétés statiques privées, conformément aux bonnes pratiques PHP
• Correction des conversions de types avec des casts explicites (int) et (float) pour éviter les erreurs de type
• Suppression de paramètres inutilisés dans les appels de méthodes (cmdColor, paramètres superflus)
• Correction des signatures de méthodes avec les bons types de paramètres dans la documentation PHPDoc
• Amélioration de la gestion des dates avec des corrections dans la logique de calcul de dates
• Correction des erreurs de concaténation de chaînes dans les messages de log
• Amélioration de la vérification des types d'objets dans les conditions

Ces modifications n'impactent pas la fonctionnalité existante mais rendent le code plus robuste et conforme aux standards de qualité PHP.

Suggested changelog entry

Correction d'avertissements PHPStan niveau 2 : amélioration de la sécurité des types et nettoyage du code

Related issues/external references

N/A

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.

[zoic21]

Je suis pas sur au niveau des changement de static en self de mémoire ca pose soucis lors de l'utilisation de la classe avec plusieurs base de données.

[pifou25]

En effet: static permet d'étendre la classe par héritage et surcharger les méthodes / attributs, alors que self ne le permet pas. Si vous avez un cas d'usage avec plusieurs bases de données ça serait bien de le documenter, on pourrait ajouter un TU pour le vérifier. Sinon à l'inverse self est évalué à la compilation (vs au runtime pour static) c'est sans doute mieux pour l'utilisation d'une unique bdd, ce qui est notre cas à tous.

source

[Mips2648]

Bonjour,

Je souhaite qu'on puisse débloquer cette PR et la clôturer ou la merge, mais on ne peut pas garder des PR ouvertes indéfiniment.
Pour cela, je propose de ne garder que ce qui est "évident" et sans risque dans l'immédiat et de reporter à une prochaine version le reste

Donc à supprimer de cette PR:

• Remplacement de static:: par self:: dans les classes DB et cache pour les propriétés statiques privées, conformément aux bonnes pratiques PHP

• Correction des conversions de types avec des casts explicites (int) et (float) pour éviter les erreurs de type

• Amélioration de la gestion des dates avec des corrections dans la logique de calcul de dates

A garder:

• Suppression de paramètres inutilisés dans les appels de méthodes (cmdColor, paramètres superflus)

• Correction des signatures de méthodes avec les bons types de paramètres dans la documentation PHPDoc

Points que je n'ai pas retrouvé (donc je dirais à ne pas garder pour l'instant):

• Correction des erreurs de concaténation de chaînes dans les messages de log

• Amélioration de la vérification des types d'objets dans les conditions

De manière générale, je suggère d'y aller petit à petit dans ce genre de modifications, d'avoir des PR avec un scope beaucoup plus restreint sinon on arrivera pas à faire progresser celles-ci.

Donc ne pas re-soumettre une PR qui adresse tous les points restants (ni 5 différentes d'un coup), mais une à la fois avec la prochaine "priorité" dans la liste; sinon c'est trop compliqué de trouver les bonnes personnes qui vont reviews

@kwizer15 est-ce que cela te convient? peux-tu adapter la PR en conséquence?
merci

[kwizer15]

Oui ça me convient tu peux close si tu le souhaite. Je referai ça proprement. Le ven. 17 avr. 2026, 13:00, Mips ***@***.***> a écrit :

…

*Mips2648* left a comment (jeedom/core#3131) <#3131?email_source=notifications&email_token=AAEKOTRZUTL2DZQEUZ5IQX34WIFF7A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMRWG42DIMJWG442M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJNLQOJPWG33NNVSW45C7N5YGK3S7MNWGSY3L#issuecomment-4267441679> Bonjour, Je souhaite qu'on puisse débloquer cette PR et la clôturer ou la merge, mais on ne peut pas garder des PR ouvertes indéfiniment. Pour cela, je propose de ne garder que ce qui est "évident" et sans risque dans l'immédiat et de reporter à une prochaine version le reste Donc à supprimer de cette PR: - *Remplacement de static:: par self::* dans les classes DB et cache pour les propriétés statiques privées, conformément aux bonnes pratiques PHP - *Correction des conversions de types* avec des casts explicites (int) et (float) pour éviter les erreurs de type - *Amélioration de la gestion des dates* avec des corrections dans la logique de calcul de dates A garder: - *Suppression de paramètres inutilisés* dans les appels de méthodes ( cmdColor, paramètres superflus) - *Correction des signatures de méthodes* avec les bons types de paramètres dans la documentation PHPDoc Points que je n'ai pas retrouvé (donc je dirais à ne pas garder pour l'instant): - *Correction des erreurs de concaténation* de chaînes dans les messages de log - *Amélioration de la vérification des types d'objets* dans les conditions De manière générale, je suggère d'y aller petit à petit dans ce genre de modifications, d'avoir des PR avec un scope beaucoup plus restreint sinon on arrivera pas à faire progresser celles-ci. Donc ne pas re-soumettre une PR qui adresse tous les points restants (ni 5 différentes d'un coup), mais une à la fois avec la prochaine "priorité" dans la liste; sinon c'est trop compliqué de trouver les bonnes personnes qui vont reviews @kwizer15 <https://github.com/kwizer15> est-ce que cela te convient? peux-tu adapter la PR en conséquence? merci — Reply to this email directly, view it on GitHub <#3131?email_source=notifications&email_token=AAEKOTRZUTL2DZQEUZ5IQX34WIFF7A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMRWG42DIMJWG442M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJNLQOJPWG33NNVSW45C7N5YGK3S7MNWGSY3L#issuecomment-4267441679>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEKOTRPB32ENSVC5DS75DD4WIFF7AVCNFSM6AAAAACHOMYHAWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DENRXGQ2DCNRXHE> . You are receiving this because you were mentioned.Message ID: ***@***.***>