Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325698 - [Java Discoverer] Inappropriate opening of the browser
Summary: [Java Discoverer] Inappropriate opening of the browser
Status: CLOSED FIXED
Alias: None
Product: MoDisco
Classification: Modeling
Component: Technologies (show other bugs)
Version: 0.8.0   Edit
Hardware: PC Windows 7
: P3 minor (vote)
Target Milestone: 0.9.0 M3   Edit
Assignee: Fabien Giquel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-19 11:26 EDT by Romain Dervaux CLA
Modified: 2010-10-18 09:39 EDT (History)
5 users (show)

See Also:
fabien.giquel: indigo+

Attachments
patch (649 bytes, patch)
2010-09-19 11:27 EDT, Romain Dervaux 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 Romain Dervaux CLA 2010-09-19 11:26:13 EDT 
Here is a small bug i detected on the current version of MoDisco on Helios.

I right-click on a Java project in my workspace, I select "MoDisco" -> "Browse corresponding java model element".

The discoverer wizard opens. If i cancel ("Cancel" button or close window), the discoverer will presume the model has been created and tries to open it.

I attached a patch for this problem.
Comment 1 Romain Dervaux CLA 2010-09-19 11:27:22 EDT 
Created attachment 179213 [details]
patch
Comment 2 Fabien Giquel CLA 2010-09-20 04:55:01 EDT 
Hi Romain,
Thanks for detecting and proposing your patch.
Could you make the usual confirmations about authoring and rights to contribute ?
Comment 3 Gabriel BARBIER CLA 2010-09-20 05:31:06 EDT 
Fixed in revision 3171
Comment 4 Romain Dervaux CLA 2010-09-20 06:03:54 EDT 
1. I, DERVAUX romain, 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

Romain DERVAUX
Comment 5 Fabien Giquel CLA 2010-09-20 12:08:50 EDT 
set as resolved.
Comment 6 Hugo Bruneliere CLA 2010-10-05 05:13:02 EDT 
The patch has been successfully applied.