From: Woochan Lee <wc0917.lee@samsung.com>
Date: Wed, 23 Aug 2017 11:40:25 +0000 (+0900)
Subject: elm_spinner: Support min,max filter for float format case.
X-Git-Tag: accepted/tizen/4.0/unified/20170901.203246~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=cd93bb0d091944f5354ba9e687d8cb8d0f24a44f;p=platform%2Fupstream%2Felementary.git

elm_spinner: Support min,max filter for float format case.

Summary: The previous min, max filter doesn't support float format case.

Test Plan: elementary_test -> spinner.

Reviewers: jpeg, cedric, woohyun, myoungwoon

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D5016

Change-Id: I60f97c1e8e27f9c3d09f85a5a4e0ccbb347d28e2
---

diff --git a/src/lib/elm_spinner.c b/src/lib/elm_spinner.c
index 7c6e7ec..e71b5b9 100644
--- a/src/lib/elm_spinner.c
+++ b/src/lib/elm_spinner.c
@@ -83,13 +83,6 @@ EO_CALLBACKS_ARRAY_DEFINE(_inc_dec_button_cb,
 static void _access_increment_decrement_info_say(Evas_Object *obj,
                                                  Eina_Bool is_incremented);
 
-typedef enum _Elm_Spinner_Format_Type
-{
-   SPINNER_FORMAT_FLOAT,
-   SPINNER_FORMAT_INT,
-   SPINNER_FORMAT_INVALID
-} Elm_Spinner_Format_Type;
-
 static Eina_Bool
 _is_valid_digit(char x)
 {
@@ -597,8 +590,8 @@ _text_insert(const char *text, const char *input, int pos)
 static void
 _min_max_validity_filter(void *data, Evas_Object *obj, char **text)
 {
-   const char *str, *new_str;
-   char *insert;
+   const char *str, *point;
+   char *insert, *new_str = NULL;
    double val;
    int max_len, len;
 
@@ -614,20 +607,32 @@ _min_max_validity_filter(void *data, Evas_Object *obj, char **text)
    insert = *text;
    new_str = _text_insert(str, insert, elm_entry_cursor_pos_get(obj));
    if (!new_str) return;
-
    max_len = log10(fabs(sd->val_max)) + 1;
-   len = evas_string_char_len_get(new_str);
-   if (len < max_len)
+
+   if (sd->format_type == SPINNER_FORMAT_INT)
      {
-        ELM_SAFE_FREE(new_str, free);
-        return;
+        len = strlen(new_str);
+        if (len < max_len) goto end;
+     }
+   else if (sd->format_type == SPINNER_FORMAT_FLOAT)
+     {
+        point = strchr(new_str, '.');
+        if (point)
+          {
+             if ((int) strlen(point + 1) > sd->decimal_points)
+               {
+                  *insert = 0;
+                  goto end;
+               }
+          }
      }
 
    val = strtod(new_str, NULL);
-   ELM_SAFE_FREE(new_str, free);
-
    if ((val < sd->val_min) || (val > sd->val_max))
      *insert = 0;
+
+end:
+   free(new_str);
 }
 
 static void
@@ -1614,22 +1619,22 @@ EOLIAN static void
 _elm_spinner_label_format_set(Eo *obj, Elm_Spinner_Data *sd, const char *fmt)
 {
    Elm_Spinner_Format_Type type;
-   if (fmt)
+
+   if (!fmt) fmt = "%.0f";
+   type = _is_label_format_integer(fmt);
+   if (type == SPINNER_FORMAT_INVALID)
      {
-        type = _is_label_format_integer(fmt);
-        if (type == SPINNER_FORMAT_INVALID)
-          {
-             ERR("format:\"%s\" is Invalid, cannot be set", fmt);
-             return;
-          }
-        else if (type == SPINNER_FORMAT_FLOAT)
-          {
-             sd->decimal_points = _decimal_points_get(fmt);
-          }
+        ERR("format:\"%s\" is invalid, cannot be set", fmt);
+        return;
+     }
+   else if (type == SPINNER_FORMAT_FLOAT)
+     {
+        sd->decimal_points = _decimal_points_get(fmt);
      }
 
    eina_stringshare_replace(&sd->label, fmt);
 
+   sd->format_type = type;
    _label_write(obj);
    elm_layout_sizing_eval(obj);
    _entry_accept_filter_add(obj);
diff --git a/src/lib/elm_widget_spinner.h b/src/lib/elm_widget_spinner.h
index 7342313..ee0b61a 100644
--- a/src/lib/elm_widget_spinner.h
+++ b/src/lib/elm_widget_spinner.h
@@ -26,6 +26,13 @@
 /**
  * Base layout smart data extended with spinner instance data.
  */
+typedef enum _Elm_Spinner_Format_Type
+{
+   SPINNER_FORMAT_FLOAT,
+   SPINNER_FORMAT_INT,
+   SPINNER_FORMAT_INVALID
+} Elm_Spinner_Format_Type;
+
 typedef struct _Elm_Spinner_Data    Elm_Spinner_Data;
 struct _Elm_Spinner_Data
 {
@@ -42,6 +49,7 @@ struct _Elm_Spinner_Data
    Ecore_Timer          *longpress_timer; /**< a timer to detect long press. After lonpress timeout,
                                           start continuous change of values until mouse up */
    Eina_List            *special_values;
+   Elm_Spinner_Format_Type format_type;
 
    Eina_Bool             entry_visible : 1;
    //TIZEN_ONLY(20160623): Add entry visible, focus check flag for reactivating entry.