Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323277 - Java and Kdm Inventory Discoverer : change behavior on "cancel" button
Summary: Java and Kdm Inventory Discoverer : change behavior on "cancel" button
Status: CLOSED FIXED
Alias: None
Product: MoDisco
Classification: Modeling
Component: Technologies (show other bugs)
Version: 0.9.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 0.9.0 M2   Edit
Assignee: Fabien Giquel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-20 13:47 EDT by Fabien Giquel CLA
Modified: 2010-10-18 08:38 EDT (History)
2 users (show)

See Also:
fabien.giquel: indigo+

Attachments
Patch for "Cancel" action (3.93 KB, patch)
2010-09-07 04:42 EDT, Nicolas Guyomar CLA 		fabien.giquel: iplog+
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 Fabien Giquel CLA 2010-08-20 13:47:12 EDT 
Launching the "Java and Kdm Inventory Discoverer" open the "Java Discoverer" parameters window. A button "Cancel" is proposed but not well encountered from calling composite discoverer :
- the whole process of discovering must be canceled (and not create any resources)
- the error message in error log is not convenient since "cancel" is a normal use case.
Comment 1 Nicolas Guyomar CLA 2010-09-07 04:42:37 EDT 
Created attachment 178302 [details]
Patch for "Cancel" action

Hi Fabien,

Please find attached a patch to fix this discoverer behavior regarding the "cancel " action in the java discoverer

1. I, Nicolas Guyomar, and others contributors listed in this bug, authored
100% of the content
2. I have the rights to donate the content to Eclipse under the EPL
3. I have already provided the filled Employer Consent Forms to the Foundation

Regards

Nicolas
Comment 2 Fabien Giquel CLA 2010-09-09 05:32:49 EDT 
The fix has been applied. Thanks.
Comment 3 Hugo Bruneliere CLA 2010-10-05 04:51:10 EDT 
The patch has been successfully applied.