Bug #51051: DataHandler::clear_cache() does not consider workspaces - TYPO3 Core - TYPO3 Forge

Actions

Bug #51051

closed

DataHandler::clear_cache() does not consider workspaces

Added by Oliver Hader over 11 years ago. Updated about 6 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Performance

Target version:

-

Start date:

2013-08-13

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

6.2

PHP Version:

5.3

Tags:

Complexity:

medium

Is Regression:

No

Sprint Focus:

Description

DataHandler::clear_cache() is called with a table and a uid that has been modified to trigger the clear cache commands. To find siblings, children, parents, etc. the pid value is directly used to find the affected pages. However, in a workspace context, the pid is always "-1" which leads to a selection of all page records of all workspaces. This amount is used to flush all defined caches by a given tag name and might result in execution time outs.

Related issues 1 (0 open — 1 closed)

Actions

#1

Updated by Gerrit Code Review over 11 years ago

Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23078

Actions

#2

Updated by Gerrit Code Review over 11 years ago

Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/23079

Actions

#3

Updated by Sascha Egerer about 11 years ago

Please add a test plan!

Actions

#4

Updated by Gerrit Code Review about 11 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23078

Actions

#5

Updated by Oliver Hader about 11 years ago

Is Regression set to No

After discussing at the T3CSH13, clearing front-end caches does not make sense.

Actions

#6

Updated by Gerrit Code Review about 11 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23078

Actions

#7

Updated by Gerrit Code Review about 11 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23078

Actions

#8

Updated by Gerrit Code Review about 11 years ago

Patch set 2 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/23079

Actions

#9

Updated by Gerrit Code Review about 11 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/24592

Actions

#10

Updated by Oliver Hader about 11 years ago

Status changed from Under Review to Resolved
% Done changed from 0 to 100

Applied in changeset 13c5bf9988ae19eb07dc499bcf4bfc5054596537.

Actions

#11

Updated by Benni Mack about 6 years ago

Status changed from Resolved to Closed

Actions

Also available in: Atom PDF