From 720dcb6c3ae90a358b3009d9243bbcf5aa688951 Mon Sep 17 00:00:00 2001 From: usama sadiq Date: Thu, 9 Jan 2020 15:40:55 +0500 Subject: [PATCH] Updated ExceptionViews handles Added additional 'exception' parameter to function headers. Disabled pylint warnings for missing docstring and unused arguments. --- cms/djangoapps/contentstore/views/error.py | 4 ++-- lms/djangoapps/static_template_view/views.py | 4 +++- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/cms/djangoapps/contentstore/views/error.py b/cms/djangoapps/contentstore/views/error.py index 6d884d4a21..88b90ba016 100644 --- a/cms/djangoapps/contentstore/views/error.py +++ b/cms/djangoapps/contentstore/views/error.py @@ -31,7 +31,7 @@ def jsonable_error(status=500, message="The Studio servers encountered an error" @jsonable_error(404, "Resource not found") -def not_found(request): +def not_found(request, exception): return render_to_response('error.html', {'error': '404'}) @@ -42,7 +42,7 @@ def server_error(request): @fix_crum_request @jsonable_error(404, "Resource not found") -def render_404(request): +def render_404(request, exception): return HttpResponseNotFound(render_to_string('404.html', {}, request=request)) diff --git a/lms/djangoapps/static_template_view/views.py b/lms/djangoapps/static_template_view/views.py index f8a26c9a18..d383313d5f 100644 --- a/lms/djangoapps/static_template_view/views.py +++ b/lms/djangoapps/static_template_view/views.py @@ -1,3 +1,5 @@ +# pylint: disable=missing-docstring,unused-argument + # View for semi-static templatized content. # # List of valid templates is explicitly managed for (short-term) @@ -91,7 +93,7 @@ def render_press_release(request, slug): @fix_crum_request -def render_404(request): +def render_404(request, exception): return HttpResponseNotFound(render_to_string('static_templates/404.html', {}, request=request))