L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MDL-14637 "allow activity modules to add sub menus of settings pages to admin menu" and MDL-14565 "allow quiz reports to add configuration settings to the admin tree" added a mechanism to admin/settings/plugins.php to allow modules to include a file settingstree.php instead of settings.php which will allow for more flexibility of what a module can add to the admin settings menu.

Browse Source 
Merged from 1.9

Added a comment to the if branch that includes config.html that this branch needs to be deleted as soon as quiz/config.html has been migrated to settingstree.php which is MDL-7010
This commit is contained in:
jamiesensei
2008-05-09 06:12:57 +00:00
parent 63a1749458
commit c37415a08d
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
admin/settings/plugins.php
+5 -1
View File
@@ -20,7 +20,9 @@ if ($hassiteconfig) {
foreach ($modulebyname as $strmodulename=>$module) {
$modulename = $module->name;
if (file_exists($CFG->dirroot.'/mod/'.$modulename.'/settings.php')) {
if (file_exists($CFG->dirroot.'/mod/'.$modulename.'/settingstree.php')) {
include($CFG->dirroot.'/mod/'.$modulename.'/settingstree.php');
} else if (file_exists($CFG->dirroot.'/mod/'.$modulename.'/settings.php')) {
// do not show disabled modules in tree, keep only settings link on manage page
$settings = new admin_settingpage('modsetting'.$modulename, $strmodulename, 'moodle/site:config', !$module->visible);
if ($ADMIN->fulltree) {
@@ -28,6 +30,8 @@ if ($hassiteconfig) {
}
$ADMIN->add('modsettings', $settings);
} else if (file_exists($CFG->dirroot.'/mod/'.$modulename.'/config.html')) {
//TODO this branch should be deleted as soon as quiz/config.html has been
//migrated to settings.php
$ADMIN->add('modsettings', new admin_externalpage('modsetting'.$modulename, $strmodulename, "$CFG->wwwroot/$CFG->admin/module.php?module=$modulename", 'moodle/site:config', !$module->visible));
}
}