Skip to content

Update scenarioExpression.class.php#2041

Merged
zoic21 merged 2 commits intojeedom:alphafrom
Phpvarious:patch-17
Aug 11, 2022
Merged

Update scenarioExpression.class.php#2041
zoic21 merged 2 commits intojeedom:alphafrom
Phpvarious:patch-17

Conversation

@Phpvarious
Copy link
Contributor

Lorsque la fonction est appelée avec une commande historisée mais sans donnée en BDD, elle renvoie une erreur "Undefined offset: 0" et ne permet pas de sauvegarder un virtuel, car celui-ci déclenche une "erreur 500".

Proposed change

Type of change

  • 3rd party lib update
  • [X ] Bugfix (non breaking change)
  • Core new feature
  • UI new functionnality
  • Code quality improvements
  • Core documentation

Test check

Documentation

beta-testing
contribute
community
plugins

Lorsque la fonction est appelée avec une commande historisée mais sans donnée en BDD, elle renvoie une erreur "Undefined offset: 0" et ne permet pas de sauvegarder un virtuel, car celui-ci déclenche une "erreur 500".
@zoic21
Copy link
Contributor

zoic21 commented Aug 10, 2022

Bonjour
Ça me semble une bonne idée par contre j'aurai plutôt renvoyé un number 0 plutôt que string 0

Lorsque la fonction est appelée avec une commande historisée mais sans donnée en BDD, elle renvoie une erreur "Undefined offset: 0" et ne permet pas de sauvegarder un virtuel, car celui-ci déclenche une "erreur 500", elle renverra désormais un number 0.
@Phpvarious
Copy link
Contributor Author

Hello,
bien vu, j'ai modifié mon PR, Merci Loic.

@zoic21 zoic21 merged commit 0b211cb into jeedom:alpha Aug 11, 2022
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