bump: version 0.6.2 → 0.6.3 [skip ci]

.github/workflows/release.yml

@@ -50,8 +50,8 @@ jobs:
 
       - name: Install Commitizen
         run: |
-          python -m pip install --upgrade pip
-          pip install commitizen
+          python3 -m pip install --upgrade pip
+          pip3 install commitizen==4.8.3
           npm install -g conventional-changelog-cli
 
       - name: Configure Git user

CHANGELOG.md

@@ -5,6 +5,22 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## v0.6.3 (2025-12-13)
+
+### Fix
+
+- Wrapped all Sonarr use of Language with Option to fix the 'null' array issue in the new Sonarr API
+
+## v0.6.2 (2025-12-12)
+
+### Fix
+
+- Fixed breaking Sonarr Episode file API calls after recent Sonarr API update
+
+### Refactor
+
+- Replaced all modulo usages of tick_until_poll with is_multiple_of
+
 ## v0.6.1 (2025-09-02)
 
 ### Fix

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "managarr"
-version = "0.6.1"
+version = "0.6.3"
 authors = ["Alex Clarke <alex.j.tusa@gmail.com>"]
 description = "A TUI and CLI to manage your Servarrs"
 keywords = ["managarr", "ratatui", "dashboard", "servarr", "tui"]

src/app/mod.rs

@@ -160,7 +160,10 @@ impl App<'_> {
   }
 
   pub async fn on_tick(&mut self) {
-    if self.tick_count % self.tick_until_poll == 0 || self.is_routing || self.should_refresh {
+    if self.tick_count.is_multiple_of(self.tick_until_poll)
+      || self.is_routing
+      || self.should_refresh
+    {
       match self.get_current_route() {
         Route::Radarr(active_radarr_block, _) => self.radarr_on_tick(active_radarr_block).await,
         Route::Sonarr(active_sonarr_block, _) => self.sonarr_on_tick(active_sonarr_block).await,

src/app/radarr/mod.rs

@@ -185,7 +185,7 @@ impl App<'_> {
       }
     }
 
-    if self.tick_count % self.tick_until_poll == 0 {
+    if self.tick_count.is_multiple_of(self.tick_until_poll) {
       self.refresh_radarr_metadata().await;
     }
   }

src/app/sonarr/mod.rs

@@ -215,7 +215,7 @@ impl App<'_> {
       }
     }
 
-    if self.tick_count % self.tick_until_poll == 0 {
+    if self.tick_count.is_multiple_of(self.tick_until_poll) {
       self.refresh_sonarr_metadata().await;
     }
   }

src/cli/mod.rs

@@ -1,7 +1,7 @@
 use std::sync::Arc;
 
 use anyhow::Result;
-use clap::{command, Subcommand};
+use clap::Subcommand;
 use clap_complete::Shell;
 use radarr::{RadarrCliHandler, RadarrCommand};
 use sonarr::{SonarrCliHandler, SonarrCommand};

src/cli/radarr/add_command_handler.rs

@@ -1,7 +1,7 @@
 use std::sync::Arc;
 
 use anyhow::Result;
-use clap::{arg, command, ArgAction, Subcommand};
+use clap::{ArgAction, Subcommand};
 use tokio::sync::Mutex;
 
 use super::RadarrCommand;

src/cli/radarr/get_command_handler.rs

@@ -1,7 +1,7 @@
 use std::sync::Arc;
 
 use anyhow::Result;
