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

SCORM JSON.parse not supported by IE7

    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.

        Gliffy Diagrams

          Attachments

            Activity

              People

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

                Dates

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