From 1915bdfc46d1e2298f3085585e794f04cf27afaa Mon Sep 17 00:00:00 2001 From: Alex Shnitman Date: Mon, 2 Mar 2026 20:29:29 +0200 Subject: [PATCH] refactor: simplify filename generation by removing unnecessary relative path computation (closes #916, closes #917) --- app/ytdl.py | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/app/ytdl.py b/app/ytdl.py index 12b9d84..8166043 100644 --- a/app/ytdl.py +++ b/app/ytdl.py @@ -214,16 +214,7 @@ class Download: filepath = d['info_dict']['filepath'] if '__finaldir' in d['info_dict']: finaldir = d['info_dict']['__finaldir'] - # Compute relative path from temp dir to preserve - # subdirectory structure from the output template. - try: - rel_path = os.path.relpath(filepath, self.temp_dir) - except ValueError: - rel_path = os.path.basename(filepath) - if rel_path.startswith('..'): - # filepath is not under temp_dir, fall back to basename - rel_path = os.path.basename(filepath) - filename = os.path.join(finaldir, rel_path) + filename = os.path.join(finaldir, os.path.basename(filepath)) else: filename = filepath self.status_queue.put({'status': 'finished', 'filename': filename})