From ec8f4680f508f6a7b2a5d16bc049cc6cdf92ce42 Mon Sep 17 00:00:00 2001 From: Sander Sweers Date: Fri, 20 Dec 2024 17:18:53 +0100 Subject: [PATCH] Revert "TransferService: small code optimization for on_transfer_completed" This reverts commit 4c8d482fba489a515afb2c1204b25beb00b2cd8d. --- blueman/plugins/applet/TransferService.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/blueman/plugins/applet/TransferService.py b/blueman/plugins/applet/TransferService.py index 792b55bdc..2ab53ccbd 100644 --- a/blueman/plugins/applet/TransferService.py +++ b/blueman/plugins/applet/TransferService.py @@ -286,12 +286,12 @@ def _on_transfer_completed(self, _manager: Manager, transfer_path: str, success: dest_dir, ignored = self._make_share_path() filename = os.path.basename(src) - dest = os.path.join(dest_dir, filename) - if os.path.exists(dest): + if os.path.exists(os.path.join(dest_dir, filename)): now = datetime.now() filename = f"{now.strftime('%Y%m%d%H%M%S')}_{filename}" logging.info(f"Destination file exists, renaming to: {filename}") + dest = os.path.join(dest_dir, filename) try: shutil.move(src, dest) except (OSError, PermissionError):