Modular Separation - PHP5 (Modules)

[eluser]Unknown[/eluser]
[quote author="Phil Sturgeon" date="1278597037"]Whenever you see the word "any" you have to expect there to be exceptions. "any application" was referring to any normal MVC app with helpers and libraries, and extended core libraries are obviously not part of run-of-the-mill application.

MY_Controller is loaded so early on that it would be a mess to hack this in, and I can't see why or how it would be useful. You could use my Base Controller code to make a different base controller for each module if you like, but this is something I have never needed to do.

I usually move shared module logic into a library and load that in the constructor of each controller as I have never seen a module with more than 4 or 5 controllers.[/quote]

I tried to use your __autoload() method of loading a base controller for just the backend, however I didn't want this function to be in the config file, I don't think that's very clean. So I tried to put it somewhere else, but I just couldn't get it to work. When I disabled MS it suddenly worked, so I found out you use

Code:

spl_autoload_register('Modules::autoload');

This overrides __autoload() if you define it afterwards, took me a while to find that one
Anyway, In Modules::autoload, only files from 'library/' are autoloaded. Isn't this supposed to be 'core/' in CI2? I changed it to

Code:

if (is_file($location = APPPATH . 'core/' . $class . EXT))

(line 164)
and now I have a base_controller in /application/core/ which will be loaded automatically if needed I'm a happy man!

So can you give your opinion please on this change, maybe it's really stupid, I don't know, but for now it works like a charm.

Thanks for all your good work for the community!