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

Replace add_to_log with an event trigger - mod_feedback

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Create a feedback activity if you don't have one, with a few questions spread over multiple pages.(Set 'Record user names' to "user's name will be logged" for the feedback)
      2. As a student submit a response and make sure an entry is created in the logs.
      3. As another student, start a response, submit the first page and then logout.
      4. Log back in as the same student and finish and submit the response. Make sure a log entry is created for it as before.
      5. As admin, delete a student's response. Make sure an entry is created for the same.
      6. Visit the view.php and index.php of the feedback module. Make sure an log entry is created for each of these visits.
      7. Run unitests in mod/feedback/tests/ . Make sure they all pass.
      Show
      Create a feedback activity if you don't have one, with a few questions spread over multiple pages.(Set 'Record user names' to "user's name will be logged" for the feedback) As a student submit a response and make sure an entry is created in the logs. As another student, start a response, submit the first page and then logout. Log back in as the same student and finish and submit the response. Make sure a log entry is created for it as before. As admin, delete a student's response. Make sure an entry is created for the same. Visit the view.php and index.php of the feedback module. Make sure an log entry is created for each of these visits. Run unitests in mod/feedback/tests/ . Make sure they all pass.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull Master Branch:
      MDL-40047-master
    • Story Points:
      20
    • Sprint:
      BACKEND Sprint 5

      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/feedback (6 calls)
      /mod/feedback/complete_guest.php:213: add_to_log($course->id,'feedback', 'startcomplete', 'view.php?id='.$cm->id, $feedback->id);
      /mod/feedback/index.php:43: add_to_log($course->id, 'feedback', 'view all', $url->out(false), $course->id);
      /mod/feedback/delete_completed.php:79: add_to_log($course->id, 'feedback', 'delete', 'view.php?id='.$cm->id, $feedback->id, $cm->id);
      /mod/feedback/complete.php:269: add_to_log($course->id, 'feedback', 'submit', 'view.php?id='.$cm->id, $feedback->id, $cm->id, $userid);
      /mod/feedback/complete.php:213: add_to_log($course->id, 'feedback', 'startcomplete', 'view.php?id='.$cm->id, $feedback->id, $cm->id, $userid);
      /mod/feedback/view.php:100: add_to_log($course->id, 'feedback', 'view', 'view.php?id='.$cm->id, $feedback->id, $cm->id);

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                ankit_frenz Ankit Agarwal
                Reporter:
                abgreeve Adrian Greeve
                Peer reviewer:
                Dan Poltawski
                Integrator:
                Eloy Lafuente (stronk7)
                Tester:
                Jérôme Mouneyrac
                Participants:
                Component watchers:
                Jake Dallimore, Jun Pataleta, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  18/Nov/13