[Expression] ! pour les connaisseurs!

Utilisation du core de Jeedom Scénarios
100

Oui il y a un explode de l’expression par la virgule pour séparer les arguments dans setTags

101

Oui j’ai vue, je pense qu’on peux améliorer çà aussi, genre vérifier qu’il y a la fonction en question devant la parenthèse. Mais step by step

102

Ne pas modifier les chaines de caractères. Pas de remplacement et/ou pas d’explode…

103

Si c’était aussi simple. Tu fait comment pour récupérer variable(mavariableinexistante,0) ? Ce n’est pas là par hasard !

104

je ne sais pas pourquoi tu dis ça mais voici la page santé à tout hasard

image
image1301×401 8.21 KB

105

Tu est bien en php 7.3
Faudrait essayer symfony v5 sur un php5

106

Juste pour tester:

Ne pas faire çà sur une prod !!

Modifs:

utils.inc.php
function evaluate()
commenter la ligne du ireplace

$string = $_string;
//$string = str_ireplace(array(' et ', ' and ', ' ou ', ' or ', ' xor '), array(' && ', ' && ', ' || ', ' || ', ' ^ '), $_string);

=> donc plus de replace, seuls les && || sont supportés (php pur)

scenarioExpression.class.php:
function setTags()
Dans if (is_array($matches)) {

else {
    $arguments = explode(',', $match[2]);

else {
    if ($match[2][0] == $match[2][strlen($match[2]) - 1] && ($match[2][0] == '"' || $match[2][0] == "'")) {
        $arguments = array($match[2]);
    } else {
        $arguments = explode(',', $match[2]);
    }

=> çà règle le strlen('to,to')

La question étant, çà pète quoi de faire çà ??

Faudrait plein de tests et pouvoir les balancer à la volée (dans un scenario avec plein de if, ou dans un bloc code)

107

C’est parti

image

N’ayant pas autre chose que && || ça va tester le reste

108

C’est pas sans effet

image
image1477×258 62.1 KB

A voir si c’est pas une reliquat pendant la durée de la modif

110

Ouais c’est mort.

On aura essayé au moins lol.

111

Non non…
J’ai purgé les logs et pour l’instant ça semble ok.

Comme ça tourne pendant l’édition du code, ça génère des merdouilles temporairement et avec du délai

112

Ça ne devrait pas faire 5 ? :thinking:

113

C’est exactement ce qu’il aurait fallu depuis le début. Mais il doit sûrement y avoir une possibilité de parser les scénarios à la recherche de espace et espace
Idem pour tout ce qui est dans le str_replace.
Et signaler aux gens pour vérification.

114

C’est le cybermonday … Avec les quotes ça fait bien 7… Il y a les quotes automatiques encore chez moi

115

En php, ça fait 5.
Dans la version non modifiée, strlen('toto') donne 4.
C’est où/quoi les quotes auto ?

116

tout en bas de Réglages => Système => configuration => Equipement

image
image704×112 4.94 KB

1 « J'aime »
117

Çela brainstorm dur :+1:

Pour le point évoqué sur ce que cela pourrait casser en passant en syntaxe strict, une phase intermédiaire (version stable actuelle) qui remonte a l’utilisateur les scénarios qui ne respecte pas les futures évolutions et un renvoi vers les règles a utiliser ne permettrait il pas de gérer la transition.
Le risque c’est que plein d’utilisateurs ne vont pas traiter, il faudrait que la version intégrant le strict check la présence de ET OU …
C’est un beau casse tête.
Courage

118

C’est bien les quotes auto qui jouent, si je les désactive et que je conserve les modifs

image

So far, so good pour l’instant

119

On retrouve l’esprit jeedom, des gens doués qui participent partagent et au final c’est le produit qui sortira grandi.

Continuez. Perso j’ai pas les compétences. Mais c’est encourageant.

2 « J'aime »
120

Content d’avoir un autre avis identique au mien ! Ça me semble aller de père avec les changements évoqués ici :wink: