Skip to content

Fix setTags() reference error in interactQuery#3407

Open
aofc wants to merge 2 commits into
jeedom:developfrom
aofc:patch-12
Open

Fix setTags() reference error in interactQuery#3407
aofc wants to merge 2 commits into
jeedom:developfrom
aofc:patch-12

Conversation

@aofc

@aofc aofc commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Correction du bogue affectant les tags passés en arguments à un scénario

Description

Les tags passés en argument aux scénarios, depuis les interactions, provoquent une erreur de type: "Erreur lors de l'exécution de scenario. Détails : scenarioExpression::setTags(): Argument #1 ($_expression) could not be passed by reference" qui empêche l'exécution du scénario.

Correction de deux appels à setTags() dans interactQuery::executeAndReply().

Suggested changelog entry

Fix erreur fatale setTags(): Argument #1 could not be passed by reference empêchant l'exécution des interactions avec tags

Related issues/external references

Fixes #
Correction de deux appels à setTags() dans interactQuery::executeAndReply(). . Testé localement

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

Correction du bogue affectant les tags passés en arguments à un scénario
@Salvialf Salvialf self-requested a review June 30, 2026 12:51
Comment thread core/class/interactQuery.class.php Outdated
@Salvialf Salvialf changed the title Refactor tag replacement logic in interactQuery class Fix setTags() reference error in interactQuery Jun 30, 2026
@Salvialf Salvialf added the changelog-fix Use to generate release notes / changelog To be apply on PR label Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-fix Use to generate release notes / changelog To be apply on PR Need review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants