Bug 13313 fixed: Set orientation did not disabled auto_orientation 07/14507/2
Calixte DENIZET [Tue, 6 May 2014 10:13:42 +0000 (12:13 +0200)]
Change-Id: I93884abd0ca6d6be94396309b96bb3dbd7efcb03

scilab/CHANGES_5.5.X
scilab/modules/graphics/src/c/getHandleProperty/set_datatip_property.c
scilab/modules/graphics/tests/nonreg_tests/bug_13313.dia.ref [new file with mode: 0644]
scilab/modules/graphics/tests/nonreg_tests/bug_13313.tst [new file with mode: 0644]

index 7b24e5f..1629d20 100644 (file)
@@ -9,6 +9,9 @@ Scilab Bug Fixes
 * Bug #13291 fixed - xmltojar([],[],'ja_JP') might lead to a crash when the
                      locale was not available system-wide.
 
+* Bug #13313 fixed - Setting datatips orientation did not disable
+                     auto_orientation mode.
+
 * Bug #13333 fixed - Selecting "Offline mode" in Scilab installer installed
                      Scilab in "Command line mode" (without GUI modules).
 
index 0b10724..b56828f 100755 (executable)
@@ -41,6 +41,8 @@ int set_tip_orientation_property(void* _pvCtx, int iObj, void* _pvData, int valu
 
     if (status == TRUE)
     {
+        int auto_orientation = 0;
+        setGraphicObjectProperty(iObj, __GO_DATATIP_AUTOORIENTATION__, &auto_orientation, jni_bool, 1);
         return SET_PROPERTY_SUCCEED;
     }
     else
@@ -193,4 +195,4 @@ int set_tip_disp_function_property(void* _pvCtx, int iObj, void* _pvData, int va
         Scierror(999, _("'%s' property does not exist for this handle.\n"), "display_function");
         return SET_PROPERTY_ERROR;
     }
-}
\ No newline at end of file
+}
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13313.dia.ref b/scilab/modules/graphics/tests/nonreg_tests/bug_13313.dia.ref
new file mode 100644 (file)
index 0000000..04dd841
--- /dev/null
@@ -0,0 +1,24 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2014 - Scilab Enterprises - Calixte DENIZET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- TEST WITH GRAPHIC -->
+// <-- Non-regression test for bug 13313 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13313
+//
+// <-- Short Description -->
+// Set orientation did not disabled auto_orientation
+clf
+plot(1:100)
+e = gce();
+c = e.children;
+d = datatipCreate(c, [73 73]);
+assert_checkequal(d.auto_orientation, "on");
+assert_checkequal(d.orientation, 1);
+d.orientation = 2;
+assert_checkequal(d.orientation, 2);
+assert_checkequal(d.auto_orientation, "off");
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13313.tst b/scilab/modules/graphics/tests/nonreg_tests/bug_13313.tst
new file mode 100644 (file)
index 0000000..28e6ac8
--- /dev/null
@@ -0,0 +1,28 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2014 - Scilab Enterprises - Calixte DENIZET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- TEST WITH GRAPHIC -->
+
+// <-- Non-regression test for bug 13313 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13313
+//
+// <-- Short Description -->
+// Set orientation did not disabled auto_orientation
+
+clf
+plot(1:100)
+e = gce();
+c = e.children;
+d = datatipCreate(c, [73 73]);
+
+assert_checkequal(d.auto_orientation, "on");
+assert_checkequal(d.orientation, 1);
+d.orientation = 2;
+assert_checkequal(d.orientation, 2);
+assert_checkequal(d.auto_orientation, "off");