Skip to content

Commit

Permalink
Merge pull request #1429 from swirlai/ds-2902
Browse files Browse the repository at this point in the history
[WIP] fix the PATCH button on the endpoints
  • Loading branch information
erikspears authored Sep 16, 2024
2 parents 89693ac + b7ec4ce commit 0dba39c
Showing 1 changed file with 16 additions and 5 deletions.
21 changes: 16 additions & 5 deletions swirl/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -313,7 +313,7 @@ def update(self, request, pk=None):
serializer.is_valid(raise_exception=True)
# security review for 1.7 - OK, saved with owner
serializer.save(owner=self.request.user)
return Response(serializer.data, status=status.HTTP_201_CREATED)
return Response(serializer.data, status=status.HTTP_200_OK)

########################################

Expand All @@ -332,6 +332,8 @@ def destroy(self, request, pk=None):
searchprovider.delete()
return Response('SearchProvider Object Deleted', status=status.HTTP_410_GONE)

def partial_update(self, request, pk=None):
return self.update(request, pk)

########################################
########################################
Expand Down Expand Up @@ -612,7 +614,7 @@ def update(self, request, pk=None):
# search_task.delay(search.id, Authenticator().get_session_data(request))
logger.info(f"{request.user} search_put {search.id}")
run_search(search.id, Authenticator().get_session_data(request), request=request)
return Response(serializer.data, status=status.HTTP_201_CREATED)
return Response(serializer.data, status=status.HTTP_200_OK)

########################################

Expand All @@ -632,6 +634,9 @@ def destroy(self, request, pk=None):
search.delete()
return Response('Search Object Deleted', status=status.HTTP_410_GONE)

def partial_update(self, request, pk=None):
return self.update(request, pk)

########################################
########################################

Expand Down Expand Up @@ -765,7 +770,7 @@ def update(self, request, pk=None):
serializer.is_valid(raise_exception=True)
# security review for 1.7 - OK, saved with owner
serializer.save(owner=self.request.user)
return Response(serializer.data, status=status.HTTP_201_CREATED)
return Response(serializer.data, status=status.HTTP_200_OK)

########################################

Expand All @@ -785,6 +790,9 @@ def destroy(self, request, pk=None):
result.delete()
return Response('Result Object Deleted!', status=status.HTTP_410_GONE)

def partial_update(self, request, pk=None):
return self.update(request, pk)

########################################
########################################

Expand Down Expand Up @@ -894,7 +902,7 @@ def update(self, request, pk=None):
serializer.is_valid(raise_exception=True)
# security review for 1.7 - OK, saved with owner
serializer.save(owner=self.request.user)
return Response(serializer.data, status=status.HTTP_201_CREATED)
return Response(serializer.data, status=status.HTTP_200_OK)

########################################

Expand All @@ -910,7 +918,10 @@ def destroy(self, request, pk=None):

searchprovider = QueryTransform.objects.get(pk=pk)
searchprovider.delete()
return Response('QueryTranformation Object Deleted', status=status.HTTP_410_GONE)
return Response('QueryTransformation Object Deleted', status=status.HTTP_410_GONE)

def partial_update(self, request, pk=None):
return self.update(request, pk)

def query_transform_form(request):
if request.method == 'POST':
Expand Down

0 comments on commit 0dba39c

Please sign in to comment.