Bug #69386
closedRemove data processing trait in favor of object
100%
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.
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
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
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
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
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
Updated by Helmut Hummel about 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset e864a5c93598bebbcae77a4cc276e50b0cb032e5.
Updated by Christian Kuhn about 3 years ago
- Related to Task #68459: Extract dataProcessors logic into a trait added