Use spl_autoload_register instead of autoload

[kmeirlaen]

autoload can step on toes if any other code tries to register an auto-loader.
Instead, we should be using http://ca.php.net/function.spl_autoload_register

By Ryan Peters on Nov 03, 2008 @ 07:49pm *
http://qcodo.com/downloads/item.php/213 may be useful

[cdhamm]

I ran into this issue when trying to convert the ORM to a plugin based system. As a side affect, my code would fix this issue, see:

http://trac.qcu.be/projects/qcubed/ticket/206

[alex94040]

Thanks a bunch for your contribution! Could you be so kind and create an SVN .patch file, so that we're able to review your work easily?

[basilieus]

I've made one... it doesn't change much, and I'm not sure if this will effect how we do plugins etc, but this is a simple direct approach that has worked for me. see below:

[VexedPanda]

Are you thinking we could just insist that every plugin define their own autoloader to ensure that the app knows about plugin classes?
We still run into the problem of setting those autoloaders without doing a recursive search of plugin folders, so I think the proposed ini/xml files parsed at design time is the best approach we've come up with.

That said, this change should make QCubed much more viable for inclusion in other applications / frameworks. Thank you. :)

[basilieus]

Vex: I'm not sure what I was thinking about plugins.

[basilieus]

Also, I might create a custom AutoLoader? class that we can expound upon and make things a bit easier and more customizable.

The patch above is a quick easy fix.

[marcosdsanchez]

I agree with Basilieus on creating a custom Autoloader class.

this is a quick draft of how it could be:

<?php

abstract class QAutoloadBase extends QBaseClass{

       
        public static function Autoload($strClassName) {
            if (array_key_exists(strtolower($strClassName), QApplication::$ClassFile)) {
                require(QApplication::$ClassFile[strtolower($strClassName)]);
                return true;
            }
        }

        public static function RegisterAutoload(){
            spl_autoload_register(array('QAutoloadBase','Autoload'));
        }

}
?>

[VexedPanda]

I wonder if we can take advice from anything in here:
http://devzone.zend.com/article/4525-Developing-a-Comprehensive-Autoloader

[VexedPanda]

I vote we commit basilieus's patch to 1.1, it looks clean to me, and solves the problem.

[alex94040]

I think this actually has been checked in already - at least, prepend.inc.php already has an spl_autoload_register() call.

[VexedPanda]

Cool. Sholud we close this ticket then, or is there more here we should be looking at?

[MikeHostetler]

Closing this ticket.