Call to undefined method teleinfoCmd::setEventOnly()

[maxi768]

Description

Etapes à reproduire (pour les bugs)

1. Modification de l'affichage du plugin

Contexte

Environnement

• Version Jeedom: 4.2.7
• Platform:
• **Version du Plugin **: 2021-02-04 01:01:47
• Type de modem:
• Compteur linky:
• Compteur standard ou historique:

Logs

0000|[Fri Feb 04 11:07:02.422631 2022] [php7:error] [pid 2538] [client 192.168.1.41:51027] PHP Fatal error:  Uncaught Error: Call to undefined method teleinfoCmd::setEventOnly() in /var/www/html/plugins/teleinfo/core/class/teleinfo.class.php:1023
0001|Stack trace:
0002|#0 /var/www/html/plugins/teleinfo/core/class/teleinfo.class.php(991): teleinfo->createOtherCmd()
0003|#1 /var/www/html/core/class/DB.class.php(232): teleinfo->postSave()
0004|#2 /var/www/html/core/class/eqLogic.class.php(1002): DB::save(Object(teleinfo), false)
0005|#3 /var/www/html/core/ajax/eqLogic.ajax.php(488): eqLogic->save()
0006|#4 {main}
0007|thrown in /var/www/html/plugins/teleinfo/core/class/teleinfo.class.php on line 1023, referer: http://192.168.1.14/index.php?v=d&m=teleinfo&p=teleinfo&id=139

Voir discussion sur forum : #https://community.jeedom.com/t/call-to-undefined-method-teleinfocmd-seteventonly/78071

[noyax]

Pour info: https://community.jeedom.com/t/modifications-plugin-teleinfo-checksum-en-mode-standard-traduction-du-champ-stge-traduction-du-champs-relais/76597

J'ai aussi fait une demande pour que ce soit intégré au code: #150

[Gis70]

Salut, j'ai la possibilité de merger, MAIS, va t'on casser la compatibilité avec Jeedom V4.1 ?

Je regarderai pour mettre une version mini dans le plugin. Je vais regarder pour mettre ça dans la develop, je vous tiens au courant.

[maxi768]

Salut,
Je ne pense pas...regarde la discussion sur le forum jeedom il semble que cette fonction était "deprecied" depuis un petit moment..

https://community.jeedom.com/t/call-to-undefined-method-teleinfocmd-seteventonly/78071

[Gis70]

Pouvez vous tester ? c'est sur la develop.

Fait moi signe et n'hésitez pas à faire des PR

[noyax]

tester quoi? Si ça fonctionne avec la 4.1? A part les 2 lignes que j'ai effacées pour rendre compatible avec la 4.2 le reste fonctionnait sous la V4.1 de jeedom

[Gis70]

tester que la branche develop fonctionne comme il faut sous jeedom v4.2, si oui alors je basculerai sur la master.

[noyax]

ok, je regarde ça demain

[noyax]

En regardant déjà avec winmerge je trouve une différence entre les fichiers

• teleinfo.class.php en ligne 548 une condition a été rajoutée (une modification en attente de validation?). Je ne pense pas qu'il y ait de pb avec celle ci.
  

• info.json a été modifié mais c'est pour indiquer les versions donc pas de pb je pense
  

A priori cela ne devrait pas poser de problèmes. Je teste.

[noyax]

la différence dans le teleinfo.class.php est liée à l'intégration du #145 , ok

[noyax]

je viens d'uploader les fichiers de la branche develop et ça fonctionne. Je n'ai pas tout testé mais ça a l'air bon

[noyax]

Il reste à réparer l'affichage dans le panel santé mais là je ne sais pas trop quoi faire...

[Gis70]

oui j'ai incrémenté les versions et mis une version mini, je ne suis pas développeur non plus. Si tu trouves qqn capable de corriger ce qui ne va pas sur le forum.

[noyax]

J'ai essayé de passer les fichiers compatibles avec python V3. Est ce que quelqu'un pourrait essayer chez lui voir s'il n'y a pas de pb? De préférence en mode historique car je suis en standard chez moi. Cela résout aussi les pb d'affichage dans le panel https://github.com/noyax/plugin-teleinfo/tree/decode_stge_et_checksum
Pour télécharger le zip directement: https://github.com/noyax/plugin-teleinfo/archive/refs/heads/decode_stge_et_checksum.zip