Bon après un petit dif entre la version en place sur mon systeme hier et la nouvelle version il n’y a pas des tonnes de modif mais il y en a une qui attire le regard. Cette ligne a été ajoutée dans la méthode « execute » pour les commandes de type info…
J’étais dessus également, c’est effectivement cette ligne 318 de ce commit qui est la cause je pense:
je ne suis pas sur de pourquoi elle a été ajoutée, j’imagine que dans certain cas si il y avait une expression à évaluer ce n’était pas la bonne valeur qui était envoyé la première fois?
Y’a un petit détail qui me dérange, ce sont ces 3 lignes :
Changelog Virtuel
IMPORTANT
Pour rappel s’il n’y a pas d’information sur la mise à jour, c’est que celle-ci concerne uniquement de la mise à jour de documentation, de traduction ou de texte
Or, il n’y a pas eu d’information sur la mise à jour de mi-mai et on a bien vu que du code avait été modifié… Un oubli ???
Merci quand même au développeur qui fait vraiment un super boulot !
je viens de découvrir une nouvelle « régression » dans le plugin virtal, en plus de faire marche arrière sur le problème rencontré précédemment une autre modification à été faite dans la dernière mise à jour.
$virtualCmd->event($result);
remplacé par
$eqLogic->checkAndUpdateCmd($virtualCmd,$result);
Et tout mon système d’interrupteurs qui utilise des virtuels ne fonctionne plus et évidement c’est en rentant ce soir que je m’en rend compte. Heureusement que j’avais vu ce matin cette modification. Mais sans en mesurer les conséquences.
Un retour à la version précédente de cette ligne et c’est repartit.
Bon on verra demain si j’arrive à expliquer mon nouveau problème ou alors si l’équipe explique la semaine prochaine cette nouvelle modification…
Finalement j’ai pris 5 minutes pour tenter de comprendre et le checkAndUpdateCmd ça doit être pour vérifier le comportement des commandes info suivant le paramétrage « Gestion de la répétition des valeurs ». Dans mon cas mes commandes d’interrupteurs ne sont pas configurées en « toujours répéter » et entre deux appuis elles ne changent par obligatoirement de valeur. Si j’ai bien compris jusqu’à la dernière modification, le plugin virtuel ne tenait pas compte de ce réglage ce qui vient d’être fait.
J’ai fait le test sur une commande et ça fonctionne.
On va dire que je suis bon pour corriger toutes mes commandes de ce type pour les passer en « toujours répéter ».
Pour ce soir je laisse l’ancienne ligne de code on verra demain mais si vous rencontrez le même problème vous pouvez corriger vos commandes.
Bonjour, Je rebondit sur ton Post @Phil56, J’avoue que je n’est jamais trop compris la différence entre « Automatique » et « toujours répéter ». A part le fait qu’on pouvait déduire dans les log que c’était une répétition ou pas.
Avant les MàJ de mémoire quand l’infos était en « Automatique » cela revenait a l’option « Toujours répéter » avec juste l’ajout (répétition) dans le log,
Automatique avant MaJ :
[2020-05-17 10:09:50][INFO] : Exécution de la commande [Maison][TEST3][ON] avec les paramètres {"utid":"xxxxxxxxxxxxx"}
[2020-05-17 10:09:50][INFO] : Evènement sur la commande [Maison][TEST3][ETAT] valeur : 1 (répétition)
Et effectivement maintenant après la MàJ, L’option « Automatique » agit comme l’option « jamais répéter ».
Automatique après MàJ :
[2020-05-17 10:14:27][INFO] : Exécution de la commande [Maison][TEST3][ON] avec les paramètres {"utid":"xxxxxxxxxxxxx"}
Donc autant enlever l’option Automatique, et avoir que les autres options disponible, car si en fonction de numérique ou binaire on a pas le même comportement en automatique, sa me semble un peu compliqué l’histoire.
@mchacher, si tu passes par ici et que tu as 5mn pour poster sur GitHub. Merci.
J’avoue que je suis un peu nouveau dans le domaine (découverte de Raspberry, Jeedom, Linux… y’a pas plus de 6mois ).
Donc je sais pas trop comment fonctionne ce petit monde de développeurs, est-ce qu’ils ont le temps de vérifier les problèmes remontés dans la communauté ou si vaut mieux posté sur GiHub.
En tout cas merci a eux.