LibrePlan / libreplan

LibrePlan - Open Web Planning
https://www.libreplan.dev
GNU Affero General Public License v3.0
289 stars 170 forks source link

Code refactor in OrderPlanningModel regarding chart panels #1485

Open kwoot opened 12 years ago

kwoot commented 12 years ago

(Original Bugzilla Bug ID: 1341)

Date: 2012-01-13 16:49:54 From: Jacobo Aragunde Perez \<jaragunde@igalia.com> To: Javier Morán \<jmoran@igalia.com> Version: libreplan-1.2 (1.2.x) Last updated: 2012-01-13 17:04:12


(Note: this issue was migrated automatically with bugzilla2github.py tool )

kwoot commented 6 years ago

Bugzilla Comment ID: 3634 Date: 2012-01-13 16:49:54 From: Jacobo Aragunde Perez \<jaragunde@igalia.com>

Diego added an internal helper class in OrderPlanningModel to gather all the operations related with "Overall Progress" panel. Unfortunately, not such a thing exists for the "Earned Value" and Load" panels, which code is spread amond different methods in OrderPlanningModel.

It could be a good idea doing a code rebase to leave the implementation of the three panels somehow similar.

This came to my mind when reading code to fix the bug #1340.