[TYPO3-core] Feature #9633: Implement autoloading for TYPO3

Steffen Kamper info at sk-typo3.de
Thu May 7 10:17:06 CEST 2009


Hi olly,

i tested the patch and it works as far as i can see.
One remark:

+
+	if (!defined('PATH_tslib')) {
+		if (@is_dir(PATH_site . TYPO3_mainDir . 'sysext/cms/tslib/')) {
+			define('PATH_tslib', PATH_site . TYPO3_mainDir . 'sysext/cms/tslib/');
+		} elseif (@is_dir(PATH_site . 'tslib/')) {
+			define('PATH_tslib', PATH_site . 'tslib/');
+		}
+	}

shouldn't it the other way arround? PATH_site . TYPO3_mainDir . 
'sysext/cms/tslib/' will be there all the time, so first check for 
symlink tslib.

I don't find all the removals of require_once in the patch, only a few.

So beside that +1 by reading and testing.

vg Steffen

Oliver Hader schrieb:
> Hi,
> 
> Sebastian KurfŸürst schrieb:
>> This is an SVN patch request.
>>
>> Type: Feature
>>
>> BT Reference: http://bugs.typo3.org/view.php?id=9633
> 
> The extdeveval patch is not part of this RFC and gets handled in the
> accordant extension.
> 
> The autoloading stuff was split up in to files:
> * integrate the autoloading functionality (..._autoload.patch)
> * enable the autoloader basically (..._enable.patch)
> 
> olly
> 


More information about the TYPO3-team-core mailing list