Details
Description
If an "organization" attribute has an ampersand in SCO settings, it gets passed to the popup window's GET string unaltered, which destroys the functionality. Only blank white window pops up, displaying nothing because $scoes array doesn't get populated.
The following helps:
--- a/mod/scorm/view.php
|
+++ b/mod/scorm/view.php
|
@@ -99,7 +99,7 @@ if ($scorm->popup == 1) {
|
}
|
|
$PAGE->requires->data_for_js('scormplayerdata', Array('launch' => $launch,
|
- 'currentorg' => $orgidentifier,
|
+ 'currentorg' => rawurlencode($orgidentifier),
|
'sco' => $scoid,
|
'scorm' => $scorm->id,
|
'courseurl' => $courseurl,
|
There may be other places in code where currentorg does not get URL-encoded.
Attachments
Issue Links
- has a non-specific relationship to
-
MDL-38060 Warn the user when uploading a package that uses identifiers not uniquely defined in imsmanifest.xml
-
- Closed
-