Task #43995

Cleanups for DataHandler (formerly known as TCEmain)

Added by Sebastian Michaelsen almost 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2012-12-14
Due date:
% Done:

0%

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

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

History

#1 Updated by Gerrit Code Review almost 7 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

#2 Updated by Gerrit Code Review almost 7 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17155

#3 Updated by Gerrit Code Review almost 7 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17155

#4 Updated by Gerrit Code Review over 6 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17155

#5 Updated by Wouter Wolters over 6 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

Also available in: Atom PDF