Skip to content

Conversation

kwizer15
Copy link
Contributor

@kwizer15 kwizer15 commented Feb 6, 2025

Documentation PHPDoc de la classe cmd

Description

Cette PR standardise la documentation PHPDoc de la classe cmd (cmd.class.php).
Aucun changement fonctionnel n'est effectué.

Note : Les types documentés sont basés sur l'analyse statique du code. Il est possible que certains types soient incomplets ou imprécis, notamment pour les valeurs de retour complexes ou les tableaux associatifs. N'hésitez pas à signaler toute erreur ou imprécision dans les types documentés pour améliorer la qualité de la documentation. Les retours d'expérience sur l'utilisation réelle des méthodes sont particulièrement appréciés pour affiner la documentation.

Merci de créer de commenter directement cette PR si vous constatez des erreurs ou avez des suggestions d'amélioration.

Points clés

  • Documentation complète des classes et méthodes avec types stricts
  • Documentation précise des types de retour
  • Spécification des dépendances entre les composants via @see
  • Respect des standards de documentation Jeedom

Suggested changelog entry

  • Added standardized PHPDoc documentation for cmd class

Related issues/external references

https://github.com/jeedom/documentations/blob/89cf4578ad04b59abebe40217b6cfdf0dacc195e/fr_FR/dev/php/phpdoc.md
https://community.jeedom.com/t/aide-phpdoc-du-core/63287

Types of changes

  • Bug fix
  • New feature
  • Breaking change
  • Documentation improvement

PR checklist

  • I have checked there is no other PR open for the same change
  • I have read the 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 Mips2648 self-requested a review February 6, 2025 16:31
Copy link
Collaborator

@Mips2648 Mips2648 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

return type is incorrecte for most of the function, e.g.: return static[]

@kwizer15 kwizer15 requested a review from Mips2648 February 7, 2025 08:19
Copy link
Contributor

@Hotfirenet Hotfirenet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kwizer15 Il faut mettre a jour il y a eu des modification depuis

@kwizer15
Copy link
Contributor Author

Ca marche je ferais un rebase

@kwizer15
Copy link
Contributor Author

kwizer15 commented May 21, 2025

C'est bon. Y a juste une petite modif de code : j'ai retiré le isset($level) car il existe forcément dans le cas où on se trouve. Ca provoquait une erreur phpstan.
De plus j'ai rajouté des void|null car on utilise le résultat dans d'autres parties du code. Et ca impliquait des changements plus tendus autrement.
Ces changements sont dans un commit séparé.

@kwizer15 kwizer15 requested a review from Hotfirenet July 6, 2025 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants