Bugzilla – Full Text Bug Listing
| Summary: | Bug in TableLabelProvider : when the column is a FacetColumn the returned label is not correct | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Vincent Lorenzo <vincent.lorenzo> | ||||||||||
| Component: | EMF-Facet | Assignee: | Gregoire Dupe <gdupe> | ||||||||||
| Status: | CLOSED FIXED | QA Contact: | |||||||||||
| Severity: | major | ||||||||||||
| Priority: | P3 | CC: | Ed.Merks, emft.facet-inbox, fabien.giquel, gdupe | ||||||||||
| Version: | unspecified | Flags: | gdupe:
indigo+
Ed.Merks: pmc_approved+ gdupe: review+ fabien.giquel: review+ |
||||||||||
| Target Milestone: | --- | ||||||||||||
| Hardware: | PC | ||||||||||||
| OS: | Windows XP | ||||||||||||
| Whiteboard: | |||||||||||||
| Bug Depends on: | |||||||||||||
| Bug Blocks: | 334240 | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Vincent Lorenzo
Created attachment 196400 [details]
This image illustrates the problem with a UML EnumerationLiteral (column "Importance")
As we can see on this image, the text displayed for the EnumerationLiteral is not correct (it should be <EnumerationLiteral> High). The TableLabelProvider use the method toString instead of the method getTextFor(...) when the column is a Facet Column (Attribute and ReferenceFacet Column)
Hello, To my mind, this problem of label display is major loss of function. That why I change the importance of this bug. I hope that it would be possible to fix this bug for 0.1.0 RC2 or 0.1.0 RC3. Regards, Gregoire Dupe Created attachment 196407 [details] Patch for Bug 346733 Hi, Please find attached the necessary code modifications: getTextFor(Object object) is now called to get the correct label for a ModelQueryResult in the TableLabelProvider. (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 Created attachment 196631 [details] Patch for Bug 346733 Non regression test Hi, Please find attached a non regression test for this bug. (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 Created attachment 196871 [details] Patch for Bug 346733 Non regression test Hi, Please find attached a non regression test using Ecore Metamodel instead of the UML one. (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 Hello Ed and Fabien, To my mind, this problem of label display is major loss of function. The path really is small: only 3 lines are modified and it does not break the unit tests. Do you agree to commit the fix and the corresponding unit test? Regard, Grégoire Comment on attachment 196407 [details] Patch for Bug 346733 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 705. Comment on attachment 196871 [details] Patch for Bug 346733 Non regression test 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 705. This bug can be mark as fixed. Regards, Gregoire Comment on attachment 196407 [details] Patch for Bug 346733 Hello, My commit (revision 705) has been dropped. (I don’t understand why.) I’ve then re-commit this patch. Committed revision 707. Regards, Gregoire Dupe Comment on attachment 196871 [details] Patch for Bug 346733 Non regression test Hello, My commit (revision 705) has been dropped. (I don’t understand why.) I’ve then re-committed this patch. Committed revision 708. Regards, Gregoire Dupe This bug can be marked as closed. |