Commit 03de84ec by Alexander Kryklia

Fix

parent bd6fed83
...@@ -143,7 +143,7 @@ class LTIModule(LTIFields, XModule): ...@@ -143,7 +143,7 @@ class LTIModule(LTIFields, XModule):
try: try:
lti_id, key, secret = lti_passport.split(':') lti_id, key, secret = lti_passport.split(':')
except ValueError: except ValueError:
raise Exception('Could not parse LTI passport: {0!r}. \ raise LTIError('Could not parse LTI passport: {0!r}. \
Should be "id:key:secret" string.'.format(lti_passport)) Should be "id:key:secret" string.'.format(lti_passport))
if lti_id == self.lti_id: if lti_id == self.lti_id:
client_key, client_secret = key, secret client_key, client_secret = key, secret
...@@ -155,7 +155,7 @@ class LTIModule(LTIFields, XModule): ...@@ -155,7 +155,7 @@ class LTIModule(LTIFields, XModule):
try: try:
param_name, param_value = custom_parameter.split('=', 1) param_name, param_value = custom_parameter.split('=', 1)
except ValueError: except ValueError:
raise Exception('Could not parse custom parameter: {0!r}. \ raise LTIError('Could not parse custom parameter: {0!r}. \
Should be "x=y" string.'.format(custom_parameter)) Should be "x=y" string.'.format(custom_parameter))
# LTI specs: 'custom_' should be prepended before each custom parameter # LTI specs: 'custom_' should be prepended before each custom parameter
......
...@@ -110,7 +110,7 @@ def add_correct_lti_to_course(_step): ...@@ -110,7 +110,7 @@ def add_correct_lti_to_course(_step):
section_name = chapter_name section_name = chapter_name
path = "/courses/{org}/{num}/{name}/courseware/{chapter}/{section}".format( path = "/courses/{org}/{num}/{name}/courseware/{chapter}/{section}".format(
org=course.org, org=course.org,
num=course.num, num=course.number,
name=course.display_name.replace(' ', '_'), name=course.display_name.replace(' ', '_'),
chapter=chapter_name, chapter=chapter_name,
section=section_name) section=section_name)
...@@ -137,7 +137,7 @@ def add_incorrect_lti_to_course(_step): ...@@ -137,7 +137,7 @@ def add_incorrect_lti_to_course(_step):
section_name = chapter_name section_name = chapter_name
path = "/courses/{org}/{num}/{name}/courseware/{chapter}/{section}".format( path = "/courses/{org}/{num}/{name}/courseware/{chapter}/{section}".format(
org=course.org, org=course.org,
num=course.num, num=course.number,
name=course.display_name.replace(' ', '_'), name=course.display_name.replace(' ', '_'),
chapter=chapter_name, chapter=chapter_name,
section=section_name) section=section_name)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment