Task #67583
closed
Epic #67492: Clean up DataHandler
Introduce early returns in DataHandler::localize()
Added by Andreas Wolf over 9 years ago.
Updated about 7 years ago.
Category:
DataHandler aka TCEmain
Description
localize() has quite a few nested levels that can be converted to early returns. The method is one of the worst examples of DataHandler code, with two lines that have 14 (!) trailing tabs (i.e. 12 indentation levels within the method).
- Status changed from In Progress 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/40385
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/40385
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/40385
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/40385
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Status changed from Resolved to Closed
Also available in: Atom
PDF