Generictype - declencheur sur etat 0 ne fonctionne pas

ngrataloup · Février 12, 2023, 10:20

Attention, c’est pas la modale de choix qui pose pb, mais bien le déclencheur si la valeur est = 0 qui ne déclenche pas si et seulement si on n’a pas indiqué de test dans le déclencheur.

Norbert

Phpvarious · Février 13, 2023, 4:46

@kiboost, je suis remonté jusqu’a ici :

github.com

jeedom/core/blob/alpha/core/class/scenario.class.php#L751

      
        
            	 * @return boolean
            	 */
            	public function testTrigger($_event) {
            		foreach (($this->getTrigger()) as $trigger) {
            			$trigger = str_replace(array('#variable(', ')#'), array('variable(', ')'), $trigger);
            			$trigger = str_replace(array('#genericType(', ')#'), array('genericType(', ')'), $trigger);
            			if ($trigger == $_event) {
            				return true;
            			} elseif (strpos($trigger, $_event) !== false && jeedom::evaluateExpression($trigger)) {
            				return true;
            			} elseif (strpos($trigger, 'genericType') !== false && jeedom::evaluateExpression($trigger)) {
            				return true;
            			}
            		}
            		return false;
            	}
            	/**
            	 *
            	 * @param type $_trigger
            	 * @param type $_message
            	 * @param type $_forceSyncMode

il y a jeedom::evaluateExpression($trigger)

qui lui renvoie vers scenarioExpression::setTags($_input, $_scenario, true)

github.com

jeedom/core/blob/beta/core/class/jeedom.class.php#L1298

      
        
            	public static function toHumanReadable($_input) {
            		return jeeObject::toHumanReadable(scenario::toHumanReadable(eqLogic::toHumanReadable(cmd::cmdToHumanReadable($_input))));
            	}
            
            
	public static function fromHumanReadable($_input) {
            		return jeeObject::fromHumanReadable(scenario::fromHumanReadable(eqLogic::fromHumanReadable(cmd::humanReadableToCmd($_input))));
            	}
            
            
	public static function evaluateExpression($_input, $_scenario = null) {
            		try {
            			return evaluate(scenarioExpression::setTags($_input, $_scenario, true));
            		} catch (Exception $exc) {
            			return $_input;
            		}
            	}
            
            
	public static function calculStat($_calcul, $_values, $_round = 1) {
            		switch ($_calcul) {
            			case 'sum':
            				return round(array_sum($_values), $_round);
            			case 'avg':

Le soucis c’est que dans la fonction setTags

$return = cmd::cmdToValue(str_replace(array_keys($replace1), array_values($replace1), str_replace(array_keys($replace2), array_values($replace2), $_expression)), $_quote);

$_expression vaut pour l’exemple genericType(MEDIA_STATE,#[object]#) si on a aucun équipement MEDIA_STATE a 1, le $return renvoie 0 ce qui équivaut pour la fonction evaluate() a un false.

Exemples :
trigger = genericType(MEDIA_STATE,#[object]#) → si tous les genType MEDIA_STATE a 0 → $return = 0 (false)

trigger = genericType(MEDIA_STATE,#[object]#) → si 1 genType MEDIA_STATE a 1 → $return = 1 (true)-> $scenario->launch

trigger = genericType(MEDIA_STATE,#[object]#)==0 → si tous les genType MEDIA_STATE a 0 → $return = 0==0 (true) → $scenario->launch

kiboost · Février 13, 2023, 5:09

Oui je me doutais un peu d’un truc comme çà, le 0 = false, mais je vois pas bien corriger çà sans casser tout le reste.

Phpvarious · Février 13, 2023, 5:11

C’est exactement se que je me disais , j’ai test quelque truc dans le setTag, mais a chaque fois sa casse autre chose.

ngrataloup · Février 13, 2023, 5:14

Faut pas casser jeedom pour corriger ça quand même
On peut vivre avec, y a un contournement

Norbert

kiboost · Février 13, 2023, 6:33

J’ai pas du tout bien comprendre …

#genericType(LIGHT_STATE,#[Salon]#)# == 0 fonctionne bien, le scénario se lance quand j’éteins la dernière lumière du salon

Ok je rattrape le train…

#genericType(LIGHT_STATE,#[Salon]#)# → KO Alors qu’effectivement il y a changement de valeur donc çà devrait triggerer

kiboost · Février 13, 2023, 6:45

çà, çà fonctionne :

	public static function evaluateExpression($_input, $_scenario = null) {
		try {
			$eval = evaluate(scenarioExpression::setTags($_input, $_scenario, true));
			if (strpos($_input, 'genericType') !== false) {
			    $eval = $eval === false ? false : true;
			}
			return $eval;
		} catch (Exception $exc) {
			return $_input;
		}
	}

La question suivante étant, qu’est-ce que çà casse ?

kiboost · Février 13, 2023, 6:55

Plus simple et plus propre,

la condition dans testTrigger :

} elseif (strpos($trigger, 'genericType') !== false && jeedom::evaluateExpression($trigger) !== false) {

@Phpvarious tu en pense quoi ?

Phpvarious · Février 13, 2023, 9:15

Après quelque test… c’est concluant

kiboost · Février 13, 2023, 9:16

Cool !! Tu a testé d’autres triggers generictype ? genre un lightstate qui change ne declenche pas un trigger shutterstate ou autre ?

Phpvarious · Février 13, 2023, 9:21

Non mais je pense pas que se soit possible étant donné que le testTrigger intervient après byGenericTrigger, donc les scénario concerné sont déja dans un array.
mais sa coute rien de tester

kiboost · Février 13, 2023, 9:22

J’aime pas faire des revert

Merci pour tes recherches en tout cas, c’est top

Phpvarious · Février 13, 2023, 9:27

testé avec plusieurs genType et pas de soucis particulier

kiboost · Février 13, 2023, 9:28

Super, je pousse

kilougari · Février 13, 2023, 9:57

Un grand merci !

Loic · Février 14, 2023, 9:57

Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.