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

Replace add_to_log call for page viewed in /report/completion/user.php

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Enable standard logstore
      2. Enable completion at course and site level
      3. Setup some completion criteria at My home ► My courses ► Miscellaneous ► course ► Course completion
      4. Goto course > reports > course completion
      5. Goto the url report/completion/user.php?id=someenrolleduserid&course=courseid
        (Errors on the page is unrelated - they are being resolved in MDL-40564)
      6. Goto log reports, select standard log store and view the logs for those two actions.
      7. click the link under "eventname" and make sure the link is not pointing to an error page.
      8. Run unittests in report/completion and make sure they pass
      Show
      Enable standard logstore Enable completion at course and site level Setup some completion criteria at My home ► My courses ► Miscellaneous ► course ► Course completion Goto course > reports > course completion Goto the url report/completion/user.php?id=someenrolleduserid&course=courseid (Errors on the page is unrelated - they are being resolved in MDL-40564 ) Goto log reports, select standard log store and view the logs for those two actions. click the link under "eventname" and make sure the link is not pointing to an error page. Run unittests in report/completion and make sure they pass
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull Master Branch:
      MDL-41123-master
    • Story Points:
      8
    • Sprint:
      BACKEND Sprint 12

      Description

      /report/completion/user.php:53: add_to_log($course->id, 'course', 'report completion', "report/completion/user.php?id=$user->id&course=$course->id", $course->id);

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              ankit_frenz Ankit Agarwal
              Reporter:
              rajeshtaneja Rajesh Taneja
              Peer reviewer:
              Adrian Greeve
              Integrator:
              Dan Poltawski
              Tester:
              Michael de Raadt
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, 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:
              2 Start watching this issue

                Dates

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