jeedom-zigate / jeedom-plugin-zigate

Plugin Jeedom for ZiGate key.
MIT License
29 stars 12 forks source link

Add thermostat command #261

Closed fairecasoimeme closed 4 years ago

doudz commented 4 years ago

Cette modification implique la mise à jour de la librarie python pour ajouter les fonctions action_thermostat_occupied_heating_setpoint et action_thermostat_system_mode Toutefois il est possible d'obtenir le même résultat en utilisant la fonction existante write_attribute_request

doudz commented 4 years ago

Pourrais-tu corriger les erreurs suivantes ?

FILE: ...d/jeedom-zigate/jeedom-plugin-zigate/core/class/zigate.class.php
----------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 3 LINES
----------------------------------------------------------------------
 1047 | ERROR | [x] There must be no space before the colon in a CASE
      |       |     statement
 1047 | ERROR | [ ] There must be a comment when fall-through is
      |       |     intentional in a non-empty case body
 1049 | ERROR | [x] There must be no space before the colon in a CASE
      |       |     statement
 1050 | ERROR | [x] Spaces must be used for alignment; tabs are not
      |       |     allowed
 1050 | ERROR | [x] Whitespace found at end of line
----------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------
fairecasoimeme commented 4 years ago

Le PB c'est qu'il ne donne pas le numero de ligne ou alors il correspond pas. Fred

doudz commented 4 years ago

Effectivement les numéros de lignes ne correspondent à rien :thinking:

fairecasoimeme commented 4 years ago

Du coup, c'est pas facile ;)

fairecasoimeme commented 4 years ago

Merci pour l'aide ;) jpense que maintenant ce sera bon...

doudz commented 4 years ago

Par contre, je vois que tu ajoutes des commandes jeedom lock et unlock mais qu'ensuite elles ne sont pas géré, je me trompe ?

fairecasoimeme commented 4 years ago

euh normalement, j'ai rajouté les commandes lock et unlock et l'action était déjà géré il me semble

doudz commented 4 years ago

lock est géré effectivement