PR touchups

Harvard-University-iCommons · Mar 8, 2018 · 41c28a8 · 41c28a8
1 parent 96d3d81
commit 41c28a8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git 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
@@ -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(