Project

General

Profile

Actions

Epic #67492

closed

Clean up DataHandler

Added by Andreas Wolf over 9 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
-
Start date:
2015-06-07
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Sprint Focus:

Description

DataHandler has a lot of hidden complexity in huge methods that nobody can fully grasp. Several aspects can be improved to make the class easier to handle, including:

  1. introduce early returns in methods/early continues in foreach loops, where possible
  2. split up long methods into smaller methods
  3. introduce parameter objects to get rid of overly long method signatures

All cleanups should keep the external interface intact for now, deprecation is allowed though.

See #67336 for one of the issues already created for this epic.


Subtasks 12 (0 open12 closed)

Task #67336: Introduce early returns in DataHandlerClosedAndreas Wolf2015-06-07

Actions
Task #67500: Early loop continues in DataHandler::process_*ClosedAndreas Wolf2015-06-16

Actions
Task #67510: Split up DataHandler::copyRecord_procBasedOnFieldType()ClosedAndreas Wolf2015-06-16

Actions
Task #67579: Remove code duplication in DataHandler::copyRecord_processManyToMany()ClosedAndreas Wolf2015-06-17

Actions
Task #67582: Introduce early continue in DataHandler::copyRecord_fixRTEmagicImages()ClosedAndreas Wolf2015-06-17

Actions
Task #67583: Introduce early returns in DataHandler::localize()ClosedAndreas Wolf2015-06-17

Actions
Task #67584: Introduce early returns in DataHandler::fillInFieldArray()ClosedAndreas Wolf2015-06-17

Actions
Bug #67585: Introduce early returns in DataHandler::deleteRecord()ClosedAndreas Wolf2015-06-17

Actions
Task #68207: Introduce early returns in DataHandler::versionize()ClosedAndreas Wolf2015-07-16

Actions
Task #68208: Introduce early returns in DataHandler::inlineLocalizeSynchronizeClosedAndreas Wolf2015-07-16

Actions
Task #68209: Introduce early returns in DataHandler::checkValue_flex_procInData_travDS()ClosedAndreas Wolf2015-07-16

Actions
Task #68775: Introduce early returns in DataHandler::canDeletePageClosedAndreas Wolf2015-08-05

Actions
Actions #1

Updated by Andreas Wolf over 9 years ago

  • Description updated (diff)
Actions #2

Updated by Benni Mack about 9 years ago

  • Status changed from In Progress to Resolved
Actions #3

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF