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

SCORM: Restorelib.php diff with fix for restoring several scorm values properly instead of restoring with default values.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 1.9.5
    • Fix Version/s: 1.9.8
    • Component/s: SCORM
    • Labels:
      None
    • Environment:
      MySql v.5.0.51a
    • Database:
      MySQL
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      Restoring a Scorm activity does not retain the following Scorm db values:

      • maxgrade
      • updatefreq
      • maxattempt
      • grademethod

      Attached is a diff file that contains a small change to the /mod/scorm/restorelib.php which resolves this issue.

      Process to reproduce issue.

      1. Create a new SCORM within a valid course. Fill values as desired/required, making sure to change the fields below as follows:

      • Under 'Other Settings', Change 'Grading Method' to any non-default value (grademethod)
      • Under 'Other Settings', change 'Maximum Grade' to any value > 0 (maxgrade)
      • Under 'Other Settings', change 'Number of Attempts' to any value > 0 (maxattempt)
      • Under 'Other Settings', change 'Auto-update Frequency' to any non-default value (updatefreq)
        2. Create a backup of the course with all Scorm information/data.
        3. Restore the backup file to a new course.
        4. Check the Scorm values for the fields mentioned in step 1.

      Expected Results:
      Restored Scorm values for Grading Method, Maximum Grade, Number of Attempts, and Auto-update Frequency equal to original backed-up scorm values.

      Actual Results:
      Restored Scorm values for Grading Method, Maximum Grade, Number of Attempts, and Auto-update Frequency are reset to default values.

        Attachments

          Activity

            People

            • Assignee:
              danmarsden Dan Marsden
              Reporter:
              mavelar Michael Avelar
              Tester:
              Nobody
              Participants:
              Component watchers:
              Damyon Wiese, Dan Marsden, Matteo Scaramuccia, Adrian Greeve, Mihail Geshoski, Peter Dias
            • Votes:
              1 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                25/Mar/10