Split settings.php using includes
Issue #216
resolved
settings.php is getting too big (in my opinion).
I would suggest splitting out each sub page (as viewed from Moodle admin) into an incude fie.
Settings php would then contain only the constructor, array definitions etc.
Below that we could then do something like:
- require_once($CFG->dirroot.'/theme/adaptable/settings/colors.php');
- require_once($CFG->dirroot.'/theme/adaptable/settings/fonts.php');
- require_once($CFG->dirroot.'/theme/adaptable/settings/buttons.php');
Comments (4)
-
-
reporter -
assigned issue to
- marked as enhancement
-
assigned issue to
-
reporter - changed status to resolved
Fix
#216splitting settings into separate php files→ <<cset e3fd99eb4944>>
-
reporter Fix
#216splitting out array definitions into a separate file→ <<cset 97b256411875>>
- Log in to comment
Yes, I totally agree. The settings are becoming too complex and even some should be moved to be more consistent.
Last time I counted the number of setting found almost 300.