search

NextDom / plugin-teleinfo

Plugin Jeedom/NextDom permettant la gestion de la Téléinfo
https://nextdom.github.io/plugin-teleinfo/
GNU General Public License v3.0
11 stars 19 forks source link

Call to undefined method teleinfoCmd::setEventOnly() #151

Open maxi768 opened 2 years ago

maxi768 commented 2 years ago

Description

Etapes à reproduire (pour les bugs)

  1. Modification de l'affichage du plugin

Contexte

Environnement

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 commented 2 years ago

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: https://github.com/NextDom/plugin-teleinfo/pull/150/

Gis70 commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

Pouvez vous tester ? c'est sur la develop.

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

noyax commented 2 years ago

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 commented 2 years ago

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

noyax commented 2 years ago

ok, je regarde ça demain

noyax commented 2 years ago

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

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

noyax commented 2 years ago

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

noyax commented 2 years ago

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 commented 2 years ago

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

Gis70 commented 2 years ago

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 commented 2 years ago

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

noyax commented 2 years ago

Il y avait un pb au niveau de l'installation des dépendances qui prenait celle pour python 2.7 au lieu de la v3. Le dernier push est là pour y pallier

Gis70 commented 2 years ago

C'est poussé, pour l'instant je laisse les fichiers txt et backup mais quand tout marchera bien on fera du nettoyage ;)

noyax commented 2 years ago

Dans le fichier permettant l'installation des dépendances il n'y a apparemment pas toutes celles nécessaires au bon fonctionnement du plugin. si les modules absents du fichier ont été installés par un autre plugin il n'y a pas de pb mais sinon cela ne fonctionne pas. Je vais essayer de faire la liste de tous les modules nécessaires et de compléter le fichier ce soir.

noyax commented 2 years ago

Petit point de la situation: