From 0dd8dabf13ed919b9fbc83a630c139d00d3050cb Mon Sep 17 00:00:00 2001 From: Arthur Barrett Date: Thu, 16 May 2013 13:18:36 -0400 Subject: [PATCH] validation errors should return 400 status code instead of 500. --- lms/djangoapps/notes/api.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lms/djangoapps/notes/api.py b/lms/djangoapps/notes/api.py index de23f78819..1162a144c0 100644 --- a/lms/djangoapps/notes/api.py +++ b/lms/djangoapps/notes/api.py @@ -136,7 +136,7 @@ def create(request, course_id): note.clean(request.body) except ValidationError as e: log.debug(e) - return ApiResponse(http_response=HttpResponse('', status=500), data=None) + return ApiResponse(http_response=HttpResponse('', status=400), data=None) note.save() response = HttpResponse('', status=303) @@ -176,7 +176,7 @@ def update(request, course_id, note_id): note.clean(request.body) except ValidationError as e: log.debug(e) - return ApiResponse(http_response=HttpResponse('', status=500), data=None) + return ApiResponse(http_response=HttpResponse('', status=400), data=None) note.save()