Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #23169 from edx/ziafazal/ENT-2597

Browse Source 
ENT-2597: Use encodeURIComponent instead of encodeURI
This commit is contained in:
Zia Fazal
2020-02-24 19:46:06 +05:00
committed by GitHub
parent bfb374c941 eeb390862c
commit 8fbe6e6fa9
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lms/static/js/spec/student_account/multiple_enterprise_spec.js
View File

@@ -9,7 +9,7 @@ define([
describe('MultipleEnterpriseInterface', function() {
var LEARNER_URL = '/enterprise/api/v1/enterprise-learner/?username=test-learner',
NEXT_URL = '/dashboard',
REDIRECT_URL = '/enterprise/select/active/?success_url=/dashboard';
REDIRECT_URL = '/enterprise/select/active/?success_url=%2Fdashboard';
beforeEach(function() {
// Mock the redirect call

2
lms/static/js/student_account/multiple_enterprise.js
View File

@@ -18,7 +18,7 @@
* @param {string} nextUrl The URL to redirect to after multiple enterprise selection.
*/
check: function(nextUrl) {
var redirectUrl = this.urls.multipleEnterpriseUrl + encodeURI(nextUrl);
var redirectUrl = this.urls.multipleEnterpriseUrl + encodeURIComponent(nextUrl);
var username = Utils.userFromEdxUserCookie().username;
var next = nextUrl || '/';
$.ajax({