add separate keybind to delete downloaded episodes

Author: xgi

castero/__init__.py

@@ -23,10 +23,11 @@
     {key_help} - show this help screen
     {key_exit} - exit the client
     {key_add_feed} - add a feed
-    {key_delete} - delete the selected feed
-    {key_reload} - reload/refresh all feeds
-    {key_reload_selected} - reload/refresh selected feed
+    {key_remove} - remove the selected feed
+    {key_reload} - reload all feeds
+    {key_reload_selected} - reload the selected feed
     {key_save} - save episode for offline playback
+    {key_delete} - delete downloaded episodes
     {key_up/key_down} - navigate up/down in menus
     {key_right/key_left} - navigate right/left in menus
     {key_scroll_up/key_scroll_down} - scroll up/down in menus

castero/config.py

@@ -130,6 +130,13 @@ def migrate(self, conf, default_conf) -> None:
             for key in default_conf[section]:
                 default_conf_dict[key] = default_conf[section][key]
 
+        # In an update, we renamed key_delete to key_remove and added a
+        # different key_delete value. If the user has key_delete but not
+        # key_removed, we swap these values manually.
+        if 'key_delete' in conf_dict and 'key_remove' not in conf_dict:
+            conf_dict['key_remove'] = conf_dict['key_delete']
+            conf_dict['key_delete'] = default_conf_dict['key_delete']
+
         with open(self._default_path, "r") as default_conf_file:
             lines = default_conf_file.readlines()
             for line in lines:

castero/display.py

@@ -562,33 +562,74 @@ def reload_selected_feed(self, feed: Feed) -> None:
     def save_episodes(self, feed=None, episode=None) -> None:
         """Save a feed or episode.
 
-        If the user is saving an episode and the episode is already saved, this
-        method will instead ask the user if they would like to delete the
-        downloaded episode. However, if the user is saving a feed, there is no
-        prompt to delete episodes, even if some are downloaded. In this case,
-        downloaded episodes are simply skipped.
-
         Exactly one of either feed or episode must be given.
 
         Args:
             feed: (optional) a feed to download all episodes of
-            episode: (optional) an episode to download or delete
+            episode: (optional) an episode to download
         """
         assert (feed is None or episode is None) and (feed is not episode)
 
         if feed is not None:
+            num_saved = 0
+            num_to_save = 0
             for episode in self.database.episodes(feed):
                 if not episode.downloaded:
-                    self._download_queue.add(episode)
+                    num_to_save += 1
+
+            if num_to_save == 0:
+                return
+
+            should_delete = self._get_y_n(
+                    "Are you sure you want to download %d"
+                    " episodes from this feed? (y/n): " % num_to_save)
+            if should_delete:
+                for episode in self.database.episodes(feed):
+                    if not episode.downloaded:
+                        self._download_queue.add(episode)
+        else:
+            if not episode.downloaded:
+                self._download_queue.add(episode)
+
+    def delete_episodes(self, feed=None, episode=None) -> None:
+        """Delete a downloaded episode, or all of those from a feed.
+
+        Exactly one of either feed or episode must be given.
+
+        Args:
+            feed: (optional) a feed to delete all episodes of
+            episode: (optional) an episode or delete
+        """
+        assert (feed is None or episode is None) and (feed is not episode)
+
+        if feed is not None:
+            num_deleted = 0
+            num_to_delete = 0
+            for episode in self.database.episodes(feed):
+                if episode.downloaded:
+                    num_to_delete += 1
+
+            if num_to_delete == 0:
+                return
+
+            should_delete = self._get_y_n(
+                    "Are you sure you want to delete %d downloaded"
+                    " episodes from this feed? (y/n): " % num_to_delete)
+            if should_delete:
+                for episode in self.database.episodes(feed):
+                    if episode.downloaded:
+                        episode.delete(self)
+                        num_deleted += 1
+                self.menus_valid = False
+                self.change_status(
+                    "Successfully deleted %d episodes" % num_deleted)
         else:
             if episode.downloaded:
                 should_delete = self._get_y_n(
                     "Are you sure you want to delete the downloaded"
                     " episode? (y/n): ")
                 if should_delete:
                     episode.delete(self)
-            else:
-                self._download_queue.add(episode)
 
     def filter_menu(self, menu: Menu) -> None:
         menu.filter_text = self._get_input_str("Filter: ")

castero/perspective.py

@@ -172,7 +172,7 @@ def _generic_handle_input(self, c) -> bool:
             queue.change_rate(-1, display=self._display)
         elif c == key_mapping[Config['key_add_feed']]:
             self._display.add_feed()
