Merge pull request #134 from AltSchool/fix/no-defer-on-delete

Exclude delete requests from having fields deferred

dynamic_rest/filters.py

@@ -504,7 +504,11 @@ def _filter_queryset(
         # use requirements at this level to limit fields selected
         # only do this for GET requests where we are not requesting the
         # entire fieldset
-        if '*' not in requirements and not self.view.is_update():
+        if (
+            '*' not in requirements and
+            not self.view.is_update() and
+            not self.view.is_delete()
+        ):
             id_fields = getattr(serializer, 'get_id_fields', lambda: [])()
             # only include local model fields
             only = [

dynamic_rest/viewsets.py

@@ -15,6 +15,7 @@
 from dynamic_rest.renderers import DynamicBrowsableAPIRenderer
 
 UPDATE_REQUEST_METHODS = ('PUT', 'PATCH', 'POST')
+DELETE_REQUEST_METHOD = 'DELETE'
 
 
 class QueryParams(QueryDict):
@@ -240,6 +241,15 @@ def is_update(self):
         else:
             return False
 
+    def is_delete(self):
+        if (
+            self.request and
+            self.request.method.upper() == DELETE_REQUEST_METHOD
+        ):
+            return True
+        else:
+            return False
+
     def get_serializer(self, *args, **kwargs):
         if 'request_fields' not in kwargs:
             kwargs['request_fields'] = self.get_request_fields()