Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 340339 - Need some Utils class for Folder/File/Project management
Summary: Need some Utils class for Folder/File/Project management
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EMF-Facet (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Nicolas Bros CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-17 11:56 EDT by Nicolas Guyomar CLA
Modified: 2020-05-01 11:26 EDT (History)
3 users (show)

See Also:
gdupe: juno+


Attachments
Patch for Bug 340339 : Utils classes (23.65 KB, patch)
2011-03-17 11:58 EDT, Nicolas Guyomar CLA
gdupe: iplog+
Details | Diff
New Plug-in : org.eclipse.emf.facet.util.core.tests (13.17 KB, application/octet-stream)
2011-05-17 11:27 EDT, Nicolas Guyomar CLA
nicolas.bros: iplog+
nicolas.bros: review+
Details

Note You need to log in before you can comment on or make changes to this bug.
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.