From 71ca5f39f2740793a2f93f1fab8066e4de57c589 Mon Sep 17 00:00:00 2001 From: Diego Prado Gesto Date: Wed, 4 Jul 2018 17:30:29 +0200 Subject: [PATCH] Fix indentation to move the remove function outside of the loop because it only needs to be done once. --- cura/CuraPackageManager.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/cura/CuraPackageManager.py b/cura/CuraPackageManager.py index 1ce3226a50..f51d0622c4 100644 --- a/cura/CuraPackageManager.py +++ b/cura/CuraPackageManager.py @@ -343,11 +343,11 @@ class CuraPackageManager(QObject): continue self.__installPackageFiles(package_id, src_dir_path, dst_dir_path) - # Remove the file - try: - os.remove(filename) - except Exception: - Logger.log("w", "Tried to delete file [%s], but it failed", filename) + # Remove the file + try: + os.remove(filename) + except Exception: + Logger.log("w", "Tried to delete file [%s], but it failed", filename) # Move the info to the installed list of packages only when it succeeds self._installed_package_dict[package_id] = self._to_install_package_dict[package_id]