Task #89136

Clean up DataHandlerHook in Workspaces

Added by Benni Mack 10 days ago. Updated 8 days ago.

Status:
Resolved
Priority:
Should have
Assignee:
Category:
Workspaces
Start date:
2019-09-10
Due date:
% Done:

100%

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

Associated revisions

Revision 29768702 (diff)
Added by Benni Mack 8 days ago

[TASK] Clean up DataHandlerHook in Workspaces

DataHandlerHook originally extracted most of the versioning
functionality of DataHandler, and can now be cleaned up:

- Most of the protected methods have better type definitions
and return types, where it was safe to do so
- Some properties / methods / arguments was
not used and could be cleaned up

Resolves: #89136
Releases: master
Change-Id: I73734f57356e31381062043be7a6056a13d16b0b
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/61662
Tested-by: Oliver Hader <>
Tested-by: TYPO3com <>
Tested-by: Daniel Gorges <>
Tested-by: Benni Mack <>
Reviewed-by: Oliver Hader <>
Reviewed-by: Daniel Gorges <>
Reviewed-by: Benni Mack <>

History

#1 Updated by Gerrit Code Review 10 days 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/c/Packages/TYPO3.CMS/+/61662

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

#3 Updated by Gerrit Code Review 9 days 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/+/61662

#4 Updated by Gerrit Code Review 9 days 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/c/Packages/TYPO3.CMS/+/61662

#5 Updated by Gerrit Code Review 9 days 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/c/Packages/TYPO3.CMS/+/61662

#6 Updated by Benni Mack 8 days ago

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

Also available in: Atom PDF