Task #79441

Change visibility for Internal caching arrays

Added by Thomas Hohn almost 4 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
DataHandler aka TCEmain
Target version:
Start date:
2017-01-24
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
Complexity:
Sprint Focus:

Description

The are a list of variables marked as internal caching array in the Datahandler.

It would make sense to declare these a private and/or in some cases move these
into runtime caches and thus change the visibility

The variables are:
  • public $recUpdateAccessCache = [];
  • public $recInsertAccessCache = [];
  • public $isRecordInWebMount_Cache = [];
  • public $isInWebMount_Cache = [];
  • public $cachedTSconfig = [];
  • public $pageCache = [];
  • public $checkWorkspaceCache = [];
#1

Updated by Thomas Hohn almost 4 years ago

  • Description updated (diff)
#2

Updated by Thomas Hohn almost 4 years ago

  • Subject changed from Deprecate Internal caching arrays to Change scope for Internal caching arrays
#3

Updated by Gerrit Code Review almost 4 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51411

#4

Updated by Gerrit Code Review almost 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51411

#5

Updated by Thomas Hohn almost 4 years ago

  • Subject changed from Change scope for Internal caching arrays to Change visibility for Internal caching arrays
  • Description updated (diff)
#6

Updated by Gerrit Code Review almost 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51411

#7

Updated by Gerrit Code Review almost 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51411

#8

Updated by Gerrit Code Review almost 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51411

#9

Updated by Thomas Hohn almost 4 years ago

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

Updated by Benni Mack about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF