Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authoratikhomirov2006-04-10 07:53:24 -0400
committeratikhomirov2006-04-10 07:53:24 -0400
commit2b871116ed64253309f3ef9ff4dcfb5ef12b8610 (patch)
tree709b77b74c046e0b267b78505ebd16d4a2368b0d /plugins/org.eclipse.gmf.graphdef.codegen/templates/attr
parent096616f398e7160e44bebb0a8ae38622b54ba4c3 (diff)
downloadorg.eclipse.gmf-tooling-2b871116ed64253309f3ef9ff4dcfb5ef12b8610.tar.gz
org.eclipse.gmf-tooling-2b871116ed64253309f3ef9ff4dcfb5ef12b8610.tar.xz
org.eclipse.gmf-tooling-2b871116ed64253309f3ef9ff4dcfb5ef12b8610.zip
Follow-up for #135016 Background colour is not used to fill polygons - most polylines are links, thus, no much sense to setFill(true) for them
Diffstat (limited to 'plugins/org.eclipse.gmf.graphdef.codegen/templates/attr')
-rw-r--r--plugins/org.eclipse.gmf.graphdef.codegen/templates/attr/Shape.javajet5
1 files changed, 3 insertions, 2 deletions
diff --git a/plugins/org.eclipse.gmf.graphdef.codegen/templates/attr/Shape.javajet b/plugins/org.eclipse.gmf.graphdef.codegen/templates/attr/Shape.javajet
index b9a70dd3a..934a11c6a 100644
--- a/plugins/org.eclipse.gmf.graphdef.codegen/templates/attr/Shape.javajet
+++ b/plugins/org.eclipse.gmf.graphdef.codegen/templates/attr/Shape.javajet
@@ -10,8 +10,9 @@ final ImportAssistant importManager = dispatcher.getImportManager();
<%
// Most of the properties are set only if corresponding eProperty isSet.
// The difference is in the default values. If default value of eProperty matches d2d one, the setter may be omitted.
-// d2d.Shape#fill == true (matches default of gmfgraph.Shape#fill), while d2d.Polyline#fill == false, thus we need additional check
-%><%if (figureInstance instanceof Polyline ^ figureInstance.eIsSet(GMFGraphPackage.eINSTANCE.getShape_Fill())) {%>
+// d2d.Shape#fill == true (matches default of gmfgraph.Shape#fill), while d2d.Polyline#fill == false, thus we need additional check.
+// Note Polygon in the check instead of Polyline - (it looks like) there's no sense to setFill(true) for polylines in any case.
+%><%if (figureInstance instanceof Polygon ^ figureInstance.eIsSet(GMFGraphPackage.eINSTANCE.getShape_Fill())) {%>
<%=figureVarName%>.setFill(<%=figureInstance.isFill()%>);
<%} if (figureInstance.eIsSet(GMFGraphPackage.eINSTANCE.getShape_Outline())) {%>
<%=figureVarName%>.setOutline(<%=figureInstance.isOutline()%>);

Back to the top