]> gerrit.simantics Code Review - simantics/platform.git/commitdiff
Fixed component type editor Lift Properties domain setting. 63/263/1
authorTuukka Lehtonen <tuukka.lehtonen@semantum.fi>
Tue, 10 Jan 2017 20:32:38 +0000 (22:32 +0200)
committerTuukka Lehtonen <tuukka.lehtonen@semantum.fi>
Tue, 10 Jan 2017 20:32:38 +0000 (22:32 +0200)
Previously Lift Properties would copy the lifted property relation over
to the user component type as-is which would also leave all the existing
L0.HasDomain relations of the relation in the copy. This can cause
totally unwanted HasDomain definitions in the property relations which
in turn can cause unexpected export-import problems later because of
those unexpected dependencies.

This changes the code to first remove all the existing HasDomain
definitions from the copied property relation and then add the HasDomain
relation to the target component type.

refs #6947

Change-Id: I10090db57ab343abcd35fe93e54265668ecb7e72

bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/componentTypeEditor/ConfigurationPropertiesSection.java

index 0d09b3bd207258b5bcabb418e375c5608746f5cf..9a2a713df0fa8807ddd55ac48935343ad824bb5f 100644 (file)
@@ -425,6 +425,7 @@ public class ConfigurationPropertiesSection implements ComponentTypeViewerSectio
                                         LiftedProperty p = result.get(index);
                                         Collection<Resource> copyAss = Layer0Utils.copyTo(graph, null, ass);
                                         if(copyAss.size() == 1) {
+                                            graph.deny(copy, L0.HasDomain);
                                             graph.claim(data.componentType, L0.DomainOf, copy);
                                             Layer0Utils.assert_(graph, data.componentType, copy, copyAss.iterator().next());
                                             CommentMetadata cm = graph.getMetadata(CommentMetadata.class);