[Systemsetting] Added logs in methods onEntered and onReturn
authorPiotr Kosko <p.kosko@samsung.com>
Wed, 27 May 2015 12:11:48 +0000 (14:11 +0200)
committerHyunjin Park <hj.na.park@samsung.com>
Mon, 1 Jun 2015 15:46:15 +0000 (00:46 +0900)
Change-Id: I08736a1ad30e2363998c01cd01725a6bce416b54
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>
src/systemsetting/systemsetting_instance.cc

index d6be5f4d96c2ae08855dcf630d2292f07273947d..f6f35950b146a2db4f6be59495fe6ce868f121a4 100644 (file)
@@ -40,6 +40,7 @@ using namespace extension::systemsetting;
 
 SystemSettingInstance::SystemSettingInstance()
 {
+  LoggerD("Enter");
   using std::placeholders::_1;
   using std::placeholders::_2;
 
@@ -54,11 +55,12 @@ SystemSettingInstance::SystemSettingInstance()
 
 SystemSettingInstance::~SystemSettingInstance()
 {
+  LoggerD("Enter");
 }
 
 void SystemSettingInstance::getProperty(const picojson::value& args, picojson::object& out)
 {
-  LoggerD("");
+  LoggerD("Enter");
   const double callback_id = args.get("callbackId").get<double>();
 
   const std::string& type = args.get("type").get<std::string>();
@@ -68,10 +70,12 @@ void SystemSettingInstance::getProperty(const picojson::value& args, picojson::o
     LoggerD("Getting platform value");
     picojson::value result = picojson::value(picojson::object());
     PlatformResult status = getPlatformPropertyValue(type, &result);
-    if(status.IsSuccess())
+    if(status.IsSuccess()) {
       ReportSuccess(result, response->get<picojson::object>());
-    else
+    } else {
+      LoggerE("Failed: getPlatformPropertyValue()");
       ReportError(status, &response->get<picojson::object>());
+    }
   };
 
   auto get_response = [this, callback_id](const std::shared_ptr<picojson::value>& response) -> void {
@@ -88,6 +92,7 @@ void SystemSettingInstance::getProperty(const picojson::value& args, picojson::o
 PlatformResult SystemSettingInstance::getPlatformPropertyValue(
     const std::string& settingType,
     picojson::value* out) {
+  LoggerD("Enter");
   picojson::object& result_obj = out->get<picojson::object>();
 
   int ret;
@@ -129,7 +134,7 @@ PlatformResult SystemSettingInstance::getPlatformPropertyValue(
 
 void SystemSettingInstance::setProperty(const picojson::value& args, picojson::object& out)
 {
-  LoggerD("");
+  LoggerD("Enter");
 
   const double callback_id = args.get("callbackId").get<double>();
 
@@ -142,10 +147,12 @@ void SystemSettingInstance::setProperty(const picojson::value& args, picojson::o
   auto get = [this, type, value](const std::shared_ptr<picojson::value>& response) -> void {
     LoggerD("Setting platform value");
     PlatformResult status = setPlatformPropertyValue(type, value);
-    if (status.IsSuccess())
+    if (status.IsSuccess()) {
       ReportSuccess(response->get<picojson::object>());
-    else
+    } else {
+      LoggerE("Failed: setPlatformPropertyValue()");
       ReportError(status, &response->get<picojson::object>());
+    }
   };
 
   auto get_response = [this, callback_id](const std::shared_ptr<picojson::value>& response) -> void {
@@ -162,6 +169,7 @@ void SystemSettingInstance::setProperty(const picojson::value& args, picojson::o
 PlatformResult SystemSettingInstance::setPlatformPropertyValue(
     const std::string& settingType,
     const std::string& settingValue) {
+  LoggerD("Enter");
   int ret;
   if (settingType == SETTING_HOME_SCREEN) {
     ret = system_settings_set_value_string(