* Bug #11554 fixed - After using xcosPalAdd, a message was printed on 'quit'. 92/8592/4
Clément DAVID [Tue, 7 Aug 2012 09:55:54 +0000 (11:55 +0200)]
Change-Id: Ia0ac3ebd78198e73432a5204527abc303f7026d9

scilab/CHANGES_5.4.X
scilab/modules/xcos/src/java/org/scilab/modules/xcos/Xcos.java
scilab/modules/xcos/tests/nonreg_tests/bug_11554.dia.ref [new file with mode: 0644]
scilab/modules/xcos/tests/nonreg_tests/bug_11554.tst [new file with mode: 0644]

index 8e37d6f..0d8cd42 100644 (file)
@@ -36,6 +36,8 @@ Bug Fixes
 
 * Bug #11509 fixed - A progression bar was missing during Atoms operations.
 
+* Bug #11554 fixed - After using xcosPalAdd, a message was printed on 'quit'.
+
 * Bug #11592 fixed - In the documentation, example of bessel failed.
 
 * Bug #11597 fixed - On Mac OS X, the Preferences windows was not accessible
index 916f011..05ad3aa 100644 (file)
@@ -748,7 +748,7 @@ public final class Xcos {
             instance.addDiagram(null, null);
 
             /* terminate any remaining simulation */
-            InterpreterManagement.putCommandInScilabQueue("haltscicos");
+            InterpreterManagement.putCommandInScilabQueue("if isdef('haltscicos'), then haltscicos(), end;");
 
             /* Saving modified data */
             instance.palette.saveConfig();
diff --git a/scilab/modules/xcos/tests/nonreg_tests/bug_11554.dia.ref b/scilab/modules/xcos/tests/nonreg_tests/bug_11554.dia.ref
new file mode 100644 (file)
index 0000000..834839c
--- /dev/null
@@ -0,0 +1,37 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2012 - Scilab Enterprises - Clément DAVID
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- TEST WITH XCOS -->
+//
+// <-- Non-regression test for bug 11554 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=11554
+//
+// <-- Short Description -->
+// quit report an error after loading xcos_toolbox_skeleton.
+// avoid building the xcos toolbox skeleton by calling xcosPalAdd(...)
+loadXcosLibs();
+function [x,y,typ]=MY_BLOCK(job,arg1,arg2)
+    x=[];y=[];typ=[];
+    select job
+    case 'set' then
+        x=arg1;
+    case 'define' then
+        model=scicos_model()
+        model.sim=list('my_block',4)
+        exprs=''
+        gr_i=''
+        x=standard_define([3 2],model,exprs,gr_i)
+    end
+endfunction
+block_img = SCI + "/modules/xcos/images/blocks/RAMP.svg";
+pal_block_img = SCI + "/modules/xcos/images/palettes/RAMP.png";
+pal = xcosPal("My palette");
+pal = xcosPalAddBlock(pal, 'MY_BLOCK', pal_block_img, block_img);
+xcosPalAdd(pal);
+quit();
+exec('SCI/etc/scilab.quit',-1);quit;
diff --git a/scilab/modules/xcos/tests/nonreg_tests/bug_11554.tst b/scilab/modules/xcos/tests/nonreg_tests/bug_11554.tst
new file mode 100644 (file)
index 0000000..165de1e
--- /dev/null
@@ -0,0 +1,44 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2012 - Scilab Enterprises - Clément DAVID
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- TEST WITH XCOS -->
+//
+// <-- Non-regression test for bug 11554 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=11554
+//
+// <-- Short Description -->
+// quit report an error after loading xcos_toolbox_skeleton.
+
+// avoid building the xcos toolbox skeleton by calling xcosPalAdd(...)
+
+loadXcosLibs();
+
+function [x,y,typ]=MY_BLOCK(job,arg1,arg2)
+    x=[];y=[];typ=[];
+    select job
+    case 'set' then
+        x=arg1;
+    case 'define' then
+        model=scicos_model()
+        model.sim=list('my_block',4)
+        exprs=''
+        gr_i=''
+        x=standard_define([3 2],model,exprs,gr_i)
+    end
+endfunction
+block_img = SCI + "/modules/xcos/images/blocks/RAMP.svg";
+pal_block_img = SCI + "/modules/xcos/images/palettes/RAMP.png";
+
+
+
+pal = xcosPal("My palette");
+pal = xcosPalAddBlock(pal, 'MY_BLOCK', pal_block_img, block_img);
+xcosPalAdd(pal);
+
+quit();