Task #77411

Remove extbase table column cache

Added by Benni Mack over 3 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Extbase
Target version:
Start date:
2016-08-04
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The extbase table column cache "extbase_typo3dbbackend_tablecolumns"
stores all DB fields of all DB tables to detect if the TCA table contains
a field named "pid".

However, if a DB table has a TCA definition but no PID, then it fails
on so many places. No need to execute a admin_get_fields call on every
method call (which caches this information - ironically - in the database).

The cache can be removed without substitution as the superfluous
check on the DB field "pid" is removed.


Related issues

Related to TYPO3 Core - Bug #77604: Keep configuration of extbase table column cache in 7.6 Closed 2016-08-23

Associated revisions

Revision 65a9d8a7 (diff)
Added by Benni Mack over 3 years ago

[TASK] Remove extbase table column cache

The extbase table column cache "extbase_typo3dbbackend_tablecolumns"
stores all DB fields of all DB tables to detect if the TCA table contains
a field named "pid".

However, if a DB table has a TCA definition but no PID, then it fails
at so many places. No need to execute a admin_get_fields call on every
method call.
(Which caches this information - ironically - in the database by default)

The cache can be removed without substitution as the superfluous
check on the DB field "pid" is removed.

Resolves: #77411
Releases: master, 7.6
Change-Id: Ib4dca4e5f895f0e4334c26482423b833dcb773ca
Reviewed-on: https://review.typo3.org/49385
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Tested-by: Bamboo TYPO3com <>
Reviewed-by: Mathias Brodala <>
Reviewed-by: Oliver Klee <>
Reviewed-by: Jigal van Hemert <>
Tested-by: Jigal van Hemert <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

Revision ca3529c3 (diff)
Added by Benni Mack over 3 years ago

[TASK] Remove extbase table column cache

The extbase table column cache "extbase_typo3dbbackend_tablecolumns"
stores all DB fields of all DB tables to detect if the TCA table contains
a field named "pid".

However, if a DB table has a TCA definition but no PID, then it fails
at so many places. No need to execute a admin_get_fields call on every
method call.
(Which caches this information - ironically - in the database by default)

The cache can be removed without substitution as the superfluous
check on the DB field "pid" is removed.

Resolves: #77411
Releases: master, 7.6
Change-Id: Ib4dca4e5f895f0e4334c26482423b833dcb773ca
Reviewed-on: https://review.typo3.org/49396
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

Revision f19438df (diff)
Added by Christian Kuhn over 3 years ago

[BUGFIX] Keep configuration of extbase table column cache

To mitigate the possible impact of the removal of table column
cache for extensions in 7.6, the cache configuration is kept
but the backend set to NullBackend.

Change-Id: Ida1b6b27d34b1703aca148f726d5ba051faaa3d1
Resolves: #77604
Related: #77411
Releases: 7.6
Reviewed-on: https://review.typo3.org/49552
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>
Reviewed-by: Georg Ringer <>
Tested-by: Georg Ringer <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review over 3 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/49385

#2 Updated by Gerrit Code Review over 3 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/49385

#3 Updated by Gerrit Code Review over 3 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/49385

#4 Updated by Gerrit Code Review over 3 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/49385

#5 Updated by Gerrit Code Review over 3 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/49385

#6 Updated by Gerrit Code Review over 3 years ago

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

#7 Updated by Gerrit Code Review over 3 years ago

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

#8 Updated by Benni Mack over 3 years ago

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

#9 Updated by Gerrit Code Review over 3 years ago

  • Status changed from Resolved to Under Review

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

#10 Updated by Gerrit Code Review over 3 years ago

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

#11 Updated by Benni Mack over 3 years ago

  • Status changed from Under Review to Resolved

#12 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF