Bug 12455 fixed: format_n had no effects 54/11054/3
Calixte DENIZET [Wed, 27 Mar 2013 13:37:09 +0000 (14:37 +0100)]
Change-Id: I853cbabf86eb30b20bcb305711b032c371cf697a

scilab/CHANGES_5.4.X
scilab/modules/graphics/src/c/getHandleProperty/set_format_n_property.c
scilab/modules/graphics/tests/nonreg_tests/bug_4964.dia.ref
scilab/modules/graphics/tests/nonreg_tests/bug_4964.tst

index bf046d6..65367ca 100644 (file)
@@ -611,6 +611,8 @@ Bug fixes
 
 * Bug #12450 fixed - xlfont() did not return font names as specified in the documentation.
 
+* Bug #12455 fixed - format_n property had no effect on "axis" handles.
+
 
                     Changes between version 5.3.3 and 5.4.0
                     =======================================
index 930d206..13494e2 100644 (file)
@@ -5,6 +5,7 @@
  * Copyright (C) 2006 - INRIA - Jean-Baptiste Silvy
  * Copyright (C) 2009 - DIGITEO - Pierre Lando
  * Copyright (C) 2010 - DIGITEO - Manuel Juliachs
+ * Copyright (C) 2013 - Scilab Enterprises - Calixte DENIZET
  *
  * This file must be used under the terms of the CeCILL.
  * This source file is licensed as described in the file COPYING, which
@@ -29,6 +30,8 @@
 #include "localization.h"
 #include "MALLOC.h"
 #include "string.h"
+#include "StringMatrix.h"
+#include "Format.h"
 
 #include "setGraphicObjectProperty.h"
 #include "graphicObjectProperties.h"
@@ -38,18 +41,34 @@ int set_format_n_property(void* _pvCtx, char* pobjUID, size_t stackPointer, int
 {
     BOOL status = FALSE;
     char* format = NULL;
-    if ( !( valueType == sci_strings ) )
+    char * oldFormat = NULL;
+    StringMatrix * labels = NULL;
+
+    if (valueType != sci_strings)
     {
         Scierror(999, _("Wrong type for '%s' property: String expected.\n"), "format_n");
         return SET_PROPERTY_ERROR;
     }
 
     format = getStringFromStack(stackPointer);
+    getGraphicObjectProperty(pobjUID, __GO_FORMATN__, jni_string, (void **)&oldFormat);
 
-    status = setGraphicObjectProperty(pobjUID, __GO_FORMATN__, format, jni_string, 1);
+    if (strcmp(format, oldFormat) == 0)
+    {
+        return SET_PROPERTY_SUCCEED;
+    }
 
+    status = setGraphicObjectProperty(pobjUID, __GO_FORMATN__, format, jni_string, 1);
     if (status == TRUE)
     {
+        labels = computeDefaultTicsLabels(pobjUID);
+        if (labels != NULL)
+        {
+            char ** data = getStrMatData(labels);
+            setGraphicObjectProperty(pobjUID, __GO_TICKS_LABELS__, data, jni_string_vector, labels->nbCol * labels->nbRow);
+            deleteMatrix(labels);
+        }
+
         return SET_PROPERTY_SUCCEED;
     }
     else
index e1b0e1b..d2956e6 100644 (file)
 plot2d(1:10,1:10,1,strf="020")
 drawaxis(x=2:7,y=4,dir='u',tics='v')
 e=gce();
-e.format_n
- ans  =
-    
-e.format_n='%0.0fcm'
+assert_checkequal(e.format_n, "");
+// Check that the display does not crash
+e.format_n = "%0.0fcm"
  e  =
  
 Handle of type "Axis" with properties:
@@ -42,3 +40,6 @@ fractional_font = "off"
 clip_state = "off"
 clip_box = []
 user_data = []
+tag = 
+assert_checkequal(e.format_n, "%0.0fcm");
+assert_checkequal(e.tics_labels, string(2:7)+"cm");
index 4b6c13a..7484eb4 100644 (file)
 plot2d(1:10,1:10,1,strf="020")
 drawaxis(x=2:7,y=4,dir='u',tics='v')
 e=gce();
-e.format_n
-e.format_n='%0.0fcm'
+assert_checkequal(e.format_n, "");
+// Check that the display does not crash
+e.format_n = "%0.0fcm"
+assert_checkequal(e.format_n, "%0.0fcm");
+assert_checkequal(e.tics_labels, string(2:7)+"cm");
+