Skip to content

Fix multiple PHPStan issues#3122

Closed
kwizer15 wants to merge 1 commit intojeedom:developfrom
kwizer15:feat/phpstan-fix-lvl1
Closed

Fix multiple PHPStan issues#3122
kwizer15 wants to merge 1 commit intojeedom:developfrom
kwizer15:feat/phpstan-fix-lvl1

Conversation

@kwizer15
Copy link
Copy Markdown
Contributor

Description

Cette PR corrige les erreurs PHPStan détectées dans le code. Les modifications incluent :

  • Correction des signatures de méthodes : Suppression de paramètres non supportés ou en trop dans plusieurs appels de méthodes statiques
  • Initialisation des variables : Ajout d'initialisations pour éviter les erreurs "variable might not be defined"
  • Correction des méthodes statiques : Conversion de méthodes d'instance en méthodes statiques (cmd::dropInfluxDatabase(), cmd::historyInfluxAll())
  • Nettoyage des valeurs de retour : Suppression d'utilisation de valeurs de retour void et variables non définies
  • Amélioration de la logique métier : Remplacement de l'appel direct à historyInflux par la création d'une tâche cron dans historyInfluxAll()

Ces corrections améliorent la qualité du code sans changer le comportement fonctionnel de l'application.

Suggested changelog entry

Correction des erreurs d'analyse statique PHPStan - amélioration de la qualité 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

@kwizer15 kwizer15 changed the title Fix multiple PHPStan issues and update baseline Fix multiple PHPStan issues Sep 20, 2025
@kwizer15 kwizer15 force-pushed the feat/phpstan-fix-lvl1 branch from 4764307 to 98e99ed Compare September 23, 2025 21:02
@zoic21 zoic21 requested a review from Salvialf January 30, 2026 10:31
@Salvialf Salvialf changed the base branch from alpha to develop March 31, 2026 18:12
@kwizer15 kwizer15 force-pushed the feat/phpstan-fix-lvl1 branch 2 times, most recently from 2a38842 to 0e8c5a0 Compare April 16, 2026 20:36
@Mips2648 Mips2648 self-requested a review April 16, 2026 20:36
@kwizer15 kwizer15 force-pushed the feat/phpstan-fix-lvl1 branch from 0e8c5a0 to d5905bd Compare April 16, 2026 20:38
Copy link
Copy Markdown
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.

Cet PR touche beaucoup trop de fichiers, il faut découper par bloc fonctionnel logique
je trouve cela trop compliqué à review en l'état.
De plus cela mix différents types de problèmes (supposés) avec des améliorations qui changent le comportement contrairement à ce que dit la description

@kwizer15
Copy link
Copy Markdown
Contributor Author

En soit il n’y a vraiment pas de changement de comportement, c’est uniquement du dépilage de la baseline phpstan, mais je comprend tout à fait ta demande. Je vais découper proprement.

@Mips2648
Copy link
Copy Markdown
Collaborator

Ok thanks, I started to review the different PR
so can we close this one then?

@kwizer15 kwizer15 closed this Apr 17, 2026
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.

2 participants