Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 316895

Summary: [Discoverer JEE] Code Factorisation for JEE Discoverer
Product: [Modeling] MoDisco Reporter: Nicolas Guyomar <nicolas.guyomar>
Component: TechnologiesAssignee: Fabien Giquel <fabien.giquel>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: fabien.giquel, hugo.bruneliere, modisco.web-inbox
Version: 0.9.0Flags: fabien.giquel: indigo+
Target Milestone: 0.9.0 M1   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
JEE Discoverer Factorisation
none
JEE Discoverer Code Factorisation fabien.giquel: iplog+

Description Nicolas Guyomar CLA 2010-06-15 09:07:40 EDT 
Hi,

In order to reduce the amount of code doing exactly the same thing (version detection or DTD detection, scheduling the editor opening) in the discovery of JEE Deploiement Descriptor file, I have factorized some methods from both webApp and ejbJar discoverer to org.eclipse.gmt.modisco.jee 

Regards

Nicolas
Comment 1 Nicolas Guyomar CLA 2010-06-15 09:08:48 EDT 
Created attachment 171922 [details]
JEE Discoverer Factorisation

This patch contains the factorisation of several methods redundant in JEE discoverer to org.eclipse.gmt.modisco.jee
Comment 2 Fabien Giquel CLA 2010-06-16 12:22:26 EDT 
Hi Nicolas,

there seems to be some problems with the patch :

- the class "DeployementDescriptorDiscoverer" does not exists in patch.
- the patch does not apply properly to the last ejbjar components from svn eclipse, please check the base version you were working on.

Thanks.
Fabien.
Comment 3 Nicolas Guyomar CLA 2010-06-17 06:56:53 EDT 
Created attachment 172105 [details]
JEE Discoverer Code Factorisation

Hi Fabien,

This is my fault, I forgot to synchronize my project for the SVN diff method to take in account new files

My mistake

This one should work

Nicolas
Comment 4 Fabien Giquel CLA 2010-06-17 07:00:55 EDT 
Hi Nicolas,

the patch has been commited into SVN. Thank for your contribution.

Fabien.
Comment 5 Fabien Giquel CLA 2010-06-21 10:04:28 EDT 
Nicolas,

Could you make the usual confirmations about contribution rights ?

Fabien.
Comment 6 Nicolas Guyomar CLA 2010-06-21 10:05:25 EDT 
Hi,

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 7 Hugo Bruneliere CLA 2010-10-04 13:39:54 EDT 
The patch has been successfully applied.