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

report_loguser: Include logstore parameter for pagination

    XMLWordPrintable

Details

    • MOODLE_27_STABLE, MOODLE_28_STABLE
    • MOODLE_27_STABLE, MOODLE_28_STABLE
    • MDL-48445-master
    • Hide

      1. View the report/log/user.php (can be accessed from user profile > activity reports > all logs) activity log for a user that has activities in both logstore_standard and a non-standard logstore (e.g. logstore_legacy), where the logs span multiple pages
      2. Change the dropdown to the alternate logstore.
      3. Select 'Page 2'.
      4. Make sure it works as expected.
      5. Repeat with another log store.

      Show
      1. View the report/log/user.php (can be accessed from user profile > activity reports > all logs) activity log for a user that has activities in both logstore_standard and a non-standard logstore (e.g. logstore_legacy), where the logs span multiple pages 2. Change the dropdown to the alternate logstore. 3. Select 'Page 2'. 4. Make sure it works as expected. 5. Repeat with another log store.

    Description

      The pagination links are built from the $PAGE->url which does not include a logstore parameter. If a site has been upgraded from the legacy logstore only the first page of logs are returned without modifying the URL manually.

      Steps to replicate:
      1. View the report/log/user.php (can be accessed from user profile > activity reports > all logs) activity log for a user that has activities in both logstore_standard and a non-standard logstore (e.g. logstore_legacy), where the logs span multiple pages
      2. Change the dropdown to the alternate logstore.
      3. Select 'Page 2'.
      4. Error 'Nothing to display'

      Attachments

        Issue Links

          Activity

            People

              moodle.com Moodle HQ
              jmarshall Josh Marshall
              Ankit Agarwal Ankit Agarwal
              Dan Poltawski Dan Poltawski
              Frédéric Massart Frédéric Massart
              David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                12/Jan/15