fix: Prevented additional empty slice errors in indexer tables

This commit is contained in:
2026-01-07 14:09:12 -07:00
parent 60c4cf1098
commit 64d8c65831
6 changed files with 18 additions and 23 deletions
-4
View File
@@ -272,10 +272,6 @@ impl HorizontallyScrollableText {
}
}
}
pub fn is_empty(&self) -> bool {
self.text.is_empty()
}
}
#[derive(Clone, PartialEq, Eq, Debug)]
-5
View File
@@ -176,11 +176,6 @@ mod tests {
assert_str_eq!(horizontally_scrollable_text.text, test_text);
}
#[test]
fn test_horizontally_scrollable_text_is_empty() {
assert_is_empty!(HorizontallyScrollableText::from(""))
}
#[test]
fn test_horizontally_scrollable_text_scroll_text_left() {
let horizontally_scrollable_text = HorizontallyScrollableText::from("Test string");
@@ -35,7 +35,7 @@ fn draw_test_all_indexers_test_results(f: &mut Frame<'_>, app: &mut App<'_>, are
let block = title_block("Test All Indexers");
let current_selection =
if let Some(test_all_results) = app.data.radarr_data.indexer_test_all_results.as_ref() {
if let Some(test_all_results) = app.data.radarr_data.indexer_test_all_results.as_ref() && !test_all_results.is_empty() {
test_all_results.current_selection().clone()
} else {
IndexerTestResultModalItem::default()
+8 -6
View File
@@ -69,12 +69,14 @@ impl DrawUi for AddMovieUi {
fn draw_add_movie_search(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
let is_loading = app.is_loading || app.data.radarr_data.add_searched_movies.is_none();
let current_selection =
if let Some(add_searched_movies) = app.data.radarr_data.add_searched_movies.as_ref() && app.error.is_empty() {
add_searched_movies.current_selection().clone()
} else {
AddMovieSearchResult::default()
};
let current_selection = if let Some(add_searched_movies) =
app.data.radarr_data.add_searched_movies.as_ref()
&& !add_searched_movies.is_empty()
{
add_searched_movies.current_selection().clone()
} else {
AddMovieSearchResult::default()
};
let [search_box_area, results_area] =
Layout::vertical([Constraint::Length(3), Constraint::Fill(0)])
@@ -34,7 +34,7 @@ impl DrawUi for TestAllIndexersUi {
fn draw_test_all_indexers_test_results(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
let is_loading = app.is_loading || app.data.sonarr_data.indexer_test_all_results.is_none();
let current_selection =
if let Some(test_all_results) = app.data.sonarr_data.indexer_test_all_results.as_ref() {
if let Some(test_all_results) = app.data.sonarr_data.indexer_test_all_results.as_ref() && !test_all_results.is_empty() {
test_all_results.current_selection().clone()
} else {
IndexerTestResultModalItem::default()
+8 -6
View File
@@ -64,12 +64,14 @@ impl DrawUi for AddSeriesUi {
fn draw_add_series_search(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
let is_loading = app.is_loading || app.data.sonarr_data.add_searched_series.is_none();
let current_selection =
if let Some(add_searched_series) = app.data.sonarr_data.add_searched_series.as_ref() && app.error.is_empty() {
add_searched_series.current_selection().clone()
} else {
AddSeriesSearchResult::default()
};
let current_selection = if let Some(add_searched_series) =
app.data.sonarr_data.add_searched_series.as_ref()
&& !add_searched_series.is_empty()
{
add_searched_series.current_selection().clone()
} else {
AddSeriesSearchResult::default()
};
let [search_box_area, results_area] =
Layout::vertical([Constraint::Length(3), Constraint::Fill(0)])