Task #87936

Remove TCA for sys_history

Added by Benni Mack 7 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Code Cleanup
Start date:
2019-03-16
Due date:
% Done:

100%

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

Description

sys_history received TCA in 6.0/6.1 when the BELog module was added so Extbase could create a Domain Model based on TCA.

In 9.0, sys_history was extracted from sys_log so all logic was removed there. The left-over "sys_history" TCA is never used, never shown and "adminOnly".


Related issues

Related to TYPO3 Core - Bug #87990: Update typo3/testing-framework to 5.0.7 Closed 2019-03-23

Associated revisions

Revision b0f74bf8 (diff)
Added by Benni Mack 7 months ago

[!!!][TASK] Remove TCA for sys_history

TCA for sys_history is not necessary at all anymore, and
can be removed.

It was previously used for creating an Extbase Domain Model
for the BELog module created in TYPO3 6.0/6.1.

However, in 9.0 sys_history was decoupled from sys_log,
whereas the Domain Model and the usages for TCA was
not necessary anymore.

The previously "generated" database fields are re-added
manually in ext_tables.sql. The field "pid" which was necessary
for a TCA table is now dropped.

Resolves: #87936
Releases: master
Change-Id: Ia541e9a5c7d85f25b6ff3ceb7e3ee311300762e6
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/60273
Tested-by: Anja Leichsenring <>
Tested-by: TYPO3com <>
Tested-by: Alexander Schnitzler <>
Tested-by: Daniel Goerz <>
Reviewed-by: Anja Leichsenring <>
Reviewed-by: Alexander Schnitzler <>
Reviewed-by: Daniel Goerz <>

History

#1 Updated by Gerrit Code Review 7 months ago

  • Status changed from New to Under Review

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/c/Packages/TYPO3.CMS/+/60273

#2 Updated by Gerrit Code Review 7 months 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/c/Packages/TYPO3.CMS/+/60273

#3 Updated by Benni Mack 7 months ago

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

#4 Updated by Oliver Hader 7 months ago

  • Related to Bug #87990: Update typo3/testing-framework to 5.0.7 added

#5 Updated by Benni Mack 5 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF