From: Jakub Skowron <j.skowron@samsung.com>
Date: Tue, 20 Sep 2016 14:39:40 +0000 (+0200)
Subject: [Notification] Fix display of number attribute in the notification
X-Git-Tag: submit/tizen/20160920.235405~2^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=17bb2a05624a2d9630c43e158ae8bc637c5580fc;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[Notification] Fix display of number attribute in the notification

std::to_string uses %f formatting (i.e x.xxxxxx) for doubles
Explicitly cast number to long, as defined:

   attribute long? number;

Change-Id: I8aacb5ca80b9b5fc9cc01452a1c227ac2da26e65
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>
---

diff --git a/src/notification/status_notification.cc b/src/notification/status_notification.cc
index cc0e1357..13fe62c4 100644
--- a/src/notification/status_notification.cc
+++ b/src/notification/status_notification.cc
@@ -1213,10 +1213,8 @@ PlatformResult StatusNotification::FromJson(const picojson::object& args,
   }
 
   if (val.contains("number") && !IsNull(noti_obj, "number")) {
-    const std::string& value_str =
-        std::to_string(common::FromJson<double>(noti_obj, "number"));
-    status =
-        SetText(noti_handle, NOTIFICATION_TEXT_TYPE_EVENT_COUNT, value_str);
+    long number = (long)common::FromJson<double>(noti_obj, "number");
+    status = SetText(noti_handle, NOTIFICATION_TEXT_TYPE_EVENT_COUNT, std::to_string(number));
     if (status.IsError()) {
       return status;
     }