[MDL-65354] Incorrect context and name of capability moodle/course:browse - Moodle Tracker

XML

Word

Printable

Affected Branches:
MOODLE_37_STABLE
Fixed Branches:
MOODLE_37_STABLE
Pull from Repository:
git://github.com/lameze/moodle.git
Pull Master Branch:
~~MDL-65354~~-master
Pull Master Diff URL:
https://github.com/lameze/moodle/compare/bef0fe2000...MDL-65354-master
Testing Instructions:
Hide

Log in as admin.

Go to Site administration > Users > Permissions > Capability overview

Search for moodle/course:browse

Make sure it does not find that capability.

Now, search for moodle/category:viewcourselist and set the roles field to All.

Make that new capability exists and it is ALLOWED to: Authenticated user and Guest

There shouldn't be any behat and PHP Unit failures.
Show
Log in as admin. Go to Site administration > Users > Permissions > Capability overview Search for moodle/course:browse Make sure it does not find that capability. Now, search for moodle/category:viewcourselist and set the roles field to All . Make that new capability exists and it is ALLOWED to: Authenticated user and Guest There shouldn't be any behat and PHP Unit failures.

the definition capability moodle/course:browse has incorrect context (should be category) and could be renamed to better indicate what it does.

has been marked as being related by

MDL-55580 Process for deprecating a capability

is a regression caused by

MDL-10965 Add new capability to view the list of non-hidden courses

Participants:: CiBoT, Eloy Lafuente (stronk7), Gladys Basiana, Helen Foster, Marina Glancy, Mary Cooch, Mihail Geshoski, Simey Lameze

Component watchers:: Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona)

Estimated:

Remaining:

Logged:

5h 35m