Summary: | Copy Appearance Properties Error | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Modeling] GMF-Runtime | Reporter: | Thomas Beyer <thomas.beyer> | ||||||
Component: | General | Assignee: | Alex Boyko <aboyko> | ||||||
Status: | RESOLVED FIXED | QA Contact: | |||||||
Severity: | normal | ||||||||
Priority: | P3 | CC: | aboyko, thomas.beyer | ||||||
Version: | 2.2 | Keywords: | contributed | ||||||
Target Milestone: | 2.2.1 | ||||||||
Hardware: | PC | ||||||||
OS: | Windows XP | ||||||||
Whiteboard: | |||||||||
Attachments: |
|
Description
Thomas Beyer
2009-06-25 10:22:50 EDT
Created attachment 140099 [details]
Patch
I'd suggest to check if the feature value isSet() then set it if it is instead of a null check. Created attachment 140189 [details]
Patch
I think both approaches don't cover all use cases. Let's say there are 2 elements with a FillStyle. Element A has the GradientStyle set, element B has not. Now I would select element B first and then element A, and invoke copy appearance properties. So the styles of B should get copied into A. The null-check resp. check if the feature is set would always avoid the resetting of the GradientStyle of element A. Attached patch above should take this idea into consideration and use appropriate EMF-methods instead of null-checks. Yes, we do need to unset the feature on a new style if it is unset on the old one. Good catch. Committed Thomas's patch to HEAD and Maintenance streams for 2.2.1 [GMF Restructure] Bug 319140 : product GMF and component Runtime Diagram was the original product and component for this bug |