Do not fail uninstall process if step error occurred

Change-Id: Id4dc962699d08e202cbd84670b6265f2cfc9d4d3

diff --git a/src/common/step/step_remove_files.cc b/src/common/step/step_remove_files.cc
index 9663a51..30dc7cf 100644 (file)
--- a/src/common/step/step_remove_files.cc
+++ b/src/common/step/step_remove_files.cc
@@ -38,7 +38,6 @@ Step::Status StepRemoveFiles::process() {
   bf::path backup_path = GetBackupPathForPackagePath(context_->pkg_path.get());
   if (!MoveDir(context_->pkg_path.get(), backup_path)) {
     LOG(ERROR) << "Cannot remove widget files from its location";
-    return Status::APP_DIR_ERROR;
   }
   LOG(DEBUG) << "Removed directory: " << context_->pkg_path.get();
   return Status::OK;

diff --git a/src/common/step/step_unregister_app.cc b/src/common/step/step_unregister_app.cc
index 4986a0c..5f34b1d 100644 (file)
--- a/src/common/step/step_unregister_app.cc
+++ b/src/common/step/step_unregister_app.cc
@@ -38,11 +38,6 @@ Step::Status StepUnregisterApplication::precheck() {
     return Status::MANIFEST_NOT_FOUND;
   }
 
-  if (context_->backup_xml_path.get().empty()) {
-    LOG(ERROR) << "Backup xml path was not set but is required";
-    return Status::INVALID_VALUE;
-  }
-
   // TODO(p.sikorski) check context_->uid.get()
 
   return Step::Status::OK;
@@ -73,7 +68,6 @@ Step::Status StepUnregisterApplication::process() {
   // Prepare certificate info for rollback operations
   if (!BackupCertInfo()) {
     LOG(ERROR) << "Failed to backup cert info";
-    return Status::CERT_ERROR;
   }
 
   if (!UnregisterAppInPkgmgr(context_->manifest_data.get(),
@@ -82,7 +76,6 @@ Step::Status StepUnregisterApplication::process() {
                              context_->uid.get(),
                              context_->request_mode.get())) {
     LOG(ERROR) << "Failed to unregister package into database";
-    return Status::REGISTER_ERROR;
   }
 
   // remove manifest file