chg: improve backup/restore error messages

- Remove duplicated text
- Update error message when trying to restore a backup when the phone is not connected
- Add the backup name in the error message when trying to restore a deleted backup

src/core/save.rs

@@ -66,10 +66,7 @@ pub async fn backup_phone(
                 Err(err) => Err(err.to_string()),
             }
         }
-        Err(err) => {
-            error!("[BACKUP]: {}", err);
-            Err(err.to_string())
-        }
+        Err(err) => Err(err.to_string()),
     }
 }
 
@@ -173,6 +170,6 @@ pub fn restore_backup(
             }
             Ok(commands)
         }
-        Err(e) => Err("[BACKUP]: ".to_owned() + &e.to_string()),
+        Err(e) => Err(e.to_string()),
     }
 }

src/gui/views/settings.rs

@@ -2,7 +2,7 @@ use crate::core::config::{BackupSettings, Config, DeviceSettings, GeneralSetting
 use crate::core::save::{
     backup_phone, list_available_backup_user, list_available_backups, restore_backup, BACKUP_DIR,
 };
-use crate::core::sync::{perform_adb_commands, CommandType, Phone};
+use crate::core::sync::{get_android_sdk, perform_adb_commands, CommandType, Phone};
 use crate::core::theme::Theme;
 use crate::core::utils::{open_url, string_to_theme, unavailable_users, DisplayablePath};
 use crate::gui::style;
@@ -158,8 +158,12 @@ impl Settings {
                         }
                     }
                     if r_packages.is_empty() {
-                        self.device.backup.backup_state =
-                            "Device state is already restored".to_string();
+                        if get_android_sdk() == 0 {
+                            self.device.backup.backup_state = "Device is not connected".to_string();
+                        } else {
+                            self.device.backup.backup_state =
+                                "Device state is already restored".to_string();
+                        }
                     }
                     info!(
                         "[RESTORE] Restoring backup {}",
@@ -168,8 +172,8 @@ impl Settings {
                     Command::batch(commands)
                 }
                 Err(e) => {
-                    self.device.backup.backup_state = format!("ERROR: {e}");
-                    error!("[BACKUP] {}", e);
+                    self.device.backup.backup_state = e.to_string();
+                    error!("{} - {}", self.device.backup.selected.as_ref().unwrap(), e);
                     Command::none()
                 }
             },