Add support for boolean query params

Author: wscourge

chartmogul/api/data_source.py

@@ -42,19 +42,6 @@ class DataSource(Resource):
         defaults=[None, None, None]
     )
 
-    @classmethod
-    def _preProcessParams(cls, params):
-        params = super()._preProcessParams(params)
-
-        for query_param in cls._bool_query_params:
-            if query_param in params and isinstance(params[query_param], bool):
-                if params[query_param] is True:
-                    params[query_param] = 'true'
-                else:
-                    del params[query_param]
-
-        return params
-
     class _Schema(Schema):
         uuid = fields.String()
         name = fields.String()

chartmogul/api/invoice.py

@@ -40,6 +40,10 @@ class Invoice(Resource):
 
     _path = "/import/customers{/uuid}/invoices"
     _root_key = "invoices"
+    _bool_query_params = [
+        'include_edit_histories',
+        'with_disabled'
+    ]
     _many = namedtuple(
         "Invoices",
         [_root_key, "cursor", "has_more", "customer_uuid"],

chartmogul/resource.py

@@ -121,6 +121,15 @@ def _preProcessParams(cls, params):
             if key in params:
                 params[replacement] = params[key]
                 del params[key]
+
+        if hasattr(cls, '_bool_query_params'):
+            for query_param in cls._bool_query_params:
+                if query_param in params and isinstance(params[query_param], bool):
+                    if params[query_param] is True:
+                        params[query_param] = 'true'
+                    elif params[query_param] is False:
+                        params[query_param] = 'false'
+
         return params
 
     @classmethod