* Bug #11776 fixed - CMSCOPE did not take into account label&Id parameter. 47/12647/2
Alexandre HERISSE [Tue, 24 Sep 2013 14:35:43 +0000 (16:35 +0200)]
Change-Id: Id403d62edab285ae315db9e7b4c7ea6f6b96d701

scilab/CHANGES_5.5.X
scilab/modules/scicos_blocks/src/c/cmscope.c
scilab/modules/scicos_blocks/src/c/cscope.c
scilab/modules/xcos/tests/nonreg_tests/bug_11776.dia.ref [new file with mode: 0644]
scilab/modules/xcos/tests/nonreg_tests/bug_11776.tst [new file with mode: 0644]
scilab/modules/xcos/tests/nonreg_tests/bug_cmscope.zcos [new file with mode: 0644]
scilab/modules/xcos/tests/nonreg_tests/bug_cscope.zcos [new file with mode: 0644]

index 49fb460..e183ee9 100644 (file)
@@ -156,6 +156,8 @@ Xcos
 * Bug #11518 fixed - CLR block-text was displayed out of the bounding box
                      when zooming.
 
+* Bug #11776 fixed - CMSCOPE did not take into account label&Id parameter.
+
 * Bug #12359 fixed - Xcos files have been converted to ZCOS to gain some space.
 
 * Bug #12384 fixed - Using a Modelica part linked with an explicit link to
index f5ce37d..0d7033a 100644 (file)
@@ -841,6 +841,11 @@ static char const* getFigure(scicos_block * block)
             setPolylinesBounds(block, i, 0);
         }
     }
+    else
+    {
+        // set configured parameters
+        setFigureSettings(pFigureUID, block);
+    }
 
     if (sco->scope.cachedFigureUID == NULL)
     {
index d19dd1c..8f7f759 100644 (file)
@@ -754,6 +754,11 @@ static char const* getFigure(scicos_block * block)
             setPolylinesBounds(block, i, 0);
         }
     }
+    else
+    {
+        // set configured parameters
+        setFigureSettings(pFigureUID, block);
+    }
 
     if (sco->scope.cachedFigureUID == NULL)
     {
diff --git a/scilab/modules/xcos/tests/nonreg_tests/bug_11776.dia.ref b/scilab/modules/xcos/tests/nonreg_tests/bug_11776.dia.ref
new file mode 100644 (file)
index 0000000..a436c01
--- /dev/null
@@ -0,0 +1,24 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2013 - Scilab Enterprises - Alexandre HERISSE
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- XCOS TEST -->
+//
+// <-- Non-regression test for bug 11776 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=11776
+//
+// <-- Short Description -->
+// CMSCOPE did not take into account label&Id parameter.
+assert_checktrue(importXcosDiagram(SCI + "/modules/xcos/tests/nonreg_tests/bug_cmscope.zcos"));
+Info = xcos_simulate(scs_m, 4);
+f = gcf();
+assert_checkequal(f.figure_name, "CmscopE");
+close(f);
+assert_checktrue(importXcosDiagram(SCI + "/modules/xcos/tests/nonreg_tests/bug_cscope.zcos"));
+Info = xcos_simulate(scs_m, 4);
+f = gcf();
+assert_checkequal(f.figure_name, "CscopE");
diff --git a/scilab/modules/xcos/tests/nonreg_tests/bug_11776.tst b/scilab/modules/xcos/tests/nonreg_tests/bug_11776.tst
new file mode 100644 (file)
index 0000000..d4d4066
--- /dev/null
@@ -0,0 +1,30 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2013 - Scilab Enterprises - Alexandre HERISSE
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- XCOS TEST -->
+//
+// <-- Non-regression test for bug 11776 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=11776
+//
+// <-- Short Description -->
+// CMSCOPE did not take into account label&Id parameter.
+
+assert_checktrue(importXcosDiagram(SCI + "/modules/xcos/tests/nonreg_tests/bug_cmscope.zcos"));
+Info = xcos_simulate(scs_m, 4);
+
+f = gcf();
+assert_checkequal(f.figure_name, "CmscopE");
+close(f);
+
+assert_checktrue(importXcosDiagram(SCI + "/modules/xcos/tests/nonreg_tests/bug_cscope.zcos"));
+Info = xcos_simulate(scs_m, 4);
+
+f = gcf();
+assert_checkequal(f.figure_name, "CscopE");
+
diff --git a/scilab/modules/xcos/tests/nonreg_tests/bug_cmscope.zcos b/scilab/modules/xcos/tests/nonreg_tests/bug_cmscope.zcos
new file mode 100644 (file)
index 0000000..fdd355a
Binary files /dev/null and b/scilab/modules/xcos/tests/nonreg_tests/bug_cmscope.zcos differ
diff --git a/scilab/modules/xcos/tests/nonreg_tests/bug_cscope.zcos b/scilab/modules/xcos/tests/nonreg_tests/bug_cscope.zcos
new file mode 100644 (file)
index 0000000..9c44ef8
Binary files /dev/null and b/scilab/modules/xcos/tests/nonreg_tests/bug_cscope.zcos differ