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

              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

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