* Bug #14228 fixed - Setting rotation_angles property to a matrix of any size did... 48/17448/2
Charlotte HECQUET [Thu, 12 Nov 2015 15:55:55 +0000 (16:55 +0100)]
Change-Id: I2d0b64ff019ae099e8cb0f9d2cddb148bf0e45b7

scilab/CHANGES_6.0.X
scilab/modules/graphics/src/c/getHandleProperty/set_rotation_angles_property.c
scilab/modules/graphics/tests/nonreg_tests/bug_14228.dia.ref [new file with mode: 0644]
scilab/modules/graphics/tests/nonreg_tests/bug_14228.tst [new file with mode: 0644]

index 3ad999c..53b800c 100644 (file)
@@ -221,6 +221,8 @@ Bug Fixes
 
 * Bug #14225 fixed - command-line option "-quit" should set the processs Exit status
 
+* Bug #14228 fixed - Setting rotation_angles property to a matrix of any size did not return error message.
+
 * Bug #14232 fixed - Typos in Xcos.
 
 
index 8f81d78..e71b5dd 100644 (file)
@@ -40,6 +40,12 @@ int set_rotation_angles_property(void* _pvCtx, int iObjUID, void* _pvData, int v
         Scierror(999, _("Wrong type for '%s' property: Real matrix expected.\n"), "rotation_angles");
         return SET_PROPERTY_ERROR;
     }
+    
+    if (nbRow != 1 || nbCol != 2)
+    {
+        Scierror(999, _("Wrong size for '%s' property: Row vector of size %d expected.\n"), "rotation_angles", 2);
+        return SET_PROPERTY_ERROR;
+    }
 
     status = setGraphicObjectProperty(iObjUID, __GO_ROTATION_ANGLES__, values, jni_double_vector, 2);
 
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_14228.dia.ref b/scilab/modules/graphics/tests/nonreg_tests/bug_14228.dia.ref
new file mode 100644 (file)
index 0000000..8206c2c
--- /dev/null
@@ -0,0 +1,16 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2015 - Scilab Enterprises - Charlotte HECQUET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- TEST WITH GRAPHIC -->
+// <-- Non-regression test for bug 14228 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/14228
+//
+// <-- Short Description -->
+// Setting rotation_angles property to a matrix of any size does not return an error message 
+errmsg = msprintf(_("Wrong size for ''%s'' property: Row vector of size %d expected.\n"), "rotation_angles", 2);
+assert_checkerror("plot2d(); a=gca(); a.rotation_angles=[0]", errmsg);
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_14228.tst b/scilab/modules/graphics/tests/nonreg_tests/bug_14228.tst
new file mode 100644 (file)
index 0000000..3e64b4f
--- /dev/null
@@ -0,0 +1,19 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2015 - Scilab Enterprises - Charlotte HECQUET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- TEST WITH GRAPHIC -->
+
+// <-- Non-regression test for bug 14228 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/14228
+//
+// <-- Short Description -->
+// Setting rotation_angles property to a matrix of any size does not return an error message 
+
+errmsg = msprintf(_("Wrong size for ''%s'' property: Row vector of size %d expected.\n"), "rotation_angles", 2);
+assert_checkerror("plot2d(); a=gca(); a.rotation_angles=[0]", errmsg);