[BUG] several calls to a method of $scenario in scenarioExpression::execute() while $scenarion is null

[Mips2648]

Description

We must fix several calls to a method of $scenario in scenarioExpression::execute() while $scenario is null

Type de matériel / Hardware type

Autre / Other

Version de Jeedom / Jeedom Version

4.5.2

Navigateur / Browser

No response

Étapes pour reproduire / Steps to reproduce

Issue will happen if method are invoked from "action on value" for example

Logs

No response

[Phpvarious]

Bien vu, et je pense que par la même occasion il faudrait réorganiser l'autoComplete qui est filtrer en fonction si scenario ou pas :

core/core/js/scenario.class.js

Lines 563 to 592 in 77e57a7

	/* Actions Autocomplete */
	jeedom.scenario.autoCompleteAction = [
	'setColoredIcon',
	'tag',
	'report',
	'exportHistory',
	'sleep',
	'variable',
	'delete_variable',
	'scenario',
	'wait',
	'gotodesign',
	'message',
	'equipement',
	'ask',
	'jeedom_poweroff',
	'alert',
	'popup',
	'event',
	'changeTheme',
	'remove_inat',
	'genericType'
	]
	/* Actions Autocomplete only for scenarios*/
	jeedom.scenario.autoCompleteActionScOnly = [
	'stop',
	'log',
	'scenario_return',
	'icon'
	]

Basculer ask, tags... dans autoCompleteActionScOnly

[Mips2648]

@Phpvarious je ne sais pas t'ajouter comme reviewer mais si tu as l'occasion de jeter un oeil c'est par là: #3228

[Phpvarious]

Je pense que tu as tous bien sécurisé.

public function execute(?scenario &$scenario = null) {
juste cette partie, que je maitrise pas, je ne suis qu'un simple électricien :-)

Tu peux ajouter ma PR dedans ? comme ca je peux la clore.

[Mips2648]

public function execute(?scenario &$scenario = null) {
juste cette partie, que je maitrise pas, je ne suis qu'un simple électricien :-)

syntaxe possible depuis php 7.1; cela veut dire que le paramètre est de type scenario mais peut être null (le ?) et que la valeur par défaut est null: https://www.php.net/manual/en/migration71.new-features.php
Cela permet d'avoir du code completion correct dans l'éditeur

Tu peux ajouter ma PR dedans ? comme ca je peux la clore.

ok, refais une PR depuis ta branche patch sur la branch https://github.com/jeedom/core/tree/fix/scenarioExpression-execute-without-scenario si possible stp, ainsi on garde la trace que tu avais corrigé :-)