[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