DRF 3.1 now uses request.data instead of request.DATA
This commit is contained in:
@@ -162,7 +162,7 @@ class AccountView(APIView):
|
||||
"""
|
||||
try:
|
||||
with transaction.commit_on_success():
|
||||
update_account_settings(request.user, request.DATA, username=username)
|
||||
update_account_settings(request.user, request.data, username=username)
|
||||
except UserNotAuthorized:
|
||||
return Response(status=status.HTTP_403_FORBIDDEN if request.user.is_staff else status.HTTP_404_NOT_FOUND)
|
||||
except UserNotFound:
|
||||
|
||||
@@ -107,7 +107,7 @@ class PreferencesView(APIView):
|
||||
"""
|
||||
PATCH /api/user/v1/preferences/{username}/
|
||||
"""
|
||||
if not request.DATA or not getattr(request.DATA, "keys", None):
|
||||
if not request.data or not getattr(request.data, "keys", None):
|
||||
error_message = _("No data provided for user preference update")
|
||||
return Response(
|
||||
{
|
||||
@@ -118,7 +118,7 @@ class PreferencesView(APIView):
|
||||
)
|
||||
try:
|
||||
with transaction.commit_on_success():
|
||||
update_user_preferences(request.user, request.DATA, username=username)
|
||||
update_user_preferences(request.user, request.data, username=username)
|
||||
except UserNotAuthorized:
|
||||
return Response(status=status.HTTP_403_FORBIDDEN)
|
||||
except UserNotFound:
|
||||
@@ -218,7 +218,7 @@ class PreferencesDetailView(APIView):
|
||||
PUT /api/user/v1/preferences/{username}/{preference_key}
|
||||
"""
|
||||
try:
|
||||
set_user_preference(request.user, preference_key, request.DATA, username=username)
|
||||
set_user_preference(request.user, preference_key, request.data, username=username)
|
||||
except UserNotAuthorized:
|
||||
return Response(status=status.HTTP_403_FORBIDDEN)
|
||||
except UserNotFound:
|
||||
|
||||
@@ -875,7 +875,7 @@ class UpdateEmailOptInPreference(APIView):
|
||||
assume False.
|
||||
|
||||
"""
|
||||
course_id = request.DATA['course_id']
|
||||
course_id = request.data['course_id']
|
||||
try:
|
||||
org = locator.CourseLocator.from_string(course_id).org
|
||||
except InvalidKeyError:
|
||||
@@ -885,6 +885,6 @@ class UpdateEmailOptInPreference(APIView):
|
||||
content_type="text/plain"
|
||||
)
|
||||
# Only check for true. All other values are False.
|
||||
email_opt_in = request.DATA['email_opt_in'].lower() == 'true'
|
||||
email_opt_in = request.data['email_opt_in'].lower() == 'true'
|
||||
update_email_opt_in(request.user, org, email_opt_in)
|
||||
return HttpResponse(status=status.HTTP_200_OK)
|
||||
|
||||
@@ -173,7 +173,7 @@ class RetrievePatchAPIView(RetrieveModelMixin, UpdateModelMixin, GenericAPIView)
|
||||
|
||||
def patch(self, request, *args, **kwargs):
|
||||
"""Checks for validation errors, then updates the model using the UpdateModelMixin."""
|
||||
field_errors = self._validate_patch(request.DATA)
|
||||
field_errors = self._validate_patch(request.data)
|
||||
if field_errors:
|
||||
return Response({'field_errors': field_errors}, status=status.HTTP_400_BAD_REQUEST)
|
||||
return self.partial_update(request, *args, **kwargs)
|
||||
|
||||
Reference in New Issue
Block a user