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

"General Exception" scorm_12.js LMSCommit function

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.9, 1.9.10, 2.0
    • Fix Version/s: 2.0.2
    • Component/s: SCORM
    • Labels:
      None
    • Environment:
      n/a
    • Database:
      Any
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      Whilst creating some SCORM 1.2 objects in CourseLab and importing them into Moodle (1.9.9+ and 2.0) I keep getting an alert message when moving between slides:

      ScormApi2004.LMSCommit:
      General exception

      Investigating the call to "mod/scorm/datamodel.php" results in "true\n0" however I found the following lines of code:

      scorm_12.js
      433: result = ('true' == result) ? 'true' : 'false';
      434: errorCode = (result ==' true')? '0' : '101';

      which results in errorCode always being set to '101' (general exception), causing the alert message. (note proceeding the space in ' true').

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              danmarsden Dan Marsden added a comment -

              thanks for the report! - good spotting, that's a definite typo there - will try to get that sorted soon!

              Show
              danmarsden Dan Marsden added a comment - thanks for the report! - good spotting, that's a definite typo there - will try to get that sorted soon!
              Hide
              mudrd8mz David Mudrák added a comment -

              Tested, closing. Thanks.

              Show
              mudrd8mz David Mudrák added a comment - Tested, closing. Thanks.

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    21/Feb/11