Bug #51051

DataHandler::clear_cache() does not consider workspaces

Added by Oliver Hader almost 8 years ago. Updated over 2 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

Related to TYPO3 Core - Feature #32996: TCEMAIN.clearCache_pageGrandParent for recordsClosed2012-01-05

Actions
#1

Updated by Gerrit Code Review almost 8 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

#2

Updated by Gerrit Code Review almost 8 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

#3

Updated by Sascha Egerer over 7 years ago

Please add a test plan!

#4

Updated by Gerrit Code Review over 7 years ago

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

#5

Updated by Oliver Hader over 7 years ago

  • Is Regression set to No

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

#6

Updated by Gerrit Code Review over 7 years ago

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

#7

Updated by Gerrit Code Review over 7 years ago

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

#8

Updated by Gerrit Code Review over 7 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

#9

Updated by Gerrit Code Review over 7 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

#10

Updated by Oliver Hader over 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#11

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF