Loading...

XML

Word

Printable

Type: Improvement
Resolution: Fixed
Priority: Minor
Fix Version/s: 5.0
Affects Version/s: 4.1, 5.0
Component/s: Report builder
Labels:

Affected Branches:
MOODLE_401_STABLE, MOODLE_500_STABLE
Fixed Branches:
MOODLE_500_STABLE
Pull from Repository:
https://github.com/paulholden/moodle.git
Pull Main Branch:
~~MDL-74488~~
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/main...paulholden:MDL-74488
Testing Instructions:
1. Covered by automated tests (PHPUnit)
Pre-check results:
Hide

Code verified against automated checks.

Checked ~~MDL-74488~~ using repository: https://github.com/paulholden/moodle.git

main (0 errors / 0 warnings) [branch: MDL-74488 | CI Job]

_{More information about this report}

Built on: Sun Jan 19 04:20:24 PM UTC 2025
Show
Code verified against automated checks. Checked MDL-74488 using repository: https://github.com/paulholden/moodle.git main (0 errors / 0 warnings) [branch: MDL-74488 | CI Job ] More information about this report Built on: Sun Jan 19 04:20:24 PM UTC 2025
Automated test results:
Hide

Launching automatic jobs for branch ~~MDL-74488~~

https://ci.moodle.org/view/Testing/job/DEV.02%20-%20Developer-requested%20PHPUnit/17925/ PHPUnit (sqlsrv)

https://ci.moodle.org/view/Testing/job/DEV.01%20-%20Developer-requested%20Behat/61102/ Behat (NonJS - boost and classic)

https://ci.moodle.org/view/Testing/job/DEV.01%20-%20Developer-requested%20Behat/61103/ Behat (Firefox - boost)

https://ci.moodle.org/view/Testing/job/DEV.01%20-%20Developer-requested%20Behat/61104/ Behat (Firefox - classic)

https://ci.moodle.org/view/Testing/job/DEV.01%20-%20Developer-requested%20Behat/61105/ App tests (stable app version)

Built on: Tue Jan 14 04:30:43 PM UTC 2025
Show
Launching automatic jobs for branch MDL-74488 https://ci.moodle.org/view/Testing/job/DEV.02%20-%20Developer-requested%20PHPUnit/17925/ PHPUnit (sqlsrv) https://ci.moodle.org/view/Testing/job/DEV.01%20-%20Developer-requested%20Behat/61102/ Behat (NonJS - boost and classic) https://ci.moodle.org/view/Testing/job/DEV.01%20-%20Developer-requested%20Behat/61103/ Behat (Firefox - boost) https://ci.moodle.org/view/Testing/job/DEV.01%20-%20Developer-requested%20Behat/61104/ Behat (Firefox - classic) https://ci.moodle.org/view/Testing/job/DEV.01%20-%20Developer-requested%20Behat/61105/ App tests (stable app version) Built on: Tue Jan 14 04:30:43 PM UTC 2025

This issue is about adding an API mechanism for callers to retrieve the number of rows in a report.

With this new API in place, we could add the pre-existing cohorts count in the new report builder system report:

https://github.com/moodle/moodle/commit/b0802f49a6cec38b9dff0624a7ae5011f1efe245#diff-c2025bc44161d1320b228d1483142fe5b0c704bd6e4692fd5ffbd078691bab4bL90-L99

has a non-specific relationship to

MDL-73141 Convert cohort listing reports to Report builder

Closed

MDL-83718 Improve reportbuilder table performance by combining count + select

Closed

has to be done before

MDL-84439 Final removal of deprecated schedule report count method

Open

Assignee:: Paul Holden

Reporter:: David Carrillo

Peer reviewer:: Carlos Castillo

Integrator:: David Carrillo

Tester:: CiBoT

Participants:: Carlos Castillo, CiBoT, David Carrillo, Jun Pataleta, Paul Holden, Sara Arjona (@sarjona)

Votes:: 0 Vote for this issue

Watchers:: 5 Start watching this issue

Created:: 12/Apr/22 9:26 PM

Updated:: 07/Feb/25 12:31 AM

Resolved:: 07/Feb/25 12:21 AM

Estimated:

Remaining:

Logged:

3h 30m

Details

Description

Attachments

Issue Links

Activity

People

Dates

Time Tracking

Clockify