Bugzilla – Full Text Bug Listing
| Summary: | [Restructuring] To be able to use non derived facet structural features and save them values. | ||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Gregoire Dupe <gdupe> | ||||||||||||||||||||
| Component: | EMF-Facet | Assignee: | Nicolas Bros <nicolas.bros> | ||||||||||||||||||||
| Status: | CLOSED FIXED | QA Contact: | |||||||||||||||||||||
| Severity: | enhancement | ||||||||||||||||||||||
| Priority: | P2 | CC: | erouille, jim.van.dam, nicolas.bros, roman.porotnikov | ||||||||||||||||||||
| Version: | unspecified | Flags: | gdupe:
juno+
|
||||||||||||||||||||
| Target Milestone: | --- | ||||||||||||||||||||||
| Hardware: | All | ||||||||||||||||||||||
| OS: | All | ||||||||||||||||||||||
| Whiteboard: | |||||||||||||||||||||||
| Bug Depends on: | |||||||||||||||||||||||
| Bug Blocks: | 352718 | ||||||||||||||||||||||
| Attachments: |
|
||||||||||||||||||||||
|
Description
Gregoire Dupe
*** Bug 346650 has been marked as a duplicate of this bug. *** Created attachment 200182 [details]
New plug-in org.eclipse.emf.facet.efacet2
Hello,
The plug-in org.eclipse.emf.facet.efacet2 was added in the previous attachment (attachment 200182 [details]) to fix this bug. This is a new facet metamodel. It has been created in collaboration with Nicolas Guyomar who added classes ContainementReferenceInstance and AbstractReferenceInstance to be able to use "containment facet references".
(a) I, Emmanuelle Rouillé, 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,
Emmanuelle Rouillé
Created attachment 200185 [details]
New plug-in org.eclipse.emf.facet.efacet2.examples.library
Here is a new plug-in to fix this bug. This is a metamodel of library. It is useful to create tests cases. It has been created in collaboration with Nicolas Guyomar who modified toString() methods in order to allow a comparison between two files.
(a) I, Emmanuelle Rouillé, 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,
Emmanuelle Rouillé
Created attachment 200186 [details]
New plug-in org.eclipse.emf.facet.efacet2.examples
Here is a new plug-in containing examples of models conformed to the new facet metamodel and to the library metamodel.
(a) I, Emmanuelle Rouillé, 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,
Emmanuelle Rouillé
Created attachment 200193 [details]
fix
Here is a patch to fix this bug. It contains the code to serialize the extension of a model with facets. It also contains code to apply a non-derived facet element to an eObject and to get its value. Classes referencing the previous version of the facet metamodel have also been duplicated when needed, in order to reference the new facet metamodel.
(a) I, Emmanuelle Rouillé, 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,
Emmanuelle Rouillé
Created attachment 200196 [details]
fix
Here is a patch to fix this bug. It contains a duplication of the javaquery metamodel, which refers to the previous version of the facet metamodel in order to refer to the new facet metamodel.
(a) I, Emmanuelle Rouillé, 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,
Emmanuelle Rouillé
Created attachment 200197 [details]
fix
Here is a patch to fix this bug. Classes referencing the previous version of the facet metamodel have been duplicated when needed, in order to reference the new facet metamodel.
(a) I, Emmanuelle Rouillé, 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,
Emmanuelle Rouillé
Created attachment 200199 [details]
fix
Here is a patch to fix this bug. It contains a duplication of the efacetcatalog
metamodel, which refers to the previous version of the facet metamodel in order
to refer to the new facet metamodel.
(a) I, Emmanuelle Rouillé, 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,
Emmanuelle Rouillé
Created attachment 200200 [details]
fix
Here is a patch to fix this bug. Code has been modified in order to return the bundle associated with a resource.
(a) I, Emmanuelle Rouillé, 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,
Emmanuelle Rouillé
Created attachment 200201 [details]
New plug-in org.eclipse.emf.facet.efacet2.tests
Here is a new plug-in providing unit tests for this bug.
(a) I, Emmanuelle Rouillé, 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,
Emmanuelle Rouillé
Comment on attachment 200182 [details] New plug-in org.eclipse.emf.facet.efacet2 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). Comment on attachment 200185 [details] New plug-in org.eclipse.emf.facet.efacet2.examples.library 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). Comment on attachment 200193 [details] fix 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). Comment on attachment 200196 [details] fix 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). Comment on attachment 200197 [details] fix 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). Comment on attachment 200199 [details] fix 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). Comment on attachment 200200 [details] fix 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). Comment on attachment 200201 [details] New plug-in org.eclipse.emf.facet.efacet2.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). I have reviewed these patches, and fixed a few things: missing about.html files, build.properties, externalization, spelling, javadoc comments, versions, added a test suite, fixed warnings, a few metamodel issues, recreated the metamodel diagram (which was empty when I opened it). I have also modified IFacetManager2#get so that it returns a typed value instead of Object. I have committed all these patches/new projects in revision 842. Except for org.eclipse.emf.facet.efacet2.examples, because its model files have references to attributes not (yet) in the metamodel, like Literal::value This bug is not completely resolved though. I have noticed the serialization of multi-valued features is not yet implemented. So I'm leaving this bug open. By the way, these patches were not in the Eclipse patch format, so Eclipse wouldn't apply them. For future reference: I fixed that with the following regex replacement on the patch files: search: "^(--- ([\w/.\-]+)\t\(revision \d+\))$" replace by: "Index: \2\n===================================================================\n\1" The commit 842 has introduce the following warning. This has to be fixed: Missing @since tag on org.eclipse.emf.facet.query.java.core.IJavaQuery2 IJavaQuery2.java (line 27) I've fixed the "missing @since tag" described in Comment 21. Committed revision 1170. I've also fixed "missing @since tag" in org.eclipse.emf.facet.query.java.core. Comment 23 -> Committed revision 1171. I've also enable the API analysis builder on the org.eclipse.emf.facet.query.java.core project. org.eclipse.emf.facet.efacet.core.IFacetManagerFactory.createFacetManager(Resource) must be able to get a null parameter: the user may want to use the facet manager without to have to save additional structural features in a resource. I've implemented this requirement. Committed revision 1183. This bug has to be closed. |