[Console] Fix 'move' command hanging when done

Console.get_torrent_name() expects string, but was given list from console.match_torrent(), so NoneType breaking join() in move message. Console.get_torrent_name() expects string, but was given list from console.match_torrent(), so NoneType breaking join() in move message. Simplified and fixed now. Co-authored-by: Calum Lind <[email protected]> Closes: #447
deluge-torrent · Sep 10, 2024 · 40d4f7e · 40d4f7e
1 parent d064ad0
commit 40d4f7e
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/deluge/ui/console/cmdline/commands/move.py b/deluge/ui/console/cmdline/commands/move.py
@@ -41,12 +41,8 @@ def handle(self, options):
             )
             return
 
-        ids = []
-        names = []
-        for t_id in options.torrent_ids:
-            tid = self.console.match_torrent(t_id)
-            ids.extend(tid)
-            names.append(self.console.get_torrent_name(tid))
+        ids = self.console.match_torrents(options.torrent_ids)
+        names = [self.console.get_torrent_name(id_) for id_ in ids]
 
         def on_move(res):
             msg = 'Moved "{}" to {}'.format(', '.join(names), options.path)