Loading...

XML

Word

Printable

Affected Branches:
MOODLE_31_STABLE, MOODLE_32_STABLE
Fixed Branches:
MOODLE_310_STABLE, MOODLE_39_STABLE
Pull from Repository:
git://github.com/lameze/moodle.git
Pull Main Branch:
~~MDL-56653~~_enrol_meta_edit_instance_validation
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/master...vtos:MDL-56653_enrol_meta_edit_instance_validation?expand=1
Testing Instructions:
1. The unit test added should ensure that the logic wasn't affected when tweaking performance.
2. Code analysis should reveal that the number of DB queries to fetch existing enrolment instances was reduced to 1.

The edit_instance_validation function checks for existing meta enrolment instances.
It does so in a loop.

It should ideally fetch all of them in a single query, indexed by the course being meta-linked.

Discovered while testing

MDL-56282 Cannot edit an instances Course meta enrolment

has been marked as being related by

MDL-70967 Error in test enrol/meta/tests/plugin_test.php

Participants:: Adrian Greeve, Andrew Lyons, CiBoT, Eloy Lafuente (stronk7), Marina Glancy, Matteo Scaramuccia, Simey Lameze, Vitaly Potenko

Estimated:

Not Specified

Remaining:

Logged:

7h 25m