* Bug #13758 fixed - x_mdialog interpreted 'F' and 'T' as boolean values 19/16019/4
Clément DAVID [Tue, 24 Feb 2015 09:59:53 +0000 (10:59 +0100)]
                     whereas it should interpret '%F' and '%T' instead.

Change-Id: I8124d49d827f9b62c079fa0089fe6acc44900d69

scilab/CHANGES_5.5.X
scilab/modules/gui/help/en_US/x_mdialog.xml
scilab/modules/gui/src/java/org/scilab/modules/gui/bridge/messagebox/SwingScilabMessageBox.java
scilab/modules/xcos/tests/nonreg_tests/bug_13758.tst [new file with mode: 0644]
scilab/modules/xcos/tests/nonreg_tests/bug_13758.zcos [new file with mode: 0644]

index 4c75328..dbac1cf 100644 (file)
@@ -94,6 +94,9 @@ Scilab Bug Fixes
 
 * Bug #13752 fixed - The 'Supported compilers' page was outdated.
 
+* Bug #13758 fixed - x_mdialog interpreted 'F' and 'T' as boolean values
+                     whereas it should interpret '%F' and '%T' instead.
+
 * Bug #13772 fixed - Xcos GUI was not locked while setting parameters.
 
 
index a9590e2..bceb009 100644 (file)
@@ -101,7 +101,7 @@ rep = x_mdialog('Enter a matrix',...
  ]]></programlisting>
         <programlisting role="example"><![CDATA[
 sz = [10,10];
-default_input_matrix = string(rand(sz(1), sz(2)) > 0.5);
+default_input_matrix = "%"+string(rand(sz(1), sz(2)) > 0.5);
 labelsv = string(1:sz(1));
 labelsh = string(1:sz(2));
 rep = x_mdialog('Enter a boolean matrix', labelsv, labelsh, default_input_matrix)
index bc5d522..f60e678 100644 (file)
@@ -459,10 +459,10 @@ public class SwingScilabMessageBox extends JDialog implements SimpleMessageBox,
                     final String initial = defaultInput[col * lineLabels.length + line];
 
                     final Component c;
-                    if ("T".equalsIgnoreCase(initial)) {
+                    if ("%T".equalsIgnoreCase(initial)) {
                         c = new JCheckBox();
                         ((JCheckBox) c).setSelected(true);
-                    } else if ("F".equalsIgnoreCase(initial)) {
+                    } else if ("%F".equalsIgnoreCase(initial)) {
                         c = new JCheckBox();
                         ((JCheckBox) c).setSelected(false);
                     } else {
diff --git a/scilab/modules/xcos/tests/nonreg_tests/bug_13758.tst b/scilab/modules/xcos/tests/nonreg_tests/bug_13758.tst
new file mode 100644 (file)
index 0000000..3d42333
--- /dev/null
@@ -0,0 +1,23 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2015 - Scilab Enterprises - Vincent COUVERT
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- XCOS TEST -->
+// <-- INTERACTIVE TEST -->
+//
+// <-- Non-regression test for bug 13758 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=13758
+//
+// <-- Short Description -->
+// x_mdialog interpreted 'F' and 'T' as boolean values
+// whereas it should interpret '%F' and '%T' instead.
+
+xcos("SCI/modules/xcos/tests/nonreg_tests/bug_13758.zcos");
+
+// Double-clic on SampleCLK block (top left block)
+// In parameters dialog, check that "Sample time" is a text field containing T and not a checkbox.
\ No newline at end of file
diff --git a/scilab/modules/xcos/tests/nonreg_tests/bug_13758.zcos b/scilab/modules/xcos/tests/nonreg_tests/bug_13758.zcos
new file mode 100644 (file)
index 0000000..db2888f
Binary files /dev/null and b/scilab/modules/xcos/tests/nonreg_tests/bug_13758.zcos differ