Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-48424

SCORM popup crashes if currentorg param has ampersand in it

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Won't Do
    • 2.6.6, 3.1.3
    • None
    • SCORM
    • MOODLE_26_STABLE, MOODLE_31_STABLE

    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

          Activity

            People

              Unassigned Unassigned
              sergeyandro Sergey Vidusov
              Dan Marsden, Matteo Scaramuccia, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
              Votes:
              1 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: