[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