[TYPO3-core] RFC: #15586: Unecessary require_once in t3lib_cache_Factory

Helmut Hummel helmut at typo3.org
Sat Aug 28 13:28:43 CEST 2010


Hi,

this is a SVN patch request:

Branches: trunk

Reference:
http://bugs.typo3.org/view.php?id=15586

Type:
Feature/ Cleanup

Problem:
Currently when configuring the available cacheBackends and
cacheFrontends a path to the class files has to be specified. This file
will then be included by require_once in the t3lib_cache_Factory.
This is not necessary because the classes are registered in the
autoloader anyway.

Solution:
Only specify a classname and rely on the autoloader to automatically
require the needed class.

Note:
For backwards compatibility, I left the current behaviour in the
t3lib_cache_Factory, but log to the deprecation log, if a file is specified.
All caching framework test run fine after this change.

How to test:
Turn on the caching famewok.
Apply the patch.
Observe that everything still works like before.

As an addition, the changes in config_default could be reverted and
still everything will work, besides the inclusion of the files will be
logged to the deprecation log.


Regards Helmut
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 15586.diff
Type: text/x-patch
Size: 6827 bytes
Desc: not available
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20100828/64d54cc1/attachment.bin>


More information about the TYPO3-team-core mailing list