Project

General

Profile

Actions

Bug #69386

closed

Remove data processing trait in favor of object

Added by Helmut Hummel about 9 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
-
Start date:
2015-08-28
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

The introduced DataProcessingTrait isn't exactly a good prototype
for Trait usage.

It incorporates functionality that isn't easily testable,
it is impossible to provided different implementations as the trait
is directly bound to the class with its concrete implementation.

Use a composite object instead for this functionality.


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #70013: TS can not be saved in template moduleClosed2015-09-22

Actions
Related to TYPO3 Core - Task #68459: Extract dataProcessors logic into a traitClosedSascha Egerer2015-07-22

Actions
Actions #1

Updated by Gerrit Code Review about 9 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 http://review.typo3.org/42904

Actions #2

Updated by Gerrit Code Review about 9 years ago

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

Actions #3

Updated by Gerrit Code Review about 9 years ago

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

Actions #4

Updated by Gerrit Code Review about 9 years ago

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

Actions #5

Updated by Gerrit Code Review about 9 years ago

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

Actions #6

Updated by Markus Sommer about 9 years ago

  • Assignee set to Helmut Hummel
Actions #7

Updated by Helmut Hummel about 9 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions #9

Updated by Christian Kuhn about 3 years ago

  • Related to Task #68459: Extract dataProcessors logic into a trait added
Actions

Also available in: Atom PDF