Espaces dans les paramètres des fonctions scenario

Suite du sujet Je constate des résultats qui me semblent erronés sur la fonction time_between :

hello
voir Je constate des résultats qui me semblent erronés sur la fonction time_between
je propose :
scenarioExpessionClass/setTags : L1177
$_expression= trim( $_expression );
/* on se débarrasse des espaces en début et fin d’expression

Bonjour,

Je pense que les améliorations du source code de Jeedom doivent être proposée sur le github afin que les développeurs en chargent de les intégrer puissent voir la modif proposée et l’accepter ou pas.

:wink:

merci Actarus.Procyon
tu as raison, mais je ne sais pas comment faire.
Je vais rechercher pour la prochaine fois. J’ai déjà regardé il y à quelque temps, mais entre les push, les pull, etc. c’est encore un peu confus

Attention a ce genre de modification ca peut semblait bien dans ton cas et dans surement beaucoup de cas mais ca peut avoir des effets de bords chez d’autre utilisateurs. Perso je ne suis pas pour car les risques sont trop grand mais si tu fais un PR je l’accepterais.

Merci Loic
à mon humble avis que l’on soit a l’analyse d’une expression, ou d’un paramètre isolé, les espaces avant et après ne sont (doivent ?) pas être pris en compte.
qu’est ce ce que c’est un PR.
si tu as de la doc sur l’utilisation de GitHub avec jeedom, je suis preneur

Bonjour,

Un PR c’est un pull request
Ce n’est pas spécifique a jeedom c’est du github

Moultes tutos existent déjà ici ou sur Google.