diff --git a/src/handlers/radarr_handlers/library/library_handler_tests.rs b/src/handlers/radarr_handlers/library/library_handler_tests.rs index 49f8738..e513af1 100644 --- a/src/handlers/radarr_handlers/library/library_handler_tests.rs +++ b/src/handlers/radarr_handlers/library/library_handler_tests.rs @@ -199,6 +199,7 @@ mod tests { #[test] fn test_movie_search_box_home_end_keys() { let mut app = App::default(); + app.push_navigation_stack(ActiveRadarrBlock::SearchMovie.into()); app .data .radarr_data @@ -252,6 +253,7 @@ mod tests { #[test] fn test_movie_filter_box_home_end_keys() { let mut app = App::default(); + app.push_navigation_stack(ActiveRadarrBlock::FilterMovies.into()); app .data .radarr_data @@ -479,6 +481,8 @@ mod tests { #[test] fn test_movie_search_box_left_right_keys() { let mut app = App::default(); + app.push_navigation_stack(ActiveRadarrBlock::SearchMovie.into()); + app.data.radarr_data.movies.set_items(vec![Movie::default()]); app.data.radarr_data.movies.search = Some("Test".into()); LibraryHandler::with( @@ -527,6 +531,8 @@ mod tests { #[test] fn test_movie_filter_box_left_right_keys() { let mut app = App::default(); + app.push_navigation_stack(ActiveRadarrBlock::FilterMovies.into()); + app.data.radarr_data.movies.set_items(vec![Movie::default()]); app.data.radarr_data.movies.filter = Some("Test".into()); LibraryHandler::with( @@ -882,6 +888,7 @@ mod tests { filtered_state: Some(TableState::default()), ..StatefulTable::default() }; + app.data.radarr_data.movies.set_items(vec![Movie::default()]); LibraryHandler::with(ESC_KEY, &mut app, active_radarr_block, None).handle(); @@ -914,6 +921,7 @@ mod tests { #[test] fn test_movies_sort_prompt_block_esc() { let mut app = App::default(); + app.data.radarr_data.movies.set_items(movies_vec()); app.push_navigation_stack(ActiveRadarrBlock::Movies.into()); app.push_navigation_stack(ActiveRadarrBlock::MoviesSortPrompt.into()); @@ -942,10 +950,6 @@ mod tests { assert_eq!(app.get_current_route(), ActiveRadarrBlock::Movies.into()); assert!(app.error.text.is_empty()); - assert_eq!(app.data.radarr_data.movies.search, None); - assert_eq!(app.data.radarr_data.movies.filter, None); - assert_eq!(app.data.radarr_data.movies.filtered_items, None); - assert_eq!(app.data.radarr_data.movies.filtered_state, None); } } @@ -1275,6 +1279,7 @@ mod tests { #[test] fn test_search_movies_box_backspace_key() { let mut app = App::default(); + app.push_navigation_stack(ActiveRadarrBlock::SearchMovie.into()); app.data.radarr_data.movies.search = Some("Test".into()); app .data @@ -1299,6 +1304,7 @@ mod tests { #[test] fn test_filter_movies_box_backspace_key() { let mut app = App::default(); + app.push_navigation_stack(ActiveRadarrBlock::FilterMovies.into()); app .data .radarr_data @@ -1323,6 +1329,7 @@ mod tests { #[test] fn test_search_movies_box_char_key() { let mut app = App::default(); + app.push_navigation_stack(ActiveRadarrBlock::SearchMovie.into()); app .data .radarr_data @@ -1347,6 +1354,7 @@ mod tests { #[test] fn test_filter_movies_box_char_key() { let mut app = App::default(); + app.push_navigation_stack(ActiveRadarrBlock::FilterMovies.into()); app .data .radarr_data diff --git a/src/handlers/radarr_handlers/library/mod.rs b/src/handlers/radarr_handlers/library/mod.rs index f8ae0b9..b77c47c 100644 --- a/src/handlers/radarr_handlers/library/mod.rs +++ b/src/handlers/radarr_handlers/library/mod.rs @@ -8,6 +8,8 @@ use crate::handlers::radarr_handlers::library::edit_movie_handler::EditMovieHand use crate::handlers::radarr_handlers::library::movie_details_handler::MovieDetailsHandler; use crate::handlers::{handle_clear_errors, handle_prompt_toggle, KeyEventHandler}; +use crate::handle_table_events; +use crate::handlers::table_handler::TableHandlingProps; use crate::models::radarr_models::Movie; use crate::models::servarr_data::radarr::radarr_data::{ ActiveRadarrBlock, DELETE_MOVIE_SELECTION_BLOCKS, EDIT_MOVIE_SELECTION_BLOCKS, LIBRARY_BLOCKS, @@ -15,7 +17,6 @@ use crate::models::servarr_data::radarr::radarr_data::{ use crate::models::stateful_table::SortOption; use crate::models::{BlockSelectionState, HorizontallyScrollableText, Scrollable}; use crate::network::radarr_network::RadarrEvent; -use crate::{handle_text_box_keys, handle_text_box_left_right_keys}; mod add_movie_handler; mod delete_movie_handler; @@ -33,24 +34,43 @@ pub(super) struct LibraryHandler<'a, 'b> { context: Option, } +impl<'a, 'b> LibraryHandler<'a, 'b> { + handle_table_events!(self, movies, self.app.data.radarr_data.movies, Movie); +} + impl<'a, 'b> KeyEventHandler<'a, 'b, ActiveRadarrBlock> for LibraryHandler<'a, 'b> { fn handle(&mut self) { - match self.active_radarr_block { - _ if AddMovieHandler::accepts(self.active_radarr_block) => { - AddMovieHandler::with(self.key, self.app, self.active_radarr_block, self.context).handle(); + let movie_table_handling_props = TableHandlingProps::new(ActiveRadarrBlock::Movies.into()) + .sorting_block(ActiveRadarrBlock::MoviesSortPrompt.into()) + .sort_by_fn(|a: &Movie, b: &Movie| a.id.cmp(&b.id)) + .sort_options(movies_sorting_options()) + .searching_block(ActiveRadarrBlock::SearchMovie.into()) + .search_error_block(ActiveRadarrBlock::SearchMovieError.into()) + .search_field_fn(|movie| &movie.title.text) + .filtering_block(ActiveRadarrBlock::FilterMovies.into()) + .filter_error_block(ActiveRadarrBlock::FilterMoviesError.into()) + .filter_field_fn(|movie| &movie.title.text); + + if !self.handle_movies_table_events(movie_table_handling_props) { + match self.active_radarr_block { + _ if AddMovieHandler::accepts(self.active_radarr_block) => { + AddMovieHandler::with(self.key, self.app, self.active_radarr_block, self.context) + .handle(); + } + _ if DeleteMovieHandler::accepts(self.active_radarr_block) => { + DeleteMovieHandler::with(self.key, self.app, self.active_radarr_block, self.context) + .handle(); + } + _ if EditMovieHandler::accepts(self.active_radarr_block) => { + EditMovieHandler::with(self.key, self.app, self.active_radarr_block, self.context) + .handle(); + } + _ if MovieDetailsHandler::accepts(self.active_radarr_block) => { + MovieDetailsHandler::with(self.key, self.app, self.active_radarr_block, self.context) + .handle(); + } + _ => self.handle_key_event(), } - _ if DeleteMovieHandler::accepts(self.active_radarr_block) => { - DeleteMovieHandler::with(self.key, self.app, self.active_radarr_block, self.context) - .handle(); - } - _ if EditMovieHandler::accepts(self.active_radarr_block) => { - EditMovieHandler::with(self.key, self.app, self.active_radarr_block, self.context).handle(); - } - _ if MovieDetailsHandler::accepts(self.active_radarr_block) => { - MovieDetailsHandler::with(self.key, self.app, self.active_radarr_block, self.context) - .handle(); - } - _ => self.handle_key_event(), } } @@ -84,109 +104,13 @@ impl<'a, 'b> KeyEventHandler<'a, 'b, ActiveRadarrBlock> for LibraryHandler<'a, ' !self.app.is_loading && !self.app.data.radarr_data.movies.is_empty() } - fn handle_scroll_up(&mut self) { - match self.active_radarr_block { - ActiveRadarrBlock::Movies => self.app.data.radarr_data.movies.scroll_up(), - ActiveRadarrBlock::MoviesSortPrompt => self - .app - .data - .radarr_data - .movies - .sort - .as_mut() - .unwrap() - .scroll_up(), - _ => (), - } - } + fn handle_scroll_up(&mut self) {} - fn handle_scroll_down(&mut self) { - match self.active_radarr_block { - ActiveRadarrBlock::Movies => self.app.data.radarr_data.movies.scroll_down(), - ActiveRadarrBlock::MoviesSortPrompt => self - .app - .data - .radarr_data - .movies - .sort - .as_mut() - .unwrap() - .scroll_down(), - _ => (), - } - } + fn handle_scroll_down(&mut self) {} - fn handle_home(&mut self) { - match self.active_radarr_block { - ActiveRadarrBlock::Movies => self.app.data.radarr_data.movies.scroll_to_top(), - ActiveRadarrBlock::SearchMovie => { - self - .app - .data - .radarr_data - .movies - .search - .as_mut() - .unwrap() - .scroll_home(); - } - ActiveRadarrBlock::FilterMovies => { - self - .app - .data - .radarr_data - .movies - .filter - .as_mut() - .unwrap() - .scroll_home(); - } - ActiveRadarrBlock::MoviesSortPrompt => self - .app - .data - .radarr_data - .movies - .sort - .as_mut() - .unwrap() - .scroll_to_top(), - _ => (), - } - } + fn handle_home(&mut self) {} - fn handle_end(&mut self) { - match self.active_radarr_block { - ActiveRadarrBlock::Movies => self.app.data.radarr_data.movies.scroll_to_bottom(), - ActiveRadarrBlock::SearchMovie => self - .app - .data - .radarr_data - .movies - .search - .as_mut() - .unwrap() - .reset_offset(), - ActiveRadarrBlock::FilterMovies => self - .app - .data - .radarr_data - .movies - .filter - .as_mut() - .unwrap() - .reset_offset(), - ActiveRadarrBlock::MoviesSortPrompt => self - .app - .data - .radarr_data - .movies - .sort - .as_mut() - .unwrap() - .scroll_to_bottom(), - _ => (), - } - } + fn handle_end(&mut self) {} fn handle_delete(&mut self) { if self.active_radarr_block == ActiveRadarrBlock::Movies { @@ -202,20 +126,6 @@ impl<'a, 'b> KeyEventHandler<'a, 'b, ActiveRadarrBlock> for LibraryHandler<'a, ' match self.active_radarr_block { ActiveRadarrBlock::Movies => handle_change_tab_left_right_keys(self.app, self.key), ActiveRadarrBlock::UpdateAllMoviesPrompt => handle_prompt_toggle(self.app, self.key), - ActiveRadarrBlock::SearchMovie => { - handle_text_box_left_right_keys!( - self, - self.key, - self.app.data.radarr_data.movies.search.as_mut().unwrap() - ) - } - ActiveRadarrBlock::FilterMovies => { - handle_text_box_left_right_keys!( - self, - self.key, - self.app.data.radarr_data.movies.filter.as_mut().unwrap() - ) - } _ => (), } } @@ -225,44 +135,6 @@ impl<'a, 'b> KeyEventHandler<'a, 'b, ActiveRadarrBlock> for LibraryHandler<'a, ' ActiveRadarrBlock::Movies => self .app .push_navigation_stack(ActiveRadarrBlock::MovieDetails.into()), - ActiveRadarrBlock::SearchMovie => { - self.app.pop_navigation_stack(); - self.app.should_ignore_quit_key = false; - - if self.app.data.radarr_data.movies.search.is_some() { - let has_match = self - .app - .data - .radarr_data - .movies - .apply_search(|movie| &movie.title.text); - - if !has_match { - self - .app - .push_navigation_stack(ActiveRadarrBlock::SearchMovieError.into()); - } - } - } - ActiveRadarrBlock::FilterMovies => { - self.app.pop_navigation_stack(); - self.app.should_ignore_quit_key = false; - - if self.app.data.radarr_data.movies.filter.is_some() { - let has_matches = self - .app - .data - .radarr_data - .movies - .apply_filter(|movie| &movie.title.text); - - if !has_matches { - self - .app - .push_navigation_stack(ActiveRadarrBlock::FilterMoviesError.into()); - } - } - } ActiveRadarrBlock::UpdateAllMoviesPrompt => { if self.app.data.radarr_data.prompt_confirm { self.app.data.radarr_data.prompt_confirm_action = Some(RadarrEvent::UpdateAllMovies); @@ -270,44 +142,17 @@ impl<'a, 'b> KeyEventHandler<'a, 'b, ActiveRadarrBlock> for LibraryHandler<'a, ' self.app.pop_navigation_stack(); } - ActiveRadarrBlock::MoviesSortPrompt => { - self - .app - .data - .radarr_data - .movies - .items - .sort_by(|a, b| a.id.cmp(&b.id)); - self.app.data.radarr_data.movies.apply_sorting(); - - self.app.pop_navigation_stack(); - } _ => (), } } fn handle_esc(&mut self) { match self.active_radarr_block { - ActiveRadarrBlock::FilterMovies | ActiveRadarrBlock::FilterMoviesError => { - self.app.pop_navigation_stack(); - self.app.data.radarr_data.movies.reset_filter(); - self.app.should_ignore_quit_key = false; - } - ActiveRadarrBlock::SearchMovie | ActiveRadarrBlock::SearchMovieError => { - self.app.pop_navigation_stack(); - self.app.data.radarr_data.movies.reset_search(); - self.app.should_ignore_quit_key = false; - } ActiveRadarrBlock::UpdateAllMoviesPrompt => { self.app.pop_navigation_stack(); self.app.data.radarr_data.prompt_confirm = false; } - ActiveRadarrBlock::MoviesSortPrompt => { - self.app.pop_navigation_stack(); - } _ => { - self.app.data.radarr_data.movies.reset_search(); - self.app.data.radarr_data.movies.reset_filter(); handle_clear_errors(self.app); } } @@ -317,21 +162,6 @@ impl<'a, 'b> KeyEventHandler<'a, 'b, ActiveRadarrBlock> for LibraryHandler<'a, ' let key = self.key; match self.active_radarr_block { ActiveRadarrBlock::Movies => match self.key { - _ if key == DEFAULT_KEYBINDINGS.search.key => { - self - .app - .push_navigation_stack(ActiveRadarrBlock::SearchMovie.into()); - self.app.data.radarr_data.movies.search = Some(HorizontallyScrollableText::default()); - self.app.should_ignore_quit_key = true; - } - _ if key == DEFAULT_KEYBINDINGS.filter.key => { - self - .app - .push_navigation_stack(ActiveRadarrBlock::FilterMovies.into()); - self.app.data.radarr_data.movies.reset_filter(); - self.app.data.radarr_data.movies.filter = Some(HorizontallyScrollableText::default()); - self.app.should_ignore_quit_key = true; - } _ if key == DEFAULT_KEYBINDINGS.edit.key => { self.app.push_navigation_stack( ( @@ -359,33 +189,8 @@ impl<'a, 'b> KeyEventHandler<'a, 'b, ActiveRadarrBlock> for LibraryHandler<'a, ' _ if key == DEFAULT_KEYBINDINGS.refresh.key => { self.app.should_refresh = true; } - _ if key == DEFAULT_KEYBINDINGS.sort.key => { - self - .app - .data - .radarr_data - .movies - .sorting(movies_sorting_options()); - self - .app - .push_navigation_stack(ActiveRadarrBlock::MoviesSortPrompt.into()); - } _ => (), }, - ActiveRadarrBlock::SearchMovie => { - handle_text_box_keys!( - self, - key, - self.app.data.radarr_data.movies.search.as_mut().unwrap() - ) - } - ActiveRadarrBlock::FilterMovies => { - handle_text_box_keys!( - self, - key, - self.app.data.radarr_data.movies.filter.as_mut().unwrap() - ) - } ActiveRadarrBlock::UpdateAllMoviesPrompt => { if key == DEFAULT_KEYBINDINGS.confirm.key { self.app.data.radarr_data.prompt_confirm = true; diff --git a/src/models/servarr_data/radarr/radarr_test_utils.rs b/src/models/servarr_data/radarr/radarr_test_utils.rs index a3c5469..aae4098 100644 --- a/src/models/servarr_data/radarr/radarr_test_utils.rs +++ b/src/models/servarr_data/radarr/radarr_test_utils.rs @@ -1,12 +1,11 @@ #[cfg(test)] pub mod utils { - use crate::models::radarr_models::{ - AddMovieSearchResult, CollectionMovie, Credit, MovieHistoryItem, RadarrRelease, - }; + use crate::models::radarr_models::{AddMovieSearchResult, BlocklistItem, Collection, CollectionMovie, Credit, DownloadRecord, Movie, MovieHistoryItem, RadarrRelease}; use crate::models::servarr_data::radarr::modals::MovieDetailsModal; use crate::models::servarr_data::radarr::radarr_data::RadarrData; use crate::models::stateful_table::StatefulTable; use crate::models::{HorizontallyScrollableText, ScrollableText}; + use crate::models::servarr_models::{Indexer, RootFolder}; pub fn create_test_radarr_data<'a>() -> RadarrData<'a> { let mut movie_details_modal = MovieDetailsModal { @@ -35,6 +34,13 @@ pub mod utils { add_searched_movies: Some(StatefulTable::default()), ..RadarrData::default() }; + radarr_data.movies.set_items(vec![Movie::default()]); + radarr_data.collection_movies.set_items(vec![CollectionMovie::default()]); + radarr_data.collections.set_items(vec![Collection::default()]); + radarr_data.downloads.set_items(vec![DownloadRecord::default()]); + radarr_data.blocklist.set_items(vec![BlocklistItem::default()]); + radarr_data.root_folders.set_items(vec![RootFolder::default()]); + radarr_data.indexers.set_items(vec![Indexer::default()]); radarr_data.movie_info_tabs.index = 1; radarr_data .add_searched_movies