From: Alexander Vasarab <alexander@wylark.com>
Date: Thu, 18 Feb 2021 17:37:48 +0000 (-0800)
Subject: Fix possibility of a NoneType raising a TypeError
X-Git-Tag: v3.2.3^2~1^2~1
X-Git-Url: https://wylark.com/src/infoex-autowx.git/commitdiff_plain/855f04140e7f7b74406661274642fc51809b23fc

Fix possibility of a NoneType raising a TypeError
---

diff --git a/infoex-autowx.py b/infoex-autowx.py
index 2bac782..25ef415 100755
--- a/infoex-autowx.py
+++ b/infoex-autowx.py
@@ -275,6 +275,9 @@ def main():
             LOG.warning("BAD KEY wx_data['%s']", element_cd)
             continue
 
+        if infoex['wx_data'][element_cd] is None:
+            continue
+
         # do the conversion before the rounding
         if station['provider'] == 'nrcs' and station['units'] == 'metric':
             infoex['wx_data'][element_cd] = convert_nrcs_units_to_metric(element_cd, infoex['wx_data'][element_cd])
@@ -286,11 +289,9 @@ def main():
         #                   direction, wind gust, snow depth
         # 1 decimal place:  air temp, baro
         # Avoid transforming None values
-        if infoex['wx_data'][element_cd] is None:
-            continue
-        elif element_cd in ['wind_speed', 'WSPD', 'wind_direction',
-                            'RHUM', 'relative_humidity', 'WDIR',
-                            'wind_gust', 'SNWD', 'snow_depth']:
+        if element_cd in ['wind_speed', 'WSPD', 'wind_direction',
+                          'RHUM', 'relative_humidity', 'WDIR',
+                          'wind_gust', 'SNWD', 'snow_depth']:
             infoex['wx_data'][element_cd] = round(infoex['wx_data'][element_cd])
         elif element_cd in ['TOBS', 'air_temp', 'PRES', 'pressure']:
             infoex['wx_data'][element_cd] = round(infoex['wx_data'][element_cd], 1)