Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 342451 - To be able to edit derived facet attributes and derived facet references in a table
Summary: To be able to edit derived facet attributes and derived facet references in a...
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: Gregoire Dupe CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-11 11:16 EDT by Nicolas Guyomar CLA
Modified: 2020-05-01 11:26 EDT (History)
2 users (show)

See Also:
gdupe: indigo+


Attachments
Patch for Bug 342451 Implementation (122.18 KB, patch)
2011-04-12 05:05 EDT, Nicolas Guyomar CLA
gdupe: iplog+
Details | Diff
Patch for Bug 342451 Sample for edition testing (63.90 KB, patch)
2011-04-12 05:20 EDT, Nicolas Guyomar CLA
gdupe: iplog+
Details | Diff
Patch for Bug 342451 Documentation (5.84 KB, patch)
2011-04-13 08:00 EDT, Nicolas Guyomar CLA
gdupe: iplog+
Details | Diff
Patch for Bug 342451 Documentation ScreenShot (64.94 KB, application/octet-stream)
2011-04-13 08:01 EDT, Nicolas Guyomar CLA
gdupe: iplog+
Details
Patch for Bug 342451 Update Ecore Example (42.63 KB, patch)
2011-04-14 04:31 EDT, Nicolas Guyomar CLA
gdupe: iplog+
Details | Diff
Patch for Bug 342451 UI Tests (44.84 KB, patch)
2011-04-22 07:50 EDT, Nicolas Guyomar CLA
no flags Details | Diff
Patch for Bug 342451 UI Tests (70.20 KB, patch)
2011-04-22 10:39 EDT, Nicolas Guyomar CLA
gdupe: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Guyomar CLA 2011-04-11 11:16:40 EDT
Hi,

Since Bug 339880 has been fixed, we should now be able to edit derived facet attributes and derived facet references in the EMF Facet table.

Regards,
Nicolas
Comment 1 Nicolas Guyomar CLA 2011-04-12 05:05:36 EDT
Created attachment 193014 [details]
Patch for Bug 342451 Implementation

Hi,

Please find attached the corresponding implementation in which I've modified org.eclipse.emf.facet.widgets.celleditors and org.eclipse.emf.facet.widgets.nattable plug-ins.

I've added NaryAttributeCellEditor and NaryReferenceCellEditor so that we could differentiate the edition of a FacetAttribute with the FacetReference one.
I've changed the commit mechanism, which was at first implemented in the editing dialog, using an editing domain. In this new version, dialog are now longer responsible of model modification, they only receive a list to edit, and return the result.

I've added several @SuppressWarnings("unchecked") because of the type erasure on generic.

I've renamed some "columnDescription" attributes with "column", since ColumnDescription was replaced with Column a while ago.

We can now edit derived FacetAttribute and derived FacetReference in a table as long as they have a "setQuery" to do so.

(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 2 Nicolas Guyomar CLA 2011-04-12 05:20:07 EDT
Created attachment 193016 [details]
Patch for Bug 342451 Sample for edition testing

Hi,

Please find attached some test material using the samplemm MetaModel.
I've created a facetSet and a querySet to demonstrate the usability of facetReference and facetAttribute in the table.

(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 3 Nicolas Guyomar CLA 2011-04-13 08:00:01 EDT
Created attachment 193154 [details]
Patch for Bug 342451 Documentation

Hi,

Please find attached an updated version of the Table documentation.
Updated screenshots are in the next archive.

(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 4 Nicolas Guyomar CLA 2011-04-13 08:01:24 EDT
Created attachment 193155 [details]
Patch for Bug 342451 Documentation ScreenShot

Hi,

Please find attached updated/new versions of the documentation screenshot.

(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-04-13 10:18:59 EDT
Comment on attachment 193154 [details]
Patch for Bug 342451 Documentation

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 512.
Comment 6 Gregoire Dupe CLA 2011-04-13 10:19:21 EDT
Comment on attachment 193155 [details]
Patch for Bug 342451 Documentation ScreenShot

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 512.
Comment 7 Gregoire Dupe CLA 2011-04-13 10:20:07 EDT
Comment on attachment 193014 [details]
Patch for Bug 342451 Implementation

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 511.
Comment 8 Gregoire Dupe CLA 2011-04-13 10:20:20 EDT
Comment on attachment 193016 [details]
Patch for Bug 342451 Sample for edition testing

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 511.
Comment 9 Gregoire Dupe CLA 2011-04-13 10:21:26 EDT
This bug can be marked as resolved.
Comment 10 Nicolas Guyomar CLA 2011-04-14 04:31:39 EDT
Created attachment 193224 [details]
Patch for Bug 342451 Update Ecore Example

Hi,

Please find attached an updated version of the Ecore example to demonstrate facet editability with a FacetAttribute and a FacetReference.

I've also added some tests in NatTableWidget to avoid "Widget already disposed" exception.

(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 11 Gregoire Dupe CLA 2011-04-14 07:42:05 EDT
Comment on attachment 193224 [details]
Patch for Bug 342451 Update Ecore Example

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 a part of this contribution.

Committed revision 514.
Comment 12 Nicolas Guyomar CLA 2011-04-22 07:50:47 EDT
Created attachment 193910 [details]
Patch for Bug 342451 UI Tests

Hi,

Please find attached several SWT Bot tests to cover editable facets on sampleMM meta-model.

(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 13 Nicolas Guyomar CLA 2011-04-22 10:39:26 EDT
Created attachment 193915 [details]
Patch for Bug 342451 UI Tests

Hi,

My previous attachment contained a reference to a Zip archive, I removed it in this updated version.

(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 14 Gregoire Dupe CLA 2011-04-26 04:18:30 EDT
Comment on attachment 193915 [details]
Patch for Bug 342451 UI Tests

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 567.
Comment 15 Gregoire Dupe CLA 2011-06-07 13:28:56 EDT
This bug can be closed.