Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
E
edx-platform
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
edx-platform
Commits
cd839063
Commit
cd839063
authored
Apr 08, 2014
by
Oleg Marshev
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #3245 from edx/oleg/fix-doublequoting
Fix doublequoting. BLD-812.
parents
d66e6cbc
015bd57d
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
4 additions
and
4 deletions
+4
-4
common/lib/xmodule/xmodule/lti_module.py
+2
-2
lms/djangoapps/courseware/tests/test_lti_integration.py
+2
-2
No files found.
common/lib/xmodule/xmodule/lti_module.py
View file @
cd839063
...
...
@@ -341,8 +341,8 @@ class LTIModule(LTIFields, XModule):
"""
return
"{context}:{resource_link}:{user_id}"
.
format
(
context
=
urllib
.
quote
(
self
.
context_id
),
resource_link
=
urllib
.
quote
(
self
.
get_resource_link_id
()
),
user_id
=
urllib
.
quote
(
self
.
get_user_id
()
)
resource_link
=
self
.
get_resource_link_id
(
),
user_id
=
self
.
get_user_id
(
)
)
def
get_course
(
self
):
...
...
lms/djangoapps/courseware/tests/test_lti_integration.py
View file @
cd839063
...
...
@@ -34,8 +34,8 @@ class TestLTI(BaseTestXmodule):
sourcedId
=
"{context}:{resource_link}:{user_id}"
.
format
(
context
=
urllib
.
quote
(
context_id
),
resource_link
=
urllib
.
quote
(
resource_link_id
)
,
user_id
=
u
rllib
.
quote
(
user_id
)
resource_link
=
resource_link_id
,
user_id
=
u
ser_id
)
lis_outcome_service_url
=
'https://{host}{path}'
.
format
(
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment