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

Scorm Auto-commit can't be disabled.

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.8.7, 2.9.1
    • 2.9.4, 3.0.2
    • SCORM
    • MOODLE_28_STABLE, MOODLE_29_STABLE
    • MOODLE_29_STABLE, MOODLE_30_STABLE
    • master_MDL-50972
    • Hide
      1. Turn on API debugging mode in scorm
      2. Add the scorm package attached to MDL-37401
      3. Make sure auto commit is turned on in scorm package settings
      4. Open the package and navigate to a SCO.
      5. Stop navigation when you see "LMSSetValue" as the last function call in the debugger.
      6. Wait for 60 seconds, "LMSCommit" should be called automatically.
      1. Test 2
        Same instructions as above but turn auto-commit setting "off" within the package
        Wait for 60 seconds, make sure "LMSCommit" is not called automatically.
      Show
      Turn on API debugging mode in scorm Add the scorm package attached to MDL-37401 Make sure auto commit is turned on in scorm package settings Open the package and navigate to a SCO. Stop navigation when you see "LMSSetValue" as the last function call in the debugger. Wait for 60 seconds, "LMSCommit" should be called automatically. Test 2 Same instructions as above but turn auto-commit setting "off" within the package Wait for 60 seconds, make sure "LMSCommit" is not called automatically.

    Description

      Even if the Auto-commit option is unchecked in both general and module's Scorm configuration, Moodle keeps auto-commiting the sco values.

      It looks like the autocommit value is set to a string "0" instead of 0 or false, confirming the test in scorm_13.js l.748

      I have to override $scorm->autocommit value in scorm_13.php l.49 to disable the Auto-commit's timeout :

      if ($scorm->autocommit == 0) $scorm->autocommit = false ;

      Attachments

        Issue Links

          Activity

            People

              danmarsden Dan Marsden
              leeloyd Loïc POZAS
              Dan Poltawski Dan Poltawski
              Andrew Lyons Andrew Lyons
              Adrian Greeve Adrian Greeve
              Dan Marsden, Matteo Scaramuccia, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Sujith Haridasan
              Votes:
              2 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                11/Jan/16