[MDL-28421] Adjust class admin_setting_configmultiselect_modules to include a default - Moodle Tracker

XML

Word

Printable

Details

Type: Improvement
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 2.1
Fix Version/s: 2.1.1
Component/s: Administration
Labels:
None

Testing Instructions:

Hide

How to test:

Just make sure that the Default allowed modules setting on Site administration / 〉 Security / 〉 Module security still works. That is, that there are no regressions.

Show
How to test: Just make sure that the Default allowed modules setting on Site administration / 〉 Security / 〉 Module security still works. That is, that there are no regressions.
Affected Branches:
MOODLE_21_STABLE
Fixed Branches:
MOODLE_21_STABLE
Pull from Repository:
https://github.com/Beedell/moodle/
Pull Master Branch:
wip-~~MDL-28421~~-master
Pull Master Diff URL:
https://github.com/Beedell/moodle/compare/master...wip-MDL-28421-master

Description

In ~~MDL-27123~~ a new admin class admin_setting_configmultiselect_modules was created that used lazy loading to help improve performance.

Unfortunately I did not include the use of a default value for this class as it was not needed at the time. Of course I now do have a need to use this class with a default value. Yes it is my own fault I know!

Attachments

Activity

People

Assignee:: John Beedell

Reporter:: John Beedell

Integrator:: Petr Skoda

Tester:: Rajesh Taneja

Participants:: John Beedell, Petr Skoda, Rajesh Taneja, Sam Hemelryk

Component watchers:: Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze

Votes:: 0 Vote for this issue

Watchers:: 1 Start watching this issue

Dates

Created:: 22/Jul/11 7:39 AM

Updated:: 27/Jul/11 1:51 AM

Resolved:: 27/Jul/11 1:51 AM

Fix Release Date:: 1/Aug/11