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

System reports should be allowed to define aggregation to allow re-use of entities.

XMLWordPrintable

    • WP Sprint 2023-I1.4 (Clones)

      The system_report_table class doesn't support aggregation - this means you have to create a custom entity with aggregated columns if you need to represent something using an aggregation - duplicating significant effort.

      Ideally in a custom report you could set something like:

      if ( $column = $this->get_column('entity:column')) {
      		 
          $column->set_aggregation('sum'); 
      }

      Current locations in code that would benefit:

      1. Cohorts member count (MDL-73141)
      2. Badges issued count (MDL-77061)
      3. LTI tools usage count (MDL-78575) (this has since been re-factored away, and LTI work is frozen anyway)
      4. Reports tags (MDL-80245)
      5. Tags count (MDL-79788)

      (Adding to MDL-84719 epic because this allow for improved native aggregation performance, instead of using derived table joins/sub-selects)

            pholden Paul Holden
            danmarsden Dan Marsden
            Pedro Jordao Pedro Jordao
            Jun Pataleta Jun Pataleta
            CiBoT CiBoT
            Votes:
            2 Vote for this issue
            Watchers:
            9 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour, 19 minutes
                1h 19m

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.