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

Replace add_to_log with an event trigger - mod_scorm

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Create a scorm activity with atleast 6-7 attempts from multiple students.
      2. Make sure no error is generated during the following actions and also make sure a log entry is created in the report at <yoursite>/report/log/index.php with correct details for all of the following:-
        1. Visit scorm/index.php (linked from activities block)
        2. View.php of a scorm activity
        3. All 4 reports of a scorm activity (course ► General ► SCORM ► Report)
        4. Delete attempts from each of the report that allow to do so (all except graph).
        5. Click on attempt number on any report and view the page userreport.php
        6. Upload a multi scoes scorm package and as a student traverse and attempt all scoes, an entry should be generated for each sco.
        7. When on userreport.php, click on interactions, tracks details, Learning objects
      Show
      Create a scorm activity with atleast 6-7 attempts from multiple students. Make sure no error is generated during the following actions and also make sure a log entry is created in the report at <yoursite>/report/log/index.php with correct details for all of the following:- Visit scorm/index.php (linked from activities block) View.php of a scorm activity All 4 reports of a scorm activity (course ► General ► SCORM ► Report) Delete attempts from each of the report that allow to do so (all except graph). Click on attempt number on any report and view the page userreport.php Upload a multi scoes scorm package and as a student traverse and attempt all scoes, an entry should be generated for each sco. When on userreport.php, click on interactions, tracks details, Learning objects
    • Affected Branches:
      MOODLE_25_STABLE, MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull Master Branch:
      MDL-40056-master
    • Story Points:
      40
    • Sprint:
      BACKEND Sprint 9

      Description

      Replace the add_to_log calls for the following area, with a call to the get_legacy_logdata function in the event class.

      mod/scorm (11 calls)
      /mod/scorm/loadSCO.php:159: add_to_log($course->id, 'scorm', 'launch', 'view.php?id='.$cm->id, $result, $cm->id);
      /mod/scorm/report.php:62: add_to_log($course->id, 'scorm', 'report', 'report.php?id='.$cm->id, $scorm->id, $cm->id);
      /mod/scorm/index.php:35: add_to_log($course->id, "scorm", "view all", "index.php?id=$course->id", "");
      /mod/scorm/report/objectives/report.php:53: add_to_log($course->id, 'scorm', 'delete attempts', 'report.php?id=' . $cm->id, implode(",", $attemptids), $cm->id);
      /mod/scorm/report/basic/report.php:46: add_to_log($course->id, 'scorm', 'delete attempts', 'report.php?id=' . $cm->id, implode(",", $attemptids), $cm->id);
      /mod/scorm/report/interactions/report.php:47: add_to_log($course->id, 'scorm', 'delete attempts', 'report.php?id=' . $cm->id, implode(",", $attemptids), $cm->id);
      /mod/scorm/userreport.php:70: add_to_log($course->id, 'scorm', 'userreport', 'userreport.php?id='.$cm->id, $scorm->id, $cm->id);
      /mod/scorm/player.php:160: add_to_log($course->id, 'scorm', 'view', "player.php?cm=$cm->id&scoid=$sco->id", "$scorm->id", $cm->id);
      /mod/scorm/view.php:107: add_to_log($course->id, 'scorm', 'pre-view', 'view.php?id='.$cm->id, "$scorm->id", $cm->id);
      report/userreportinteractions.php:add_to_log($course->id, 'scorm', 'userreportinteractions', 'userreportinteractions.php?id='.$cm->id, $scorm->id, $cm->id)
      report/userreporttracks.php:add_to_log($course->id, 'scorm', 'userreporttracks', 'userreporttracks.php?id='.$id, $scorm->id, $id);

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              ankit_frenz Ankit Agarwal
              Reporter:
              abgreeve Adrian Greeve
              Peer reviewer:
              Mark Nelson
              Integrator:
              Damyon Wiese
              Tester:
              David Monllaó
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Damyon Wiese, Dan Marsden, Matteo Scaramuccia, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/May/14