-
Improvement
-
Resolution: Fixed
-
Minor
-
4.3
-
MOODLE_403_STABLE
-
MOODLE_403_STABLE
-
This was discussed during implementation of the initial "default column sorting" of a report source in MDL-75525 here, but currently the invalid column is silently discarded
This is a problem because it can introduce subtle bugs if the developer thinks their report is being sorted by default (and writes tests with that assumption), but because of a typo in the column name they end up with random sorting test failures
This is exactly what happened when I was looking at MDL-78755. Let's be consistent and a bit stricter - throw a \coding_exception and fail early so the developer knows to fix their report source
- Discovered while testing
-
MDL-78755 Course participants report source should show active enrolments by default
- Closed
- has a non-specific relationship to
-
MDL-75525 Allow to easily specify default sortorder in reportbuilder datasources
- Closed
- has been marked as being related by
-
MDL-76295 Implement default report sorting in all report sources
- Closed