From ce1b62311be83e88e6fca1f06edbea0319048d0a Mon Sep 17 00:00:00 2001 From: louyihua Date: Fri, 21 Mar 2014 23:24:28 +0800 Subject: [PATCH] Fix a url processing error in 02_html5_video.js As the code in this .js will break the url's integrity when there is already a query string inside the video source's url and cause the url to be invalid in some cases (for example, when working with a url from Windows Azure's Media Service, the appended '?' will cause the url being invalid). I modified the code by first checking whether the url has already had a query string, and if so, then use '&' instead of '?' to prevent the breaking, and it works for the url from Windows Azure's Media Service. --- .../xmodule/xmodule/js/src/video/02_html5_video.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/common/lib/xmodule/xmodule/js/src/video/02_html5_video.js b/common/lib/xmodule/xmodule/js/src/video/02_html5_video.js index 78a81d2d95..bcd1529207 100644 --- a/common/lib/xmodule/xmodule/js/src/video/02_html5_video.js +++ b/common/lib/xmodule/xmodule/js/src/video/02_html5_video.js @@ -186,16 +186,16 @@ function () { // Create HTML markup for individual sources of the HTML5