Issue #170: set version to status when appmanager recv install comman…

…d from user, Compatibility with previous incorrect state settings when uninstall App
buckyos · Apr 13, 2023 · 314d358 · 314d358
1 parent c8de656
commit 314d358
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/service/app-manager/src/app_cmd_executor.rs b/src/service/app-manager/src/app_cmd_executor.rs
@@ -307,6 +307,7 @@ impl AppCmdExecutor {
             error!("{}", err);
             return Err(BuckyError::from((BuckyErrorCode::InvalidParam, err)));
         }
+
         status.lock().unwrap().set_version(&ver);
 
         self.pre_change_status(
@@ -437,7 +438,7 @@ impl AppCmdExecutor {
         {
             let status = status.lock().unwrap();
             web_id = status.web_dir().cloned();
-            ver = status.version().unwrap().to_owned();
+            ver = status.version().unwrap_or("noversion").to_owned();
         }
 
         let mut target_status_code = AppLocalStatusCode::Uninstalled;

diff --git a/src/service/app-manager/src/app_manager_ex.rs b/src/service/app-manager/src/app_manager_ex.rs
@@ -847,6 +847,9 @@ impl AppManager {
                 cmd_code, status_code, next_status_code, app_id, cmd_group_code
             );
             status.set_status(next_status_code);
+            if let CmdCode::Install(install) = cmd_code {
+                status.set_version(&install.ver);
+            }
             status_clone = status.clone();
         }