* Bug 15228 fixed - Graphics: get() error message fixed 27/19427/2
Paul Bignier [Sun, 24 Sep 2017 16:20:21 +0000 (09:20 -0700)]
Change-Id: Id8233f1eacae7c0ddaba36097bea753583d77c7c

scilab/CHANGES.md
scilab/modules/graphics/sci_gateway/cpp/sci_get.cpp
scilab/modules/graphics/tests/nonreg_tests/bug_15228.tst [new file with mode: 0644]

index 5f03d37..0979d64 100644 (file)
@@ -406,6 +406,7 @@ the [development mailing list](dev@lists.scilab.org) for a particular toolbox.
 * [#15199](http://bugzilla.scilab.org/show_bug.cgi?id=15199): Scilab crash when using fsolve for functions containing other functions.
 * [#15205](http://bugzilla.scilab.org/show_bug.cgi?id=15205): `get_scicos_version` returned the previous version.
 * [#15207](http://bugzilla.scilab.org/show_bug.cgi?id=15207): `xcos(scs_m)` did not work for sub-systems.
+* [#15228](http://bugzilla.scilab.org/show_bug.cgi?id=15228): `get` error message fixed
 * [#15239](http://bugzilla.scilab.org/show_bug.cgi?id=15239): The console's properties .tag and .userdata were not displayed nor documented.
 * [#15236](http://bugzilla.scilab.org/show_bug.cgi?id=15236): The `isglobal` help page was inaccurate. Examples were erroneous.
 * [#15260](http://bugzilla.scilab.org/show_bug.cgi?id=15260): `sci2exp` was broken for cells, structures, and all types of hypermatrix.
index 874045f..e515bfa 100644 (file)
@@ -71,7 +71,7 @@ types::Function::ReturnValue sci_get(types::typed_list &in, int _iRetCount, type
         types::Double* pDbll1 = p1->getAs<types::Double>();
         if (pDbll1->isScalar() == false)
         {
-            Scierror(202, _("%s: Wrong type for argument #%d: A real expected.\n"), "get", 1);
+            Scierror(202, _("%s: Wrong size for argument #%d: A real scalar expected.\n"), "get", 1);
             return types::Function::Error;
         }
 
@@ -105,7 +105,7 @@ types::Function::ReturnValue sci_get(types::typed_list &in, int _iRetCount, type
         types::String* pStr = in[1]->getAs<types::String>();
         if (pStr->isScalar() == false)
         {
-            Scierror(999, _("%s: Wrong type for input argument #%d: string expected.\n"), "get", 2);
+            Scierror(999, _("%s: Wrong size for input argument #%d: scalar string expected.\n"), "get", 2);
             return types::Function::Error;
         }
 
@@ -159,7 +159,7 @@ types::Function::ReturnValue sci_get(types::typed_list &in, int _iRetCount, type
 
         if (pS->isScalar() == false)
         {
-            Scierror(202, _("%s: Wrong type for argument #%d: string expected.\n"), "get", 2);
+            Scierror(202, _("%s: Wrong size for argument #%d: scalar string expected.\n"), "get", 2);
             return types::Function::Error;
         }
 
@@ -176,7 +176,7 @@ types::Function::ReturnValue sci_get(types::typed_list &in, int _iRetCount, type
         pS = in[0]->getAs<types::String>();
         if (pS->isScalar() == false)
         {
-            Scierror(202, _("%s: Wrong type for argument #%d: string expected.\n"), "get", 1);
+            Scierror(202, _("%s: Wrong size for argument #%d: scalar string expected.\n"), "get", 1);
             return types::Function::Error;
         }
 
@@ -212,7 +212,7 @@ types::Function::ReturnValue sci_get(types::typed_list &in, int _iRetCount, type
                 pS = in[1]->getAs<types::String>();
                 if (pS->isScalar() == false)
                 {
-                    Scierror(202, _("%s: Wrong type for argument #%d: string expected.\n"), "get", 2);
+                    Scierror(202, _("%s: Wrong size for argument #%d: scalar string expected.\n"), "get", 2);
                     return types::Function::Error;
                 }
             }
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_15228.tst b/scilab/modules/graphics/tests/nonreg_tests/bug_15228.tst
new file mode 100644 (file)
index 0000000..db06f78
--- /dev/null
@@ -0,0 +1,23 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2017 - ESI Group - Paul Bignier
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- NO CHECK REF -->
+// <-- TEST WITH GRAPHIC -->
+
+// <-- Non-regression test for bug 15228 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/15228
+//
+// <-- Short Description -->
+// get() returned a wrong error message
+
+plot2d()
+e = gce();
+e = e.children(1);
+refMsg = msprintf(_("%s: Wrong size for argument #%d: scalar string expected.\n"), "get", 2);
+assert_checkerror("get(e, [""background"" ""user_data""])", refMsg);