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

SCORM JSON.parse not supported by IE7

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4.3, 2.5
    • Fix Version/s: 2.4.4
    • Component/s: SCORM
    • Testing Instructions:
      Hide
      1. Add a SCORM package to a course
      2. View the SCORM package
        Expected outcome:
      • SCORM package displays correctly

      Needs checking with IE7 as well as officially supported browsers.

      Show
      Add a SCORM package to a course View the SCORM package Expected outcome: SCORM package displays correctly Needs checking with IE7 as well as officially supported browsers.
    • Affected Branches:
      MOODLE_24_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:
      MDL-39219_scorm_ie7

      Description

      I am fully aware that IE7 is no longer supported by Moodle.

      However, simply changing the 3 occurrences, in mod/scorm/module.js, of:

      JSON.parse

      to:

      Y.JSON.parse

      Changes the SCORM module from not supporting IE7 at all, to fully working (and continuing to work in all other browsers).

      I'll understand if this is rejected, but as I've just been asked to fix this for a client (and it is a very small change), I thought I'd share the patch here as well.

        Attachments

          Activity

            People

            • Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                13/May/13