Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@kwizer15
Copy link
Contributor

Description

Cette PR corrige une erreur PHPStan dans la classe scenarioExpression où la variable $_parameters était utilisée au lieu de $options pour accéder au paramètre theme lors de la génération de rapports.

Problème identifié :

  • Le commit 362e4c2 (19 janvier 2023) avait ajouté la fonctionnalité de sélection de thème pour les rapports
  • Dans scenarioExpression.class.php, le code utilisait incorrectement $_parameters['theme'] au lieu de $options['theme']
  • Cette variable $_parameters n'existe pas dans ce contexte, causant une erreur PHPStan

Solution appliquée :

  • Remplacement de $_parameters['theme'] par $options['theme'] dans les 4 occurrences (lignes 1858, 1868, 1879, 1889)
  • Cohérence avec les autres paramètres utilisés dans la même fonction (ex: $options['export_type'])
  • Cohérence avec l'interface utilisateur qui stocke la valeur dans $options['theme']

Suggested changelog entry

  • Core : Correction du paramètre thème pour les rapports de scénarios

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

@Sekiro-kost Sekiro-kost merged commit f9313dd into jeedom:alpha Sep 24, 2025
8 checks passed
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.

3 participants