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

report_loguser: Include logstore parameter for pagination

    XMLWordPrintable

    Details

    • Testing Instructions:
      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.
    • Affected Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE
    • Pull Master Branch:
      MDL-48445-master

      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

              Assignee:
              moodle.com moodle.com
              Reporter:
              jmarshall Josh Marshall
              Peer reviewer:
              Ankit Agarwal
              Integrator:
              Dan Poltawski
              Tester:
              Frédéric Massart
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/Jan/15