Project

General

Profile

Actions

Bug #63808

closed

DataHandler::clear_cacheCmd() does not handle comma separated cacheTags

Added by Michiel Roos over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2014-12-12
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
5.4
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

The code:

    // flush cache by tag
    if (GeneralUtility::isFirstPartOfStr(strtolower($cacheCmd), 'cachetag:')) {
        $cacheTag = substr($cacheCmd, 9);
        $tagsToFlush[] = $cacheTag;
    }

Does not seem to properly handle a comma separated list of cacheTags: TCEMAIN.clearCacheCmd = cacheTag:firstTag,cacheTag:secondTag

Actions #1

Updated by Markus Klein over 9 years ago

  • Status changed from New to Needs Feedback

clear_cacheCmd() only accepts a single command.

The command are collected here:

\TYPO3\CMS\Core\DataHandling\DataHandler::processClearCacheQueue

// Clear cache for pages entered in TSconfig:
if (!empty($TSConfig['clearCacheCmd'])) {
    $commands = GeneralUtility::trimExplode(',', $TSConfig['clearCacheCmd'], TRUE);
    $clearCacheCommands = array_unique($commands);
    unset($commands);
}

and executed here:

\TYPO3\CMS\Core\DataHandling\DataHandler::processClearCacheQueue

// Execute collected clear cache commands from page TSConfig
foreach ($clearCacheCommands as $command) {
    $this->clear_cacheCmd($command);
}

So this should be correct

Actions #2

Updated by Michiel Roos over 9 years ago

Ah, ok. So this one can be closed then.

Actions #3

Updated by Markus Klein over 9 years ago

  • Status changed from Needs Feedback to Closed
Actions

Also available in: Atom PDF