From ab3be1ad495b373b656697e4f156f6a308f80c98 Mon Sep 17 00:00:00 2001 From: Tuukka Lehtonen Date: Tue, 10 Jan 2017 22:32:38 +0200 Subject: [PATCH] Fixed component type editor Lift Properties domain setting. 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 --- .../ui/componentTypeEditor/ConfigurationPropertiesSection.java | 1 + 1 file changed, 1 insertion(+) diff --git a/bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/componentTypeEditor/ConfigurationPropertiesSection.java b/bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/componentTypeEditor/ConfigurationPropertiesSection.java index 0d09b3bd2..9a2a713df 100644 --- a/bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/componentTypeEditor/ConfigurationPropertiesSection.java +++ b/bundles/org.simantics.modeling.ui/src/org/simantics/modeling/ui/componentTypeEditor/ConfigurationPropertiesSection.java @@ -425,6 +425,7 @@ public class ConfigurationPropertiesSection implements ComponentTypeViewerSectio LiftedProperty p = result.get(index); Collection 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); -- 2.43.2