Bug #15284 fixed (again) - SuperBlocks port labels were not working 08/20708/4
Clément DAVID [Fri, 18 Jan 2019 09:33:24 +0000 (10:33 +0100)]
Change-Id: I09c4285929ffb81b0296405aee2529c57eef6f50

scilab/modules/xcos/src/java/org/scilab/modules/xcos/actions/EditFormatAction.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/graph/XcosDiagram.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/io/writer/PortWriter.java

index 869a297..02683f3 100644 (file)
@@ -58,7 +58,7 @@ import java.util.List;
 import org.scilab.modules.xcos.JavaController;
 import org.scilab.modules.xcos.Kind;
 import org.scilab.modules.xcos.ObjectProperties;
-import org.scilab.modules.xcos.VectorOfDouble;
+import org.scilab.modules.xcos.VectorOfInt;
 import org.scilab.modules.xcos.Xcos;
 import org.scilab.modules.xcos.block.SuperBlock;
 import org.scilab.modules.xcos.block.io.ContextUpdate;
@@ -347,7 +347,7 @@ public final class EditFormatAction extends DefaultAction {
         if (fontSize != mxConstants.DEFAULT_FONTSIZE) {
             identifierStyle.put(mxConstants.STYLE_FONTSIZE, Integer.toString(fontSize));
         } else {
-            identifierStyle.remove(mxConstants.DEFAULT_FONTSIZE);
+            identifierStyle.remove(mxConstants.STYLE_FONTSIZE);
         }
 
         if (!textColor.equals(DEFAULT_BORDERCOLOR)) {
@@ -394,9 +394,9 @@ public final class EditFormatAction extends DefaultAction {
         if (cell instanceof ContextUpdate) {
             JavaController controller = new JavaController();
 
-            VectorOfDouble ipar = new VectorOfDouble();
+            VectorOfInt ipar = new VectorOfInt();
             controller.getObjectProperty(((ContextUpdate) cell).getUID(), Kind.BLOCK, ObjectProperties.IPAR, ipar);
-            int portNumber = (int) (ipar.size() > 0 ? ipar.get(0) : 1);
+            int portNumber = ipar.size() > 0 ? ipar.get(0) : 1;
 
             XcosCell parent = (XcosCell) graph.getDefaultParent();
             XcosDiagram parentGraph = Xcos.findParent(controller, parent.getUID(), parent.getKind());
index 2e715e5..f429ff1 100644 (file)
@@ -4,7 +4,7 @@
  * Copyright (C) 2009-2010 - DIGITEO - Clement DAVID
  * Copyright (C) 2011-2017 - Scilab Enterprises - Clement DAVID
  * Copyright (C) 2015 - Marcos Cardinot
- * Copyright (C) 2017 - ESI Group - Clement DAVID
+ * Copyright (C) 2017-2019 - ESI Group - Clement DAVID
  *
  * Copyright (C) 2012 - 2016 - Scilab Enterprises
  *
@@ -1949,15 +1949,8 @@ public class XcosDiagram extends ScilabGraph {
         if (getKind() == Kind.DIAGRAM) {
             controller.getObjectProperty(getUID(), getKind(), ObjectProperties.TITLE, property);
         } else { // Kind.BLOCK
-            // if an annotation is present use it, otherwise use the one-line description
-            long[] annotation = { 0 };
-            controller.getObjectProperty(getUID(), getKind(), ObjectProperties.LABEL, annotation);
-
-            if (annotation[0] != 0) {
-                controller.getObjectProperty(annotation[0], Kind.ANNOTATION, ObjectProperties.DESCRIPTION, property);
-            } else {
-                controller.getObjectProperty(getUID(), getKind(), ObjectProperties.DESCRIPTION, property);
-            }
+            // use the one-line description
+            controller.getObjectProperty(getUID(), getKind(), ObjectProperties.DESCRIPTION, property);
         }
 
         if (property[0].isEmpty()) {
index 89fde1e..ed59974 100644 (file)
@@ -105,5 +105,9 @@ public class PortWriter extends ScilabWriter {
         String[] style = new String[1];
         shared.controller.getObjectProperty(uid, Kind.PORT, ObjectProperties.STYLE, style);
         shared.stream.writeAttribute("style", style[0]);
+
+        String[] value = new String[1];
+        shared.controller.getObjectProperty(uid, Kind.PORT, ObjectProperties.LABEL, value);
+        shared.stream.writeAttribute("value", value[0]);
     }
 }