Fixed #34 - The way it parsed an boolean was incorrect

web/api/management/samplingpoints/update_handler.py

@@ -8,22 +8,22 @@ class UpdateHandler:
     def get_validation_rules():
         rules = [
             {'name': 'id', 'required': True, 'type': str},
-			{'name': 'media_monitored', 'required': True, 'type': str},
-			{'name': 'station_id', 'required': True, 'type': str},
-			{'name': 'measurement_regime', 'required': True, 'type': str},
-			{'name': 'mobile', 'required': True, 'type': Utils.is_bool},
-			{'name': 'assessment_type', 'required': True, 'type': str},
-			{'name': 'station_classification', 'required': True, 'type': str},
-			{'name': 'used_aqd', 'required': True, 'type': Utils.is_bool},
-			{'name': 'main_emission_sources', 'required': True, 'type': str},
-			{'name': 'traffic_emissions', 'required': True, 'type': str},
-			{'name': 'heating_emissions', 'required': True, 'type': str},
-			{'name': 'industrial_emissions', 'required': True, 'type': str},
-			{'name': 'distance_source', 'required': True, 'type': str},
-			{'name': 'change_aei_stations', 'required': True, 'type': str},
-			{'name': 'begin_position', 'required': True, 'type': str},
-			{'name': 'end_position', 'required': True, 'type': str},
-			{'name': 'logger_id', 'required': True, 'type': str}
+            {'name': 'media_monitored', 'required': True, 'type': str},
+            {'name': 'station_id', 'required': True, 'type': str},
+            {'name': 'measurement_regime', 'required': True, 'type': str},
+            {'name': 'mobile', 'required': True, 'type': Utils.as_bool},
+            {'name': 'assessment_type', 'required': True, 'type': str},
+            {'name': 'station_classification', 'required': True, 'type': str},
+            {'name': 'used_aqd', 'required': True, 'type': Utils.as_bool},
+            {'name': 'main_emission_sources', 'required': True, 'type': str},
+            {'name': 'traffic_emissions', 'required': True, 'type': str},
+            {'name': 'heating_emissions', 'required': True, 'type': str},
+            {'name': 'industrial_emissions', 'required': True, 'type': str},
+            {'name': 'distance_source', 'required': True, 'type': str},
+            {'name': 'change_aei_stations', 'required': True, 'type': str},
+            {'name': 'begin_position', 'required': True, 'type': str},
+            {'name': 'end_position', 'required': True, 'type': str},
+            {'name': 'logger_id', 'required': True, 'type': str}
         ]
         return rules
 

web/api/management/stations/update_handler.py

@@ -11,24 +11,24 @@ class UpdateHandler:
             {'name': 'name', 'required': True, 'type': str},
             {'name': 'begin_position', 'required': True, 'type': str},
             {'name': 'end_position', 'required': True, 'type': str},
-			{'name': 'network_id', 'required': True, 'type': str},
-			{'name': 'city', 'required': True, 'type': str},
-			{'name': 'national_station_code', 'required': True, 'type': str},
-			{'name': 'media_monitored', 'required': True, 'type': str},
-            {'name': 'mobile', 'required': True, 'type': Utils.is_bool},
-			{'name': 'measurement_regime', 'required': True, 'type': str},
-			{'name': 'area_classification', 'required': True, 'type': str},
+            {'name': 'network_id', 'required': True, 'type': str},
+            {'name': 'city', 'required': True, 'type': str},
+            {'name': 'national_station_code', 'required': True, 'type': str},
+            {'name': 'media_monitored', 'required': True, 'type': str},
+            {'name': 'mobile', 'required': True, 'type': Utils.as_bool},
+            {'name': 'measurement_regime', 'required': True, 'type': str},
+            {'name': 'area_classification', 'required': True, 'type': str},
             {'name': 'distance_junction', 'required': True, 'type': Utils.as_int},
-			{'name': 'traffic_volume', 'required': True, 'type': Utils.as_int},
+            {'name': 'traffic_volume', 'required': True, 'type': Utils.as_int},
             {'name': 'heavy_duty_fraction', 'required': True, 'type': Utils.as_numeric},
             {'name': 'street_width', 'required': True, 'type': Utils.as_int},
             {'name': 'height_facades', 'required': True, 'type': Utils.as_int},
-			{'name': 'latitude', 'required': True, 'type': Utils.as_numeric},
+            {'name': 'latitude', 'required': True, 'type': Utils.as_numeric},
             {'name': 'longitude', 'required': True, 'type': Utils.as_numeric},
             {'name': 'epsg', 'required': True, 'type': Utils.as_int},
             {'name': 'altitude', 'required': True, 'type': Utils.as_int},
-			{'name': 'municipality', 'required': True, 'type': str},
-			{'name': 'eoi_code', 'required': True, 'type': str}
+            {'name': 'municipality', 'required': True, 'type': str},
+            {'name': 'eoi_code', 'required': True, 'type': str}
         ]
         return rules
 

web/helpers/utils.py

@@ -52,7 +52,7 @@ class Utils:
     @staticmethod
     def as_bool(val):
         if Utils.is_bool(val):
-            return val.lower() == "true"
+            return str(val).lower() == "true"
         else:
             raise ValueError("Value is not of type bool")