diff options
| author | Tom Christie | 2013-02-06 21:28:03 +0000 |
|---|---|---|
| committer | Tom Christie | 2013-02-06 21:28:03 +0000 |
| commit | c18fb0d6953940f63cd8747a5ce543d31999996f (patch) | |
| tree | 08915d4cf65b22ed24483f7371a36d1cd96d6482 /rest_framework/mixins.py | |
| parent | 123c6cbc7cebf797ed0b310a5a8d398fa6323ce4 (diff) | |
| download | django-rest-framework-c18fb0d6953940f63cd8747a5ce543d31999996f.tar.bz2 | |
Added a `post_save` hook. Closes #558.
Diffstat (limited to 'rest_framework/mixins.py')
| -rw-r--r-- | rest_framework/mixins.py | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/rest_framework/mixins.py b/rest_framework/mixins.py index 73a3d200..61ac225b 100644 --- a/rest_framework/mixins.py +++ b/rest_framework/mixins.py @@ -22,6 +22,7 @@ class CreateModelMixin(object): if serializer.is_valid(): self.pre_save(serializer.object) self.object = serializer.save() + self.post_save(self.object, created=True) headers = self.get_success_headers(serializer.data) return Response(serializer.data, status=status.HTTP_201_CREATED, headers=headers) @@ -34,9 +35,6 @@ class CreateModelMixin(object): except (TypeError, KeyError): return {} - def pre_save(self, obj): - pass - class ListModelMixin(object): """ @@ -88,12 +86,15 @@ class UpdateModelMixin(object): """ def update(self, request, *args, **kwargs): partial = kwargs.pop('partial', False) + self.object = None try: self.object = self.get_object() - success_status_code = status.HTTP_200_OK except Http404: - self.object = None + created = True success_status_code = status.HTTP_201_CREATED + else: + created = False + success_status_code = status.HTTP_200_OK serializer = self.get_serializer(self.object, data=request.DATA, files=request.FILES, partial=partial) @@ -101,6 +102,7 @@ class UpdateModelMixin(object): if serializer.is_valid(): self.pre_save(serializer.object) self.object = serializer.save() + self.post_save(self.object, created=created) return Response(serializer.data, status=success_status_code) return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST) |
