Modular Extensions - HMVC version 5.2

[eluser]autoreverse[/eluser]
I like the elegance of the __autoload solution but, in addition to the PHP 4 incompatibility, autoloading is not available if using PHP from the command line. This probably means that cron jobs have to be run over HTTP rather than via the shell.

The following code added to the end of Controller.php (release 4026 - I've just started looking at the ME project today) after the Controller class declaration works in both PHP 4 and PHP 5 and should also solve the CLI problem:

Code:

/* PHP4/5/CLI autoload helper - PHP4 and CLI require modules.php config file*/
if (phpversion() >= 5 & !isset($argc))
{
    function module_autoloader($class)
    {
        if (is_file($filename = APPPATH.'libraries/'.$class.EXT))
            require_once $filename;
    }
    spl_autoload_register('module_autoloader');
}
else
{
    if (is_file(APPPATH.'config/modules'.EXT))
    {
        require(APPPATH.'config/modules'.EXT);
        foreach ($config['modules_controller_extensions'] as $class)
        {
            require_once APPPATH.'libraries/'.$class.EXT;
        }
    }
}

(note the use of spl_autoload_register to reduce the chance of name clashes with other __autoload functions)

and the corresponding config file, config/modules.php:

Code:

<?php  if ( ! defined('BASEPATH')) exit('No direct script access allowed');

/*
|--------------------------------------------------------------------------
| Controller Extensions
|--------------------------------------------------------------------------
|
| Contoller extensions to be autoloaded after instantiation of the Modular
| Extensions Controller class. Files are loaded in the order listed in the
| array - list earliest descendants first.
|
*/
$config['modules_controller_extensions']    = array('Base_Controller', 'Public_Controller');

Another approach would be to specify an alternative prefix to 'MY_' in a config file for the controller extension file.

I'd prefer to see a more elegant solution this works for me for now :-)

* Edited to more clearly show that this mod is best used for controller extensions only. Regular custom libraries should be loaded via conventional methods such autoloading.