Split settings.php using includes

Issue #216 resolved
Jeremy Hopkins created an issue

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)

  1. Info 3bits

    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.

  2. Log in to comment