Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[12/05] Todo's for Intermediary phase. #12

Open
vagosdim opened this issue May 5, 2020 · 0 comments
Open

[12/05] Todo's for Intermediary phase. #12

vagosdim opened this issue May 5, 2020 · 0 comments

Comments

@vagosdim
Copy link
Collaborator

vagosdim commented May 5, 2020

Next steps:

  1. Fix existing bugs and finish Factory class. (edimoulis)
    i. Some right clicks on table squares throw exceptions.
    ii. Implement the interface at ZoomTableHandler

  2. Develop more tests regarding the Data package.
    The Use Cases we want to test are:
    i. Show PLD --> showGeneralLifetimeIDUAction
    ii. Show Phases PLD --> showGeneralLifetimePhasesPLDAction
    iii. Show Phases with Clusters PLD --> showGeneralLifetimePhasesWithClustersPLDAction
    iv. Show Full Detailed LifeTime Table --> showLifetimeTableAction

  3. Move logic related from data processing to the DataKeeper.
    i. Moved InfoAction() from BusinessLogic to GlobalDataKeeper
    ii. Moves Gui.elements.tableConstructors moved to data hyper-packagae as data.tableConstructors
    iii. Moved data population methods from BusinessLogic to Datakeeper

Tips For refactoring the GlobalDataKeeper see:

  1. PPLSchemaManager
  2. PPLTransitionManager
  3. DataKeeper has about only get and set methods but the clients of this class have similar methods that are almost duplicate.

Note In the next report we need to mention that we moved an entire package, the gui.treeElements, to the "Data" hyper-package naming it to data.treeElements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant