[Notification] Fix display of number attribute in the notification 09/88809/1
authorJakub Skowron <j.skowron@samsung.com>
Tue, 20 Sep 2016 14:39:40 +0000 (16:39 +0200)
committerJakub Skowron <j.skowron@samsung.com>
Tue, 20 Sep 2016 14:39:40 +0000 (16:39 +0200)
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>
src/notification/status_notification.cc

index cc0e1357b88f1c6546b6c38255c47780f60ceb5a..13fe62c40797c001dba39144a5d305dac6b663fe 100644 (file)
@@ -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;
     }