This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 228270 - Full/Partial registration/deregistration dialog boxes should display icons of mdrs
Summary: Full/Partial registration/deregistration dialog boxes should display icons of...
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Cosmos (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: amehrega CLA
QA Contact:
URL:
Whiteboard: useability
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-22 15:58 EDT by Sheldon Lee-Loy CLA
Modified: 2012-01-03 13:46 EST (History)
1 user (show)

See Also:

Attachments
Patch (61.10 KB, patch)
2008-04-29 14:57 EDT, amehrega CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sheldon Lee-Loy CLA 2008-04-22 15:58:05 EDT 
The dialog boxes that display the mdr to the user should also show the associated icons in the dialog box.
Comment 1 amehrega CLA 2008-04-29 14:55:51 EDT 
Taking over this defect
Comment 2 amehrega CLA 2008-04-29 14:57:33 EDT 
Created attachment 98045 [details]
Patch

Sheldon,

Please review the patch (which also includes a fix for 221549).  The fix adds icons to the full registration dialog.  I'll recreate another patch to do the same for partial registration.
Comment 3 Sheldon Lee-Loy CLA 2008-05-05 10:56:48 EDT 
(In reply to comment #2)
You will need to create a patch for the deregistration scenarios as well.

I reviewed the patch and the only thing I see is whether the modification to org.eclipse.cosmos.dc.cmdbf.services.internal.registration.transform.output.artifacts.Response
is necessary for this bug.

I also notice you cleaned up some code in the Broker Outputter which is good.  

You can check in this patch and work on the partial registration and deregistration during i11.
Comment 4 amehrega CLA 2008-05-06 09:43:27 EDT 
Thanks for reviewing Sheldon.  I'm workings towards a patch to also modify dialogs for full de-registration, partial registration/de-registration.  I'll check-in what I currently have as soon as Hubert completes archiving plug-ins not required.
Comment 5 amehrega CLA 2008-05-06 09:44:41 EDT 
(In reply to comment #3)
org.eclipse.cosmos.dc.cmdbf.services.internal.registration.transform.output.artifacts.Response
> is necessary for this bug.
> 

Yes, the modification is required.
Comment 6 Jimmy Mohsin CLA 2008-05-08 19:03:41 EDT 
Fixed typo in summary field....
Comment 7 amehrega CLA 2008-05-14 14:49:55 EDT 
Fix checked into CVS - HEAD.
Comment 8 Sheldon Lee-Loy CLA 2008-06-20 15:45:31 EDT 
Verified bug with build COSMOS-1.0.0-200806201443
Comment 9 Sheldon Lee-Loy CLA 2008-07-28 13:10:39 EDT 
Marked as closed based on previous comments.