From d3a852667e33e27ebc089360debd6d26981f671c Mon Sep 17 00:00:00 2001 From: Chris Hennes Date: Wed, 13 Aug 2025 20:04:17 -0500 Subject: [PATCH] Improve backup deletion code --- AddonManager.py | 6 ++++-- package.xml | 4 ++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/AddonManager.py b/AddonManager.py index f267e458..6f3475a5 100644 --- a/AddonManager.py +++ b/AddonManager.py @@ -845,9 +845,11 @@ def revert_to_backup(addon: Addon) -> None: def delete_old_backups(backups) -> None: """Delete old backups found in the Mod directory.""" for backup in backups: - full_path = os.path.join(fci.DataPaths().mod_dir, backup) + full_path = str(os.path.join(fci.DataPaths().mod_dir, backup)) if os.path.exists(full_path): - shutil.rmtree(full_path, ignore_errors=True) + success = utils.rmdir(full_path) + if not success: + fci.Console.PrintError(f"Failed to delete {full_path}\n") # @} diff --git a/package.xml b/package.xml index e2ab19e6..81d18376 100644 --- a/package.xml +++ b/package.xml @@ -5,8 +5,8 @@ Addon Manager Tool to install workbenches, macros, themes, etc. Resources/icons/addon_manager.svg - 2025.08.08 - 2025-08-08 + 2025.08.13 + 2025-08-13 Chris Hennes Yorik van Havre Jonathan Wiedemann