Bug Résumé Objets

schlew · Août 8, 2022, 7:03

C’est pour ça que pour la porte d’entrée j’ai: fermé/ouvert et verrouillée/déverrouillée et pour la porte de garage ouvert ou fermé. C’est du texte.

Phpvarious · Août 8, 2022, 7:24

Attention je me répète mais je cherche pas a dire c’est normal / pas normale, mais juste a comprendre comment devrait se comporter le résumé.

Je comprend mais comment ceci remonte dans le résumé en 4.1 ? si tu as une porte qui renvoi fermé/ouvert et une autre verrouillée/déverrouillée ? seul un numérique peut s’afficher et non un texte !

schlew · Août 8, 2022, 7:31

Regarde mon premier poste… Du texte peut s’afficher

Phpvarious · Août 8, 2022, 7:39

J’ai bien saisi qu’il y avait un bug entre résumé global et résumé objet ainsi que le refresh lors d’un update (je reproduit aussi), mais si je prend ton exemple en 4.1 du résumé global « présence » je peut savoir quelles commandes sont derrière celle-ci ? un virtuel qui converti (plusieurs présences) en texte « Présence » lorsqu’au moins une personnes est présente, ou alors c’est plusieurs commande qui renvoi présent ou absent ?

schlew · Août 8, 2022, 7:44

Je crois qu’on ne se comprend pas… .
Pour mon exemple, une seule commande qui dit si « présence » ou « absence »!

Phpvarious · Août 8, 2022, 7:49

Si justement je comprend que tu utilise 1 seul équipement pour 1 résumé, si tu duplique ton équipement et que tu l’intègre aussi dans le résumé global, comment ce comporte l’affichage du résumé globale ?
car en 4.2 il affiche tous ce quil trouve :
exemple 2 équipements 1 a off et l’autre a on dans le même résumé « Alarme » me renvoie :

schlew · Août 8, 2022, 7:58

Comme tu peux le voir dans le premier post, j’ai bien « verrouillée, fermé » pour la porte d’entrée dans le global mais pas dans la pièce/objet « entrée » où j’ai « 0 » en 4.2.
Pareil pour présence où j’ai présence dans le global et 0 dans l’objet maison.
Ce qui n’est pas le cas en 4.1.

Phpvarious · Août 8, 2022, 8:16

Ah oui donc le résumé global ce comporte comme en 4.2 juste un soucis lors d’un update et résumé objet, le bug sur l’update résumé global je pense l’avoir ciblé.
si sa peut aider @Loic ou autre devs :

github.com

jeedom/core/blob/6c378e491cd62381dbf94487acc76efc84edb07a/core/class/jeedom.class.php#L1295

      
        
            	}
            
            
	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) {
            		switch ($_calcul) {
            			case 'sum':
            				return array_sum($_values);
            			case 'avg':
            				return array_sum($_values) / count($_values);
            			case 'text':
            				return trim(implode(',', $_values), ',');
            		}
            		return null;
            	}

la fonction calculStat() fait un implode des valeurs si on est dans la case text et la retourne donc au format string (valeurs séparé par le caractère ,)

hors dans JeeObject.class (la class qui est appelée lors d’un update du résumé global)

github.com

jeedom/core/blob/alpha/core/class/jeeObject.class.php#L392

      
        
            			}
            			$result = $object->getSummary($_key, true);
            			if ($result === null || !is_array($result)) {
            				continue;
            			}
            			$value = array_merge($value, $result);
            		}
            		if (count($value) == 0) {
            			return null;
            		}
            		return round(jeedom::calculStat($def[$_key]['calcul'], $value), 1);
            	}
            
            
	public static function getGlobalHtmlSummary($_version = 'dashboard') {
            		$virtual = eqLogic::byLogicalId('summaryglobal', 'virtual');
            		$objects = self::all();
            		$def = config::byKey('object:summary');
            		$values = array();
            		$return = '<span class="objectSummaryContainer objectSummaryglobal" data-version="' . $_version . '">';
            		foreach ($def as $key => $value) {
            			foreach ($objects as $object) {

la fonction getGlobalSummary fait appel a cette précédente fonction, mais on peut voir quelle exécute un round() sur la valeur renvoyé, donc un round sur une valeur string se qui renvoie toujours 0.

schlew · Août 8, 2022, 8:21

Et ben, doué dis donc !

Loic · Août 9, 2022, 1:30

Merci pour l’analyse j’ai fait un changement en alpha 4.3 a voir si ca corrige bien le soucis, si vous me confirmez je passerais la corrections en 4.2

Phpvarious · Août 9, 2022, 6:52

Bonsoir,
Alpha mis a jour à l’instant :

debug_resume_1

sa me semble correspondre a la demande

schlew · Août 10, 2022, 4:26

Merci.
Je teste ça sur une machine hors prod et je te dis.

schlew · Août 10, 2022, 5:47

Ca à l’air d’être bien (voir le 1er résumé « verrouillée, fermé » dans global et objet Maison):
4.1.28:

MAJ en 4.2.20:

Après MAJ des infos:

MAJ en alpha 4.3.0:

Après MAJ des infos:

TOP!

Loic · Août 10, 2022, 5:49

Super merci pour le retour je passerai la correction en beta demain (normalement)

schlew · Août 10, 2022, 5:51

Merci bcp!!!

Loic · Août 11, 2022, 8:18

Voila c’est en beta (pas de date de stable pour le moment). Merci pour vos analyses et tests et retours.

system · Août 12, 2022, 8:18

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