* Bug #15185 fixed - "Region to Superblock" did not restore broken links. 22/19322/1
Clément DAVID [Wed, 5 Jul 2017 16:00:55 +0000 (18:00 +0200)]
Change-Id: I0ea3d06c9318f4eed8002b5196f6282909b4a5fa

scilab/CHANGES.md
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/actions/RegionToSuperblockAction.java

index bfc122d..963b133 100644 (file)
@@ -384,6 +384,7 @@ the [development mailing list](dev@lists.scilab.org) for a particular toolbox.
 * [#15138](http://bugzilla.scilab.org/show_bug.cgi?id=15138): Scilab crashes at start on Windows
 * [#15139](http://bugzilla.scilab.org/show_bug.cgi?id=15139): Fix return in try/catch expression
 * [#15144](http://bugzilla.scilab.org/show_bug.cgi?id=15144): `mean` and `stdev` could not be overloaded.
+* [#15185](http://bugzilla.scilab.org/show_bug.cgi?id=15185): "Region to Superblock" did not restore broken links.
 * [#15188](http://bugzilla.scilab.org/show_bug.cgi?id=15188): Failed to load hdf5_is_file function.
 * [#15205](http://bugzilla.scilab.org/show_bug.cgi?id=15205): `get_scicos_version` returned the previous version.
 
index 8528524..3a41b3f 100644 (file)
@@ -2,6 +2,7 @@
  * Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
  * Copyright (C) 2010 - DIGITEO - Clement DAVID
  * Copyright (C) 2011-2016 - Scilab Enterprises - Clement DAVID
+ * Copyright (C) 2017 - ESI Group - Clement DAVID
  *
  * Copyright (C) 2012 - 2016 - Scilab Enterprises
  *
@@ -416,11 +417,7 @@ public class RegionToSuperblockAction extends VertexSelectionDependantAction {
              */
             moveToChild(controller, parentGraph, superBlock, brokenLinks, toBeMoved);
 
-            /*
-             * Append the port to the superblock (in case of IN_f / OUT_f selected)
-             */
-            updateIO(controller, parentGraph, superBlock, toBeMoved);
-
+            // Update the superblock I/O blocks positionning
             BlockPositioning.updateBlockView(parentGraph, superBlock);
         } catch (ScilabInterpreterManagement.InterpreterException ex) {
             // Scilab seems to be blocked, just consume the exception at this point
@@ -451,6 +448,7 @@ public class RegionToSuperblockAction extends VertexSelectionDependantAction {
         VectorOfScicosID children = new VectorOfScicosID();
         controller.setObjectProperty(superBlock.getUID(), superBlock.getKind(), ObjectProperties.CHILDREN, children);
 
+
         /*
          * Place the super block
          */
@@ -759,22 +757,5 @@ public class RegionToSuperblockAction extends VertexSelectionDependantAction {
 
         controller.setObjectProperty(superBlock.getUID(), superBlock.getKind(), ObjectProperties.CHILDREN, children);
     }
-
-    /**
-     * Update the ports according to the IOBlocks moved
-     * @param controller the shared controller
-     * @param parent the parent diagram
-     * @param superblock the superblock cell
-     * @param toBeMoved the moved blocks
-     */
-    private void updateIO(JavaController controller, XcosDiagram parent, SuperBlock superblock, List<XcosCell> toBeMoved) {
-        Map<Object, Object> context = new HashMap<>();
-        XcosDiagram.UpdateSuperblockPortsTracker.updateContext(context, toBeMoved, controller);
-
-        syncPorts(controller, superblock, ObjectProperties.INPUTS, (List<ContextUpdate>) context.get(IN), parent);
-        syncPorts(controller, superblock, ObjectProperties.OUTPUTS, (List<ContextUpdate>) context.get(OUT), parent);
-        syncPorts(controller, superblock, ObjectProperties.EVENT_INPUTS, (List<ContextUpdate>) context.get(EIN), parent);
-        syncPorts(controller, superblock, ObjectProperties.EVENT_OUTPUTS, (List<ContextUpdate>) context.get(EOUT), parent);
-    }
 }
 // CSON: ClassFanOutComplexity