Refactor #3205

extract report importing logic into a separate service class

Added by Ohad Levy over 4 years ago. Updated over 4 years ago.

Status:Closed
Priority:Normal
Assigned To:Ohad Levy
Category:Reporting
Target version:Sprint 16
Difficulty: Bugzilla link:
Found in release: Pull request:
Story points-
Velocity based estimate-
Release1.4.0Release relationshipAuto

Description

currently report importing is handled in the report class it self, since this is a contain set of logic, it could be moved into its own set of class.

later on, it should be easier to alias these methods for async operations.

Associated revisions

Revision 974075d7
Added by Ohad Levy over 4 years ago

fixes #3205 - Extract and refactor report importing logic

This patch extracts all report importing logic into seperate importer and status classes.
long term this could allow:

1. aliases methods for async operations
2. different types of reports imports (based on reporting type).

History

#1 Updated by Dominic Cleal over 4 years ago

  • Category set to Reporting
  • Status changed from New to Ready For Testing
  • Target version set to 1.4.0

#2 Updated by Greg Sutcliffe over 4 years ago

  • Target version deleted (1.4.0)

#3 Updated by Dominic Cleal over 4 years ago

  • Target version set to Sprint 16

#4 Updated by Dominic Cleal over 4 years ago

  • Release set to 1.4.0

#5 Updated by Ohad Levy over 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF