Bug 846322 - Guide models duplicates the code of the ListModel
Summary: Guide models duplicates the code of the ListModel
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-webadmin
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.5.0
Assignee: Einav Cohen
QA Contact: bugs@ovirt.org
URL:
Whiteboard: ux
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-08-07 13:15 UTC by Alona Kaplan
Modified: 2016-02-10 19:46 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-06-16 15:33:22 UTC
oVirt Team: UX
Embargoed:


Attachments (Terms of Use)

Description Alona Kaplan 2012-08-07 13:15:27 UTC
Description of problem:
Guide model duplicates the code of the list model.
For example- VmGuide model has code for addNetwork, this code is very similar to the code of new networks in the VmInterfaceListModel (both of the models have the same presenter and view).
If we want to add or to fix something is the add dialog, and the fix has to be done in the model ,the same fix has to be done twice. And in most of the cases the Guide model is forgotten.



Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Itamar Heim 2013-02-10 06:55:08 UTC
still relevant?

Comment 2 Vojtech Szocs 2013-12-09 14:28:48 UTC
Yes, this is still relevant.

The general problem is code duplication throughout UiCommon code (across different model classes). This is unrelated to GWT(P) application infra code, which simply binds/delegates to UiCommon code.

Not sure if this is UX-related, though. Ideally, different people from different groups should dedicate some effort to analyze possible code duplications and fix them for their domain of responsibility.

We can also consider running some Java code duplication detector tool.

Comment 3 Itamar Heim 2014-01-12 08:42:04 UTC
setting target release to current version for consideration and review. please do not push non-RFE bugs to an undefined target release to make sure bugs are reviewed for relevancy, fix, closure, etc.

Comment 4 Itamar Heim 2014-06-16 15:33:22 UTC
Closing old bugs. If this issue is still relevant/important in current version, please re-open the bug.


Note You need to log in before you can comment on or make changes to this bug.