Loading...

XML

Word

Printable

Affected Branches:
MOODLE_34_STABLE
Fixed Branches:
MOODLE_34_STABLE
Epic Link:
Merge Participants and Enrolled Users
Pull from Repository:
git://github.com/markn86/moodle.git
Pull Main Branch:
~~MDL-59977~~_master
Pull Main Diff URL:
https://github.com/markn86/moodle/compare/master...MDL-59977_master
Testing Instructions:
Hide

Do a grep for ":viewparticipants" and confirm there are no occurrences in the code where it should be calling course_can_view_participants() instead. Ignore the case in message/classes/api.php, course/lib.php, PHPUnit and Behat tests.

Ensure Behat is passing.
Show
Do a grep for ":viewparticipants" and confirm there are no occurrences in the code where it should be calling course_can_view_participants() instead. Ignore the case in message/classes/api.php, course/lib.php, PHPUnit and Behat tests. Ensure Behat is passing.

We now allow the following capabilities in order to check the participant page -

'moodle/course:viewparticipants' (or 'moodle/site:viewparticipants' depending on context).
'moodle/course:enrolreview'

We need to go through Moodle and convert the direct check on 'viewparticipants' with a function that uses or logic.

Discovered while testing

MDL-59832 Remove enrolled users link if user cannot see participants

Participants:: CiBoT, Damyon Wiese, Jun Pataleta, Marina Glancy, Mark Nelson