Merge pull request #45 from cclauss/patch-1

GitHub Action to lint Python code
Many thanks Christian for your help in reviewing the code !!

Author: philippelt

.github/workflows/lint_python.yml

@@ -0,0 +1,16 @@
+name: lint_python
+on: [pull_request, push]
+jobs:
+  lint_python:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-python@v2
+      - run: pip install black codespell flake8 isort pytest pyupgrade
+      - run: black --check . || true
+      - run: codespell --quiet-level=2 || true  # --ignore-words-list="" --skip=""
+      - run: flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+      - run: isort --profile black . || true
+      - run: pip install -r requirements.txt || true
+      - run: pytest . || true
+      - run: shopt -s globstar && pyupgrade --py36-plus **/*.py || true

lnetatmo.py

@@ -102,7 +102,7 @@ def getParameter(key, default):
 
 #TODO# Undocumented (but would be very usefull) API : Access currently forbidden (403)
 
-# _POST_UPDATE_HOME_REQ  = _BASE_URL + "/api/updatehome"
+_POST_UPDATE_HOME_REQ  = _BASE_URL + "/api/updatehome"
 
 # For presence setting (POST BODY):
 # _PRES_BODY_REC_SET     = "home_id=%s&presence_settings[presence_record_%s]=%s"   # (HomeId, DetectionKind, DetectionSetup.index)
@@ -275,7 +275,7 @@ def __init__(self, authData, home=None):
         resp = postRequest(_GETTHERMOSTATDATA_REQ, postParams)
         self.rawData = resp['body']['devices']
         if not self.rawData : raise NoDevice("No thermostat available")
-        self.thermostatData = filter_home_data(rawData, home)
+        self.thermostatData = filter_home_data(self.rawData, home)
         if not self.thermostatData : raise NoHome("No home %s found" % home)
         self.thermostatData['name'] = self.thermostatData['home_name']
         for m in self.thermostatData['modules']:
@@ -824,7 +824,7 @@ def getStationMinMaxTH(station=None, module=None, home=None):
             r = devList.MinMaxTH(module=m)
             result[m] = (r[0], lastD[m]['Temperature'], r[1])
     else:
-        if time.time()-lastD[mname]['When'] > 3600 : result = ["-", "-"]
+        if time.time()-lastD[module]['When'] > 3600 : result = ["-", "-"]
         else : 
             result = [lastD[module]['Temperature'], lastD[module]['Humidity']]
             result.extend(devList.MinMaxTH(module))