Plugin WES V4 : Beta

Je viens de faire la modification pour forcer le cast pour les relais et switchs.

1 « J'aime »

Il faut aussi le type pour le wes lui même avec le type « general », non ?

il faut donc faire une vérification si wes et si rien dans setConfiguration type alors

https://github.com/jeedom/plugin-wes/commit/9b2d556494d7388dd0d112d4f729a040ee7f599c

Il y a une coquille ligne 50 :wink:, Il manque les « > »

if($wes-getConfiguration('type') == null || $wes-getConfiguration('type') == ""){

au lieu de :

if($wes->getConfiguration('type') == null || $wes->getConfiguration('type') == ""){

J’ai fait le PR pour la coquille

https://github.com/jeedom/plugin-wes/pull/8

En revanche, il y a toujours le soucis avec la fonction configPush().

En laissant tel quel, cela génère une erreur :

ConfigPushError

Voir plus haut mon précédent post :

je suis de déplacement ce soir et toute la journée de demain. je ne pourrai pas regarder cette partie. je regarde donc vendredi merci pour ton aide :wink:

Il faudrait aussi faire la modification pour les commandes dans le fichier d’upgrade. Si je peux, j’essaie de te faire le PR ce soir.

Désolé encore en déplacement, il me reste encore un peux de kilomètres avant d’enfin retourner à la maison et au bureau.
Donc demain je peux possiblement regarde en après-midi.

J’ai préparé un PR https://github.com/jeedom/plugin-wes/compare/beta…arnog23:beta mais je ne l’ai pas poussé. Je te laisse y jeter un oeil avant.

Ce PR devrait fonctionner pour ceux qui n’ont pas encore fait la mise à jour, en revanche, pour ceux pour qui les eqLogic sont déjà renommés en « wes » mais pas les commandes, je ne vois pas comment faire dans le script d’upgrade.

Il faut aussi faire attention à ce que la boucle

foreach ($wess as $wes){

ne cassent pas les eqLogic et les cmd déjà renommés lors de la mise à jour suivante car tous les eqLogic et commandes seront déjà de type « wes ».

J’ai refais une correction car il y avait une erreur de typo :

isObject

au lieu de

is_object

Et j’ai poussé le PR : https://github.com/jeedom/plugin-wes/pulls

J’ai fait des tests en repartant du plugin original en V3 et j’ai forcé une mise à jour. Cela semble bien fonctionner. Les « eqType_name » des eqLogic passent bien en « wes » et le type est bien ajouté. Idem pour les commandes, les « eqType » passent bien en « wes » et le type est bien ajouté.

J’ai reforcé un deuxième update et les modifications sont bien conservées.

C’est merge :wink:

C’est demain en stable et bêta sur le market !

Merci à toi !

Pas de souci, avec plaisir :slight_smile:

Ayant acquis un WES recemment, je suis le premier intéréssé par le plugin.

Encore une petite correction pour la route concernant le menu déroulant pour choisir la tarification sur les téléinfo qui est passé à l’as avec la réécriture : https://github.com/jeedom/plugin-wes/pull/12

J’y ai aussi supprimé le paramètre $url de la fonction configPush() qui était bloquante et qui empechait d’utiliser cette fonction. Cela génèrait une erreur :

500 : [object Object]

Du coup, la partie Push pour le téléinfo ne pourra pas fonctionner (mais je n’ai toujours pas compris comment il pouvait fonctionner avant) mais au moins elle fonctionnera pour les autres (boutons, relais, switchs) en attendant de comprendre comment le Push est utilisé pour le téléinfo.

J’ai aussi refait une modif pour le cast car j’en avais oublié une partie. En revanche, il y a la partie pour la téléinfo ou je ne sais pas si cela est aussi necessaire et je n’ai pas encore pu faire de test.

  1. Pour la duplication des équipements lors du passage de la version multi classes à la version mono classe :

J’ai déplacé la migration pour les équipements de type « wes » en dernier dans la fonction wes_update() car lorsqu’elle était au début, lors du save de l’équipement, la fonction postUpdate() étant apellée, lorsque la vérification de l’existence des sous modules était faite, le type de ceux-ci n’ayant pas encore été changés, les équipements se créaient de nouveau.

  1. Pour le souci de remonté d’info sur les teleinfo :

J’ai corrigé le cast qui n’était pas bon

Le PR correspondant : https://github.com/jeedom/plugin-wes/pull/15

merci :wink: je viens de tester c’est bien mieux oui :wink:

Une petite question concernant la fonction « setEventOnly(1) » utiliser sur les commandes du WES. D’après ce que j’ai compris ici, elle est obselète :

Et dans le core , on trouve ca :

public function setEventOnly($eventOnly) {
		trigger_error('This method is deprecated', E_USER_DEPRECATED);
	}

Que faisait cette fonction ? Par quoi doit-on la remplacer si necessaire ou peut-on simplement la supprimer ?

Au passage, intéressant le Webinar avec Nicolas de Cartelectronic :wink:

je l’ai bien supprimer dans mon plugin pour info

Merci pour la confirmation.

@Alexandre le plugin en version stable ne fonctionne pas sur ma version jeedom 4.1 Buster.

Je dois mettre la version beta pour obtenir les champs pour configurer le WES (IP, login, …), une idée ?
EDIT : apres avoir fait la configuration en BETA, j’ai reinstalle la version STABLE et là je vois bien les champs d’identification → quid ?

Sinon serait il possible de permettre dans les commandes de pouvoir ajouter des commandes pour calculer la puissance par exemple sans passer par un virtuel ?

Merci