-        elif c == key_mapping[Config['key_delete']]:
+        elif c == key_mapping[Config['key_remove']]:
             if self._active_window == 0:
                 self._display.delete_feed(self._feed_menu.item)
                 self.update_menus()
@@ -190,6 +190,11 @@ def _generic_handle_input(self, c) -> bool:
                 self._display.save_episodes(feed=self._feed_menu.item)
             elif self._active_window == 1 and self._episode_menu.item:
                 self._display.save_episodes(episode=self._episode_menu.item)
+        elif c == key_mapping[Config['key_delete']]:
+            if self._active_window == 0 and self._feed_menu.item:
+                self._display.delete_episodes(feed=self._feed_menu.item)
+            elif self._active_window == 1 and self._episode_menu.item:
+                self._display.delete_episodes(episode=self._episode_menu.item)
         elif c == key_mapping[Config['key_execute']]:
             if self._active_window == 1 and self._episode_menu.item:
                 self._display.execute_command(self._episode_menu.item)

castero/perspectives/downloadedperspective.py

@@ -138,6 +138,10 @@ def handle_input(self, c) -> bool:
             if self._downloaded_menu.item:
                 self._display.save_episodes(episode=self._downloaded_menu.item)
                 self._display.menus_valid = False
+        elif c == key_mapping[Config['key_delete']]:
+            if self._downloaded_menu.item:
+                self._display.delete_episodes(episode=self._downloaded_menu.item)
+                self._display.menus_valid = False
         elif c == key_mapping[Config['key_mark_played']]:
             if self._active_window == 0:
                 episode = self._downloaded_menu.item
@@ -149,7 +153,9 @@ def handle_input(self, c) -> bool:
             episode = self._downloaded_menu.item
             if episode is not None:
                 self._display.execute_command(episode)
-        elif c == key_mapping[Config['key_delete']]:
+        elif c == key_mapping[Config['key_reload_selected']]:
+            pass
+        elif c == key_mapping[Config['key_remove']]:
             pass
         else:
             keep_running = self._generic_handle_input(c)

castero/perspectives/queueperspective.py

@@ -132,7 +132,7 @@ def handle_input(self, c) -> bool:
             queue.stop()
             queue.clear()
             self._display.menus_valid = False
-        elif c == key_mapping[Config['key_delete']]:
+        elif c == key_mapping[Config['key_remove']]:
             self._remove_selected_from_queue()
         elif c == key_mapping[Config['key_show_url']]:
             item = self._queue_menu.item
@@ -146,6 +146,8 @@ def handle_input(self, c) -> bool:
             pass
         elif c == key_mapping[Config['key_save']]:
             pass
+        elif c == key_mapping[Config['key_delete']]:
+            pass
         elif c == key_mapping[Config['key_mark_played']]:
             pass
         elif c == key_mapping[Config['key_filter']]:

castero/templates/castero.conf

@@ -164,9 +164,9 @@ key_exit = q
 # default: a
 key_add_feed = a
 
-# Delete the selected feed.
+# Remove the selected feed.
 # default: d
-key_delete = d
+key_remove = d
 
 # Reload/refresh all feeds.
 # default: r
@@ -180,6 +180,10 @@ key_reload_selected = R
 # default: s
 key_save = s
 
+# Delete downloaded episodes.
+# default: x
+key_delete = x
+
 # Navigate up.
 # default: UP
 key_up = UP

tests/datafiles/working_no_comments.conf

@@ -39,7 +39,9 @@ key_exit = q
 
 key_add_feed = a
 
-key_delete = d
+key_remove = d
+
+key_delete = x
 
 key_reload = r
 

tests/test_perspective_downloaded.py

@@ -86,6 +86,7 @@ def test_perspective_downloaded_input_keys(display):
 
     operation_keys = [
         display.KEY_MAPPING[Config['key_delete']],
+        display.KEY_MAPPING[Config['key_remove']],
         display.KEY_MAPPING[Config['key_reload']],
         display.KEY_MAPPING[Config['key_reload_selected']],
         display.KEY_MAPPING[Config['key_save']],

tests/test_perspective_primary.py

@@ -89,6 +89,7 @@ def test_perspective_primary_input_keys(display):
     operation_keys = [
         display.KEY_MAPPING[Config['key_add_feed']],
         display.KEY_MAPPING[Config['key_delete']],
+        display.KEY_MAPPING[Config['key_remove']],
         display.KEY_MAPPING[Config['key_reload']],
         display.KEY_MAPPING[Config['key_reload_selected']],
         display.KEY_MAPPING[Config['key_save']],