Task #67583
closed
Epic #67492: Clean up DataHandler
Introduce early returns in DataHandler::localize()
100%
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).
Updated by Gerrit Code Review over 9 years ago
- 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
Updated by Gerrit Code Review over 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/40385
Updated by Gerrit Code Review over 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/40385
Updated by Gerrit Code Review over 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/40385
Updated by Andreas Wolf over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 805520e869bc862d5a8cd93120cf0f889ed0e595.
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed