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

Enable usage of external log storage drivers in "report/outline" module

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.5.15, 3.7.9, 3.8.6, 3.9.3, 3.10
    • Fix Version/s: None
    • Component/s: Reports
    • Labels:
      None
    • Affected Branches:
      MOODLE_310_STABLE, MOODLE_35_STABLE, MOODLE_37_STABLE, MOODLE_38_STABLE, MOODLE_39_STABLE

      Description

      The "report/outline" module currently checks whether legacy or internal log are enabled and refuses to proceed otherwise. Whether there is some technical reason to do so or not, it would be swell if external log support can be included.

      Relevant section of report/outline/index.php:

      // If no legacy and no internal log then don't proceed.
              if (!$uselegacyreader && !$useinternalreader) {
                  echo $OUTPUT->box_start('generalbox', 'notice');
                  echo $OUTPUT->notification(get_string('nologreaderenabled', 'report_outline'));
                  echo $OUTPUT->box_end();
                  echo $OUTPUT->footer();
                  die();
              }

       

      Following sections handle legacy and internal drivers differently as well. As far as I know, external logging is supposed to use the same table structure as internal logging, so this might be an entry point to tackling this.

       

      To reproduce:
      In Website-Administration -> Plugins -> Logging, disable all logstores but external, then go to Course Settings -> Reports -> Activity report.

      Request should fail with error message "No log reader enabled".

        Attachments

          Activity

            People

            Assignee:
            Unassigned Unassigned
            Reporter:
            tsbartel Thorsten Bartel
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
            Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated: