Always set bundle + coerce query params

worf/views/base.py

@@ -204,10 +204,24 @@ def set_bundle(self, raw_bundle):
             self.bundle[field] = raw_bundle[key]
             self.keymap[field] = key
 
+    def set_bundle_from_request(self, request):
+        if request.method == "GET":
+            self.set_bundle_from_query_string(request)
+            return
+
+        self.set_bundle_from_request_body(request)
+
     def set_bundle_from_query_string(self, request):
         raw_bundle = self.flatten_bundle(parse_qs(request.META["QUERY_STRING"]))
 
-        self.set_bundle(raw_bundle)
+        strings = dict(true=True, false=False, null=None)
+
+        coerced_bundle = {
+            key: strings.get(str(value).lower(), value)
+            for key, value in raw_bundle.items()
+        }
+
+        self.set_bundle(coerced_bundle)
 
     def set_bundle_from_request_body(self, request):
         raw_bundle = {}
@@ -233,7 +247,7 @@ def dispatch(self, request, *args, **kwargs):
 
         try:
             self._check_permissions()  # only returns 200 or HTTP_EXCEPTIONS
-            self.set_bundle_from_request_body(request)
+            self.set_bundle_from_request(request)
             return handler(request, *args, **kwargs)  # calls self.serialize()
         except HTTP_EXCEPTIONS as e:
             return self.render_to_response(dict(message=e.message), e.status)

worf/views/list.py

@@ -86,8 +86,6 @@ def set_search_lookup_kwargs(self):
         if not self.filter_fields and not self.search_fields:
             return
 
-        self.set_bundle_from_query_string(self.request)
-
         # Whatever is not q or page as a querystring param will
         # be used for key-value search.
         query = self.bundle.pop("q", "").strip()