[jbosstools-issues] [JBoss JIRA] Commented: (JBIDE-7558) remove duplicate action code for instance, image and cloud view

Andre Dietisheim (JIRA) jira-events at lists.jboss.org
Fri Nov 12 12:01:36 EST 2010


    [ https://jira.jboss.org/browse/JBIDE-7558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12563141#comment-12563141 ] 

Andre Dietisheim commented on JBIDE-7558:
-----------------------------------------

Cleaned Image View:
Corrected to use DeltaCloudImage (used CVImageElement before) so that it may be used in DeltaCloudView and ImageView
Added refresh images handler, added context menu to ImageView, removed old action code

There's no duplicate action code any more. All views trigger the same handlers.

> remove duplicate action code for instance, image and cloud view
> ---------------------------------------------------------------
>
>                 Key: JBIDE-7558
>                 URL: https://jira.jboss.org/browse/JBIDE-7558
>             Project: Tools (JBoss Tools)
>          Issue Type: Bug
>          Components: deltacloud
>    Affects Versions: 3.2.0.Beta2
>            Reporter: Andre Dietisheim
>            Assignee: Andre Dietisheim
>             Fix For: 3.2.0.CR1
>
>
> current implementation has a lot of duplicate code in order to handle actions on the same deltacloud instances and images in the different views. 
> The cloud view uses:
> CVInstanceElement and CVImageElement 
> The image view uses:
> DeltaCloudImage
> The instance view uses: 
> DeltaCloudInstance
> The current code does not use adapters and therefore duplicated action-, providers etc. to handle these different object types. The result is that changes (a lot of the current issues) get tedious.
> The solution is pretty simple:
> Use the DeltaCloudInstance and DeltaCloudImage everywhere, wire UI by using adapters.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


More information about the jbosstools-issues mailing list