-use clap::{command, Subcommand};
+use clap::Subcommand;
 use tokio::sync::Mutex;
 
 use crate::{

src/cli/radarr/list_command_handler.rs

@@ -1,7 +1,7 @@
 use std::sync::Arc;
 
 use anyhow::Result;
-use clap::{command, Subcommand};
+use clap::Subcommand;
 use tokio::sync::Mutex;
 
 use crate::{

src/handlers/sonarr_handlers/blocklist/blocklist_handler_tests.rs

@@ -445,13 +445,25 @@ mod tests {
       let a_languages = a
         .languages
         .iter()
-        .map(|lang| lang.name.to_lowercase())
+        .map(|lang| {
+          lang
+            .as_ref()
+            .unwrap_or(&Default::default())
+            .name
+            .to_lowercase()
+        })
         .collect::<Vec<String>>()
         .join(", ");
       let b_languages = b
         .languages
         .iter()
-        .map(|lang| lang.name.to_lowercase())
+        .map(|lang| {
+          lang
+            .as_ref()
+            .unwrap_or(&Default::default())
+            .name
+            .to_lowercase()
+        })
         .collect::<Vec<String>>()
         .join(", ");
 
@@ -607,10 +619,10 @@ mod tests {
       BlocklistItem {
         id: 3,
         source_title: "test 1".to_owned(),
-        languages: vec![Language {
+        languages: vec![Some(Language {
           id: 1,
           name: "telgu".to_owned(),
-        }],
+        })],
         quality: QualityWrapper {
           quality: Quality {
             name: "HD - 1080p".to_owned(),
@@ -623,10 +635,10 @@ mod tests {
       BlocklistItem {
         id: 2,
         source_title: "test 2".to_owned(),
-        languages: vec![Language {
+        languages: vec![Some(Language {
           id: 3,
           name: "chinese".to_owned(),
-        }],
+        })],
         quality: QualityWrapper {
           quality: Quality {
             name: "SD - 720p".to_owned(),
@@ -639,10 +651,10 @@ mod tests {
       BlocklistItem {
         id: 1,
         source_title: "test 3".to_owned(),
-        languages: vec![Language {
+        languages: vec![Some(Language {
           id: 1,
           name: "english".to_owned(),
-        }],
+        })],
         quality: QualityWrapper {
           quality: Quality {
             name: "HD - 1080p".to_owned(),

src/handlers/sonarr_handlers/blocklist/mod.rs

@@ -210,13 +210,25 @@ fn blocklist_sorting_options() -> Vec<SortOption<BlocklistItem>> {
         let a_languages = a
           .languages
           .iter()
-          .map(|lang| lang.name.to_lowercase())
+          .map(|lang| {
+            lang
+              .as_ref()
+              .unwrap_or(&Default::default())
+              .name
+              .to_lowercase()
+          })
           .collect::<Vec<String>>()
           .join(", ");
         let b_languages = b
           .languages
           .iter()
-          .map(|lang| lang.name.to_lowercase())
+          .map(|lang| {
+            lang
+              .as_ref()
+              .unwrap_or(&Default::default())
+              .name
+              .to_lowercase()
+          })
           .collect::<Vec<String>>()
           .join(", ");
 

src/handlers/sonarr_handlers/history/history_handler_tests.rs

@@ -245,8 +245,19 @@ mod tests {
         id: 1,
         name: "_".to_owned(),
       };
-      let language_a = &a.languages.first().unwrap_or(&default_language);
-      let language_b = &b.languages.first().unwrap_or(&default_language);
+      let default_language_option = Some(default_language.clone());
+      let language_a = &a
+        .languages
+        .first()
+        .unwrap_or(&default_language_option)
+        .as_ref()
+        .unwrap_or(&default_language);
+      let language_b = &b
+        .languages
+        .first()
+        .unwrap_or(&default_language_option)
+        .as_ref()
+        .unwrap_or(&default_language);
 
       language_a.cmp(language_b)
     };
@@ -385,10 +396,10 @@ mod tests {
         id: 3,
         source_title: "test 1".into(),
         event_type: SonarrHistoryEventType::Grabbed,
-        languages: vec![Language {
+        languages: vec![Some(Language {
           id: 1,
           name: "telgu".to_owned(),
-        }],
+        })],
         quality: QualityWrapper {
           quality: Quality {
             name: "HD - 1080p".to_owned(),
@@ -401,10 +412,10 @@ mod tests {
         id: 2,
         source_title: "test 2".into(),
         event_type: SonarrHistoryEventType::DownloadFolderImported,
-        languages: vec![Language {
+        languages: vec![Some(Language {
           id: 3,
           name: "chinese".to_owned(),
-        }],
+        })],
         quality: QualityWrapper {
           quality: Quality {
             name: "SD - 720p".to_owned(),
@@ -417,10 +428,10 @@ mod tests {
         id: 1,
         source_title: "test 3".into(),
         event_type: SonarrHistoryEventType::EpisodeFileDeleted,
-        languages: vec![Language {
+        languages: vec![Some(Language {
           id: 1,
           name: "english".to_owned(),
-        }],
+        })],
         quality: QualityWrapper {
           quality: Quality {
             name: "HD - 1080p".to_owned(),

src/handlers/sonarr_handlers/history/mod.rs

@@ -150,8 +150,19 @@ pub(in crate::handlers::sonarr_handlers) fn history_sorting_options(
           id: 1,
           name: "_".to_owned(),
         };
-        let language_a = &a.languages.first().unwrap_or(&default_language);
-        let language_b = &b.languages.first().unwrap_or(&default_language);
+        let default_language_option = Some(default_language.clone());
+        let language_a = &a
+          .languages
+          .first()
+          .unwrap_or(&default_language_option)
+          .as_ref()
+          .unwrap_or(&default_language);
+        let language_b = &b
+          .languages
+          .first()
+          .unwrap_or(&default_language_option)
+          .as_ref()
+          .unwrap_or(&default_language);
 
         language_a.cmp(language_b)
       }),

src/handlers/sonarr_handlers/library/season_details_handler.rs

@@ -515,12 +515,17 @@ pub(in crate::handlers::sonarr_handlers::library) fn releases_sorting_options(
     SortOption {
       name: "Language",
       cmp_fn: Some(|a, b| {
-        let default_language_vec = vec![Language {
+        let default_language = Language {
           id: 1,
           name: "_".to_owned(),
-        }];
-        let language_a = &a.languages.as_ref().unwrap_or(&default_language_vec)[0];
-        let language_b = &b.languages.as_ref().unwrap_or(&default_language_vec)[0];
+        };
+        let default_language_vec = vec![Some(default_language.clone())];
+        let language_a = &a.languages.as_ref().unwrap_or(&default_language_vec)[0]
+          .as_ref()
+          .unwrap_or(&default_language);
+        let language_b = &b.languages.as_ref().unwrap_or(&default_language_vec)[0]
+          .as_ref()
+          .unwrap_or(&default_language);
 
         language_a.cmp(language_b)
       }),

src/handlers/sonarr_handlers/library/season_details_handler_tests.rs

@@ -1116,12 +1116,17 @@ mod tests {
   #[test]
   fn test_releases_sorting_options_language() {
     let expected_cmp_fn: fn(&SonarrRelease, &SonarrRelease) -> Ordering = |a, b| {
-      let default_language_vec = vec![Language {
+      let default_language = Language {
         id: 1,
         name: "_".to_owned(),
-      }];
-      let language_a = &a.languages.as_ref().unwrap_or(&default_language_vec)[0];
-      let language_b = &b.languages.as_ref().unwrap_or(&default_language_vec)[0];
+      };
+      let default_language_vec = vec![Some(default_language.clone())];
+      let language_a = a.languages.as_ref().unwrap_or(&default_language_vec)[0]
+        .as_ref()
+        .unwrap_or(&default_language);
+      let language_b = b.languages.as_ref().unwrap_or(&default_language_vec)[0]
+        .as_ref()
+        .unwrap_or(&default_language);
 
       language_a.cmp(language_b)
     };
@@ -1160,10 +1165,10 @@ mod tests {
       size: 1,
       rejected: true,
       seeders: Some(Number::from(1)),
-      languages: Some(vec![Language {
+      languages: Some(vec![Some(Language {
         id: 1,
         name: "Language A".to_owned(),
-      }]),
+      })]),
       quality: QualityWrapper {
         quality: Quality {
           name: "Quality A".to_owned(),
@@ -1179,10 +1184,10 @@ mod tests {
       size: 2,
       rejected: false,
       seeders: Some(Number::from(2)),
-      languages: Some(vec![Language {
+      languages: Some(vec![Some(Language {
         id: 2,
         name: "Language B".to_owned(),
-      }]),
+      })]),
       quality: QualityWrapper {
         quality: Quality {
           name: "Quality B".to_owned(),

src/handlers/sonarr_handlers/sonarr_handler_test_utils.rs

@@ -228,7 +228,7 @@ pub(in crate::handlers::sonarr_handlers) mod utils {
       path: "/nfs/tv/series/season 1/episode 1.mkv".to_owned(),
       size: 3543348019,
       quality: quality_wrapper(),
-      languages: vec![language()],
+      languages: vec![Some(language())],
       date_added: DateTime::from(DateTime::parse_from_rfc3339("2024-02-10T07:28:45Z").unwrap()),
       media_info: Some(media_info()),
     }

src/models/sonarr_models.rs

@@ -84,7 +84,7 @@ pub struct BlocklistItem {
   pub series_title: Option<String>,
   pub episode_ids: Vec<Number>,
   pub source_title: String,
-  pub languages: Vec<Language>,
+  pub languages: Vec<Option<Language>>,
   pub quality: QualityWrapper,
   pub date: DateTime<Utc>,
   pub protocol: String,
@@ -215,7 +215,7 @@ pub struct EpisodeFile {
   pub path: String,
   #[serde(deserialize_with = "super::from_i64")]
   pub size: i64,
-  pub languages: Vec<Language>,
+  pub languages: Vec<Option<Language>>,
   pub quality: QualityWrapper,
   pub date_added: DateTime<Utc>,
   pub media_info: Option<MediaInfo>,
@@ -509,7 +509,7 @@ pub struct SonarrHistoryItem {
   #[serde(deserialize_with = "super::from_i64")]
   pub episode_id: i64,
   pub quality: QualityWrapper,
-  pub languages: Vec<Language>,
+  pub languages: Vec<Option<Language>>,
   pub date: DateTime<Utc>,
   pub event_type: SonarrHistoryEventType,
   pub data: SonarrHistoryData,
@@ -545,7 +545,7 @@ pub struct SonarrRelease {
   pub rejections: Option<Vec<String>>,
   pub seeders: Option<Number>,
   pub leechers: Option<Number>,
-  pub languages: Option<Vec<Language>>,
+  pub languages: Option<Vec<Option<Language>>>,
   pub quality: QualityWrapper,
   pub full_season: bool,
 }

src/network/sonarr_network/library/episodes/mod.rs

@@ -297,7 +297,13 @@ impl Network<'_, '_> {
             Date Added: {}",
             file.relative_path,
             file.path,
-            file.languages.first().unwrap_or(&Language::default()).name,
+            file
+              .languages
+              .first()
+              .unwrap_or(&Some(Language::default()))
+              .as_ref()
+              .unwrap_or(&Language::default())
+              .name,
             file.date_added,
           );
 

src/network/sonarr_network/sonarr_network_test_utils.rs

@@ -123,7 +123,7 @@ pub(in crate::network::sonarr_network) mod test_utils {
       series_title: None,
       episode_ids: vec![Number::from(1)],
       source_title: "Test Source Title".to_owned(),
-      languages: vec![language()],
+      languages: vec![Some(language())],
       quality: quality_wrapper(),
       date: DateTime::from(DateTime::parse_from_rfc3339("2024-02-10T07:28:45Z").unwrap()),
       protocol: "usenet".to_owned(),
@@ -180,7 +180,7 @@ pub(in crate::network::sonarr_network) mod test_utils {
       path: "/nfs/tv/series/season 1/episode 1.mkv".to_owned(),
       size: 3543348019,
       quality: quality_wrapper(),
-      languages: vec![language()],
+      languages: vec![Some(language())],
       date_added: DateTime::from(DateTime::parse_from_rfc3339("2024-02-10T07:28:45Z").unwrap()),
       media_info: Some(media_info()),
     }
@@ -206,7 +206,7 @@ pub(in crate::network::sonarr_network) mod test_utils {
       source_title: "Test source".into(),
       episode_id: 1,
       quality: quality_wrapper(),
-      languages: vec![language()],
+      languages: vec![Some(language())],
       date: DateTime::from(DateTime::parse_from_rfc3339("2024-02-10T07:28:45Z").unwrap()),
       event_type: SonarrHistoryEventType::Grabbed,
       data: history_data(),
@@ -377,7 +377,7 @@ pub(in crate::network::sonarr_network) mod test_utils {
       rejections: Some(rejections()),
       seeders: Some(Number::from(2)),
       leechers: Some(Number::from(1)),
-      languages: Some(vec![language()]),
+      languages: Some(vec![Some(language())]),
       quality: quality_wrapper(),
       full_season: false,
     }

src/ui/mod.rs

@@ -121,7 +121,7 @@ fn draw_error(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
   app.error.scroll_left_or_reset(
     area.width as usize,
     true,
-    app.tick_count % app.ticks_until_scroll == 0,
+    app.tick_count.is_multiple_of(app.ticks_until_scroll),
   );
 
   let paragraph = Paragraph::new(Text::from(app.error.to_string().failure()))

src/ui/radarr_ui/blocklist/mod.rs

@@ -97,7 +97,7 @@ fn draw_blocklist_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
       movie.title.scroll_left_or_reset(
         get_width_from_percentage(area, 20),
         current_selection == *blocklist_item,
-        app.tick_count % app.ticks_until_scroll == 0,
+        app.tick_count.is_multiple_of(app.ticks_until_scroll),
       );
 
       let languages_string = languages

src/ui/radarr_ui/collections/collection_details_ui.rs

@@ -90,7 +90,7 @@ pub fn draw_collection_details(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect)
     movie.title.scroll_left_or_reset(
       get_width_from_percentage(table_area, 20),
       current_selection == *movie,
-      app.tick_count % app.ticks_until_scroll == 0,
+      app.tick_count.is_multiple_of(app.ticks_until_scroll),
     );
     let (hours, minutes) = convert_runtime(movie.runtime);
     let imdb_rating = movie

src/ui/radarr_ui/collections/mod.rs

@@ -71,7 +71,7 @@ pub(super) fn draw_collections(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect)
       collection.title.scroll_left_or_reset(
         get_width_from_percentage(area, 25),
         *collection == current_selection,
-        app.tick_count % app.ticks_until_scroll == 0,
+        app.tick_count.is_multiple_of(app.ticks_until_scroll),
       );
       let monitored = if collection.monitored { "🏷" } else { "" };
       let search_on_add = if collection.search_on_add {

src/ui/radarr_ui/downloads/mod.rs

@@ -88,7 +88,7 @@ fn draw_downloads(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
       output_path.as_ref().unwrap().scroll_left_or_reset(
         get_width_from_percentage(area, 18),
         current_selection == *download_record,
-        app.tick_count % app.ticks_until_scroll == 0,
+        app.tick_count.is_multiple_of(app.ticks_until_scroll),
       );
     }
 

src/ui/radarr_ui/indexers/test_all_indexers_ui.rs

@@ -46,7 +46,7 @@ fn draw_test_all_indexers_test_results(f: &mut Frame<'_>, app: &mut App<'_>, are
     result.validation_failures.scroll_left_or_reset(
       get_width_from_percentage(area, 86),
       *result == current_selection,
-      app.tick_count % app.ticks_until_scroll == 0,
+      app.tick_count.is_multiple_of(app.ticks_until_scroll),
     );
     let pass_fail = if result.is_valid { "✔" } else { "❌" };
     let row = Row::new(vec![

src/ui/radarr_ui/library/add_movie_ui.rs

@@ -134,7 +134,7 @@ fn draw_add_movie_search(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
     movie.title.scroll_left_or_reset(
       get_width_from_percentage(area, 27),
       *movie == current_selection,
-      app.tick_count % app.ticks_until_scroll == 0,
+      app.tick_count.is_multiple_of(app.ticks_until_scroll),
     );
 
     Row::new(vec![

src/ui/radarr_ui/library/mod.rs

@@ -90,7 +90,7 @@ fn draw_library(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
       movie.title.scroll_left_or_reset(
         get_width_from_percentage(area, 27),
         *movie == current_selection,
-        app.tick_count % app.ticks_until_scroll == 0,
+        app.tick_count.is_multiple_of(app.ticks_until_scroll),
       );
       let monitored = if movie.monitored { "🏷" } else { "" };
       let studio = movie.studio.clone().unwrap_or_default();

src/ui/radarr_ui/library/movie_details_ui.rs

@@ -241,7 +241,7 @@ fn draw_movie_history(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
       movie_history_item.source_title.scroll_left_or_reset(
         get_width_from_percentage(area, 34),
         current_selection == *movie_history_item,
-        app.tick_count % app.ticks_until_scroll == 0,
+        app.tick_count.is_multiple_of(app.ticks_until_scroll),
       );
 
       Row::new(vec![
@@ -393,7 +393,7 @@ fn draw_movie_releases(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
         get_width_from_percentage(area, 30),
         current_selection == *release
           && current_route != ActiveRadarrBlock::ManualSearchConfirmPrompt.into(),
-        app.tick_count % app.ticks_until_scroll == 0,
+        app.tick_count.is_multiple_of(app.ticks_until_scroll),
       );
       let size = convert_to_gb(*size);
       let rejected_str = if *rejected { "⛔" } else { "" };

src/ui/sonarr_ui/blocklist/mod.rs

@@ -90,7 +90,7 @@ fn draw_blocklist_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
       let title = series_title.as_ref().unwrap_or(&String::new()).to_owned();
       let languages_string = languages
         .iter()
-        .map(|lang| lang.name.to_owned())
+        .map(|lang| lang.as_ref().unwrap_or(&Default::default()).name.to_owned())
         .collect::<Vec<String>>()
         .join(", ");
 

src/ui/sonarr_ui/downloads/mod.rs

@@ -88,7 +88,7 @@ fn draw_downloads(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
       output_path.as_ref().unwrap().scroll_left_or_reset(
         get_width_from_percentage(area, 18),
         current_selection == *download_record,
-        app.tick_count % app.ticks_until_scroll == 0,
+        app.tick_count.is_multiple_of(app.ticks_until_scroll),
       );
     }
 

src/ui/sonarr_ui/history/mod.rs

@@ -1,5 +1,6 @@
 use crate::app::App;
 use crate::models::servarr_data::sonarr::sonarr_data::{ActiveSonarrBlock, HISTORY_BLOCKS};
+use crate::models::servarr_models::Language;
 use crate::models::sonarr_models::{SonarrHistoryEventType, SonarrHistoryItem};
 use crate::models::Route;
 use crate::ui::styles::ManagarrStyle;
@@ -68,7 +69,7 @@ fn draw_history_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
       source_title.scroll_left_or_reset(
         get_width_from_percentage(area, 40),
         current_selection == *history_item,
-        app.tick_count % app.ticks_until_scroll == 0,
+        app.tick_count.is_multiple_of(app.ticks_until_scroll),
       );
 
       Row::new(vec![
@@ -77,7 +78,13 @@ fn draw_history_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
         Cell::from(
           languages
             .iter()
-            .map(|language| language.name.to_owned())
+            .map(|language| {
+              language
+                .as_ref()
+                .unwrap_or(&Language::default())
+                .name
+                .to_owned()
+            })
             .collect::<Vec<String>>()
             .join(","),
         ),

src/ui/sonarr_ui/indexers/test_all_indexers_ui.rs

@@ -44,7 +44,7 @@ fn draw_test_all_indexers_test_results(f: &mut Frame<'_>, app: &mut App<'_>, are
     result.validation_failures.scroll_left_or_reset(
       get_width_from_percentage(area, 86),
       *result == current_selection,
-      app.tick_count % app.ticks_until_scroll == 0,
+      app.tick_count.is_multiple_of(app.ticks_until_scroll),
     );
     let pass_fail = if result.is_valid { "✔" } else { "❌" };
     let row = Row::new(vec![

src/ui/sonarr_ui/library/add_series_ui.rs

@@ -120,7 +120,7 @@ fn draw_add_series_search(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
     series.title.scroll_left_or_reset(
       get_width_from_percentage(area, 27),
       *series == current_selection,
-      app.tick_count % app.ticks_until_scroll == 0,
+      app.tick_count.is_multiple_of(app.ticks_until_scroll),
     );
 
     Row::new(vec![

src/ui/sonarr_ui/library/episode_details_ui.rs

@@ -1,5 +1,6 @@
 use crate::app::App;
 use crate::models::servarr_data::sonarr::sonarr_data::{ActiveSonarrBlock, EPISODE_DETAILS_BLOCKS};
+use crate::models::servarr_models::Language;
 use crate::models::sonarr_models::{
   DownloadRecord, DownloadStatus, Episode, SonarrHistoryEventType, SonarrHistoryItem, SonarrRelease,
 };
@@ -271,7 +272,7 @@ fn draw_episode_history_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect)
           source_title.scroll_left_or_reset(
             get_width_from_percentage(area, 40),
             current_selection == *history_item,
-            app.tick_count % app.ticks_until_scroll == 0,
+            app.tick_count.is_multiple_of(app.ticks_until_scroll),
           );
 
           Row::new(vec![
@@ -280,7 +281,13 @@ fn draw_episode_history_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect)
             Cell::from(
               languages
                 .iter()
-                .map(|language| language.name.to_owned())
+                .map(|language| {
+                  language
+                    .as_ref()
+                    .unwrap_or(&Language::default())
+                    .name
+                    .to_owned()
+                })
                 .collect::<Vec<String>>()
                 .join(","),
             ),
@@ -415,7 +422,7 @@ fn draw_episode_releases(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
               get_width_from_percentage(area, 30),
               current_selection == *release
                 && active_sonarr_block != ActiveSonarrBlock::ManualEpisodeSearchConfirmPrompt,
-              app.tick_count % app.ticks_until_scroll == 0,
+              app.tick_count.is_multiple_of(app.ticks_until_scroll),
             );
             let size = convert_to_gb(*size);
             let rejected_str = if *rejected { "⛔" } else { "" };
@@ -441,7 +448,11 @@ fn draw_episode_releases(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
             };
 
             let language = if languages.is_some() {
-              languages.clone().unwrap()[0].name.clone()
+              languages.clone().unwrap()[0]
+                .as_ref()
+                .unwrap_or(&Default::default())
+                .name
+                .clone()
             } else {
               String::new()
             };

src/ui/sonarr_ui/library/mod.rs

@@ -95,7 +95,7 @@ fn draw_library(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
       series.title.scroll_left_or_reset(
         get_width_from_percentage(area, 23),
         *series == current_selection,
-        app.tick_count % app.ticks_until_scroll == 0,
+        app.tick_count.is_multiple_of(app.ticks_until_scroll),
       );
       let monitored = if series.monitored { "🏷" } else { "" };
       let certification = series.certification.clone().unwrap_or_default();

src/ui/sonarr_ui/library/season_details_ui.rs

@@ -267,7 +267,7 @@ fn draw_season_history_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
           source_title.scroll_left_or_reset(
             get_width_from_percentage(area, 40),
             current_selection == *history_item,
-            app.tick_count % app.ticks_until_scroll == 0,
+            app.tick_count.is_multiple_of(app.ticks_until_scroll),
           );
 
           Row::new(vec![
@@ -276,7 +276,13 @@ fn draw_season_history_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
             Cell::from(
               languages
                 .iter()
-                .map(|language| language.name.to_owned())
+                .map(|language| {
+                  language
+                    .as_ref()
+                    .unwrap_or(&Default::default())
+                    .name
+                    .to_owned()
+                })
                 .collect::<Vec<String>>()
                 .join(","),
             ),
@@ -372,7 +378,7 @@ fn draw_season_releases(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
             get_width_from_percentage(area, 30),
             current_selection == *release
               && active_sonarr_block != ActiveSonarrBlock::ManualSeasonSearchConfirmPrompt,
-            app.tick_count % app.ticks_until_scroll == 0,
+            app.tick_count.is_multiple_of(app.ticks_until_scroll),
           );
           let size = convert_to_gb(*size);
           let rejected_str = if *rejected { "⛔" } else { "" };
@@ -398,7 +404,11 @@ fn draw_season_releases(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
           };
 
           let language = if languages.is_some() {
-            languages.clone().unwrap()[0].name.clone()
+            languages.clone().unwrap()[0]
+              .as_ref()
+              .unwrap_or(&Default::default())
+              .name
+              .clone()
           } else {
             String::new()
           };

src/ui/sonarr_ui/library/series_details_ui.rs

@@ -315,7 +315,7 @@ fn draw_series_history_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
           source_title.scroll_left_or_reset(
             get_width_from_percentage(area, 40),
             current_selection == *history_item,
-            app.tick_count % app.ticks_until_scroll == 0,
+            app.tick_count.is_multiple_of(app.ticks_until_scroll),
           );
 
           Row::new(vec![
@@ -324,7 +324,13 @@ fn draw_series_history_table(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
             Cell::from(
               languages
                 .iter()
-                .map(|language| language.name.to_owned())
+                .map(|language| {
+                  language
+                    .as_ref()
+                    .unwrap_or(&Default::default())
+                    .name
+                    .to_owned()
+                })
                 .collect::<Vec<String>>()
                 .join(","),
             ),