From 41c28a8b9db69ac544a46be055fecdbec080c543 Mon Sep 17 00:00:00 2001 From: Chris-Thornton-Harvard Date: Thu, 8 Mar 2018 10:05:06 -0500 Subject: [PATCH] PR touchups --- canvas_manage_course/urls.py | 11 +++++++---- canvas_manage_course/views.py | 3 +-- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/canvas_manage_course/urls.py b/canvas_manage_course/urls.py index 713885d..074ddaf 100644 --- a/canvas_manage_course/urls.py +++ b/canvas_manage_course/urls.py @@ -17,12 +17,15 @@ url(r'^tool_config$', views.tool_config, name='tool_config'), ] -try: - if settings.DEBUG: +# Import the debug toolbar and handle any namespace issues that may occur +# ie: 'djdt' is not a registered namespace +# https://github.com/jazzband/django-debug-toolbar/issues/529 +if settings.DEBUG: + try: import debug_toolbar urlpatterns += [ url(r'^__debug__/', include(debug_toolbar.urls)), ] -except: - pass + except: + pass diff --git a/canvas_manage_course/views.py b/canvas_manage_course/views.py index 22feafe..a223a8a 100644 --- a/canvas_manage_course/views.py +++ b/canvas_manage_course/views.py @@ -21,8 +21,7 @@ @require_http_methods(['GET']) def tool_config(request): - host = 'https://' + request.get_host() - url = host + reverse('lti_launch') + url = "https://{}{}".format(request.get_host(), reverse('lti_launch')) title = 'Manage Course' lti_tool_config = ToolConfig(