Task #43995
closed
Cleanups for DataHandler (formerly known as TCEmain)
0%
Description
Be honest. Thinking about refactoring TCEmain (now \TYPO3\CMS\Core\DataHandling\DataHandler) scares the s*** out of you. But at some point we have to do it.
I think a basic cleanup will make further steps easier. So this change is about:
- Provide proper phpDoc comments for all properties and methods
- Fix some nobrainer code smells
Updated by Gerrit Code Review over 11 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17155
Updated by Gerrit Code Review over 11 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17155
Updated by Gerrit Code Review over 11 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17155
Updated by Gerrit Code Review about 11 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17155
Updated by Wouter Wolters almost 11 years ago
- Status changed from Under Review to Closed
Author: "@todo Define visilibility" shouldn't be removed and patch can't merge anymore and was too big anyway