Bug 13404 fixed: Rect property was not taken into account in polarplot 25/14525/2
Calixte DENIZET [Tue, 13 May 2014 13:13:08 +0000 (15:13 +0200)]
Change-Id: I94b8d2ee63b0549314e93b4ca26429148eef4ca4

scilab/CHANGES_5.5.X
scilab/modules/graphics/macros/polarplot.sci
scilab/modules/graphics/tests/nonreg_tests/bug_13404.dia.ref [new file with mode: 0644]
scilab/modules/graphics/tests/nonreg_tests/bug_13404.tst [new file with mode: 0644]

index 04f03d7..4200b9e 100644 (file)
@@ -39,6 +39,8 @@ Scilab Bug Fixes
 
 * Bug #13403 fixed - Regression on axes labeling fixed.
 
+* Bug #13404 fixed - rect property was not taken into account in polarplot.
+
 * Bug #13409 fixed - permute(x, dims) failed when dims was greater than the dimensions of size(x)
                      now permute treats extra dimensions as 1
 
index 74330c1..89390c1 100644 (file)
@@ -161,9 +161,6 @@ function polarplot(theta,rho,style,strf,leg,rect)
     end
     drawlater()
     execstr("plot2d(x,y,"+strcat(opts,",")+")")
-    a=gca();
-    a.data_bounds=[rect(1:2);rect(3:4)]
-    a.margins=[0.07 0.07 0.12 0.07]
 
     fcolor=color("grey70");
     xset("dashes",1)
@@ -228,5 +225,10 @@ function polarplot(theta,rho,style,strf,leg,rect)
         e = gce();
         e.clip_state = "off";
     end
+
+    a=gca();
+    a.data_bounds=[rect(1:2);rect(3:4)]
+    a.margins=[0.07 0.07 0.12 0.07]
+
     drawnow()
 endfunction
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13404.dia.ref b/scilab/modules/graphics/tests/nonreg_tests/bug_13404.dia.ref
new file mode 100644 (file)
index 0000000..d38f8fa
--- /dev/null
@@ -0,0 +1,18 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2014 - Scilab Enterprises - Vincent COUVERT
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- TEST WITH GRAPHIC -->
+// <-- Non-regression test for bug 13404 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13404
+//
+// <-- Short Description -->
+// rect property was not taken into account in polarplot.
+t=linspace(0,1,100)*%pi*2;
+polarplot(t/4-1,sin(t/10)-0.3,rect=[-0.18 -0.02 0.27 0.3]);
+a=gca();
+assert_checkequal(a.data_bounds, [-0.18,-0.02;0.27,0.3]);
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13404.tst b/scilab/modules/graphics/tests/nonreg_tests/bug_13404.tst
new file mode 100644 (file)
index 0000000..7aba66e
--- /dev/null
@@ -0,0 +1,21 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2014 - Scilab Enterprises - Vincent COUVERT
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- TEST WITH GRAPHIC -->
+
+// <-- Non-regression test for bug 13404 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13404
+//
+// <-- Short Description -->
+// rect property was not taken into account in polarplot.
+
+t=linspace(0,1,100)*%pi*2;
+polarplot(t/4-1,sin(t/10)-0.3,rect=[-0.18 -0.02 0.27 0.3]);
+a=gca();
+assert_checkequal(a.data_bounds, [-0.18,-0.02;0.27,0.3]);
\ No newline at end of file