Project

General

Profile

Actions

Feature #23473

closed

Unecessary require_once in t3lib_cache_Factory

Added by Helmut Hummel about 14 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Should have
Category:
Caching
Target version:
-
Start date:
2010-08-28
Due date:
% Done:

0%

Estimated time:
PHP Version:
5.2
Tags:
Complexity:
Sprint Focus:

Description

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.

(issue imported from #M15586)


Files

15586.diff (6.67 KB) 15586.diff Administrator Admin, 2010-08-28 13:25

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #21994: Defining new cache backend in extension must use absolut pathClosed2010-01-19

Actions
Actions #1

Updated by Steffen Kamper about 14 years ago

committed to trunk rev 8718

Actions #2

Updated by Susanne Moog over 13 years ago

  • Target version deleted (4.5.0)
Actions

Also available in: Atom PDF