Fix error logging when adding/removing/replacing step 10/207710/6
authorSangyoon Jang <jeremy.jang@samsung.com>
Tue, 11 Jun 2019 08:41:27 +0000 (17:41 +0900)
committerSangyoon Jang <jeremy.jang@samsung.com>
Thu, 20 Jun 2019 11:44:14 +0000 (11:44 +0000)
HandleStepError is for sending signal when the step returns fail.

Change-Id: I4ba079aee5a399e11079950954c30e42583f3371
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
src/common/app_installer.h

index e2589b3..ea7817f 100644 (file)
@@ -80,9 +80,8 @@ class AppInstaller {
           return step_name == s->name();
         });
     if (steps_.end() == it) {
-      HandleStepError(Step::Status::INVALID_VALUE,
-            "Can't find the step : " + step_name);
-        return;
+      LOG(ERROR) << "Can't find the step : " << step_name;
+      return;
     }
     steps_.erase(it);
   }
@@ -134,8 +133,7 @@ class AppInstaller {
         });
 
     if (steps_.end() == it) {
-      HandleStepError(Step::Status::INVALID_VALUE,
-          "Can't find the step : " + step_name);
+      LOG(ERROR) << "Can't find the step : " << step_name;
       return;
     }
 
@@ -166,8 +164,7 @@ class AppInstaller {
         });
 
     if (steps_.end() == it) {
-      HandleStepError(Step::Status::INVALID_VALUE,
-          "Can't find the step : " + step_name);
+      LOG(ERROR) << "Can't find the step : " << step_name;
       return;
     }
 
@@ -198,8 +195,7 @@ class AppInstaller {
         });
 
     if (steps_.end() == it) {
-      HandleStepError(Step::Status::INVALID_VALUE,
-          "Can't find the step : " + step_name);
+      LOG(ERROR) << "Can't find the step : " << step_name;
       return;
     }