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

Bug 340339

Summary: Need some Utils class for Folder/File/Project management
Product: z_Archived Reporter: Nicolas Guyomar <nicolas.guyomar>
Component: EMF-FacetAssignee: Nicolas Bros <nicolas.bros>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P2 CC: emft.facet-inbox, gdupe, nicolas.bros
Version: unspecifiedFlags: gdupe: juno+
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Patch for Bug 340339 : Utils classes
gdupe: iplog+
New Plug-in : org.eclipse.emf.facet.util.core.tests nicolas.bros: iplog+, nicolas.bros: review+

Description Nicolas Guyomar CLA 2011-03-17 11:56:54 EDT
Hi,

As part of the implementation of the new Facet framework, I'm facing a need to use some old Utils from the old EMF Facet infra layer.

In the attached patch on org.eclipse.emf.facet.util.core, I've extracted only the method I need, from EMF Facet independent Utils class.

Original authors where Gregoire Dupe, Gabriel Barbier, Fabien Giquel and Nicolas Bros, all four are committers.

Regards,
Nicolas Guyomar
Comment 1 Nicolas Guyomar CLA 2011-03-17 11:58:34 EDT
Created attachment 191432 [details]
Patch for Bug 340339 :  Utils classes

Hi,

(a) I, Nicolas Guyomar, wrote 100% of the code I've provided.
(b) I have the right to contribute the code to Eclipse.
(c) I contribute the content under the EPL.

Regards,
Nicolas Guyomar
Comment 2 Gregoire Dupe CLA 2011-03-17 14:34:29 EDT
Comment on attachment 191432 [details]
Patch for Bug 340339 :  Utils classes

Here is a contribution from one employee of Mia-Software, targeting future Indigo release. The company has signed a Member Commiter Agreement. The contribution does not need a CQ (see bug 322327).

I've committed this contribution. (Committed revision 431.)

I've also:
 - cleaned the code and changed to private the methods that do not look useful for the developer.
 - copied the file Activator.java.template from /org.eclipse.emf.facet.infra.common.core/resources/Activator.java.template
 - added a checkstyle configuration file.

Regards,
Gregoire Dupe
Comment 3 Gregoire Dupe CLA 2011-03-17 14:53:36 EDT
Please, note that I've merge the classes PluginUtils and ProjectUtils. Now, we only have the class PluginUtil.

Regards,
Gregoire
Comment 4 Nicolas Guyomar CLA 2011-05-17 11:27:32 EDT
Created attachment 195878 [details]
New Plug-in : org.eclipse.emf.facet.util.core.tests

Hi,

Please find attached a test plug-in which covers every public methods of the classes FileUtils FolderUtils and PluginUtils

(a) I, Nicolas Guyomar, wrote 100% of the code I've provided.
(b) I have the right to contribute the code to Eclipse.
(c) I contribute the content under the EPL.
(d) This contribution contains no Cryptography features.

Regards,
Nicolas Guyomar
Comment 5 Gregoire Dupe CLA 2011-07-20 11:05:38 EDT
The commit of unit test is needed to close this bug => P1.
Comment 6 Nicolas Bros CLA 2011-09-30 10:37:55 EDT
Comment on attachment 195878 [details]
New Plug-in : org.eclipse.emf.facet.util.core.tests

This is a contribution from an employee of Mia-Software, targeting a future
Juno release. The company has signed a Member Committer Agreement. The
contribution does not need a CQ (see bug 322327).

Committed revision 897.
Comment 7 Nicolas Bros CLA 2011-09-30 10:39:01 EDT
Marking as fixed since the patches were committed.
Comment 8 Gregoire Dupe CLA 2012-05-23 08:16:38 EDT
This bug can be closed.