[PR #26] [MERGED] Workaround for JF crashing when scanning Series #65

New issue

Closed

opened 2026-03-23 20:35:36 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 20:35:36 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/tubearchivist/tubearchivist-jf-plugin/pull/26
Author: @wolffshots
Created: 10/22/2024
Status: ✅ Merged
Merged: 11/4/2024
Merged by: @DarkFighterLuke

Base: master ← Head: workaround-series-crash

📝 Commits (1)

2fee654 chore: wrap itemYTId stuff in try catch

📊 Changes

1 file changed (+16 additions, -8 deletions)

View changed files

📝 Jellyfin.Plugin.TubeArchivistMetadata/Plugin.cs (+16 -8)

📄 Description

This just wraps the problematic code (as noted in #25) in a try catch to stop the server from crashing and logs a line for the item.

The end result is to still show the error but just continue as seen in the logs here:

[2024-10-21 17:11:40.376 +02:00] [ERR] Error while processing item path: "null"
System.ArgumentNullException: Value cannot be null. (Parameter 'source')
   at System.Linq.ThrowHelper.ThrowArgumentNullException(ExceptionArgument argument)
   at System.Linq.Enumerable.Count[TSource](IEnumerable`1 source, Func`2 predicate)
   at Jellyfin.Plugin.TubeArchivistMetadata.Utilities.Utils.DetectDirectorySeparator(String path)
   at Jellyfin.Plugin.TubeArchivistMetadata.Utilities.Utils.GetVideoNameFromPath(String path)
   at Jellyfin.Plugin.TubeArchivistMetadata.Plugin.OnWatchedStatusChange(Object sender, UserDataSaveEventArgs eventArgs)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/tubearchivist/tubearchivist-jf-plugin/pull/26 **Author:** [@wolffshots](https://github.com/wolffshots) **Created:** 10/22/2024 **Status:** ✅ Merged **Merged:** 11/4/2024 **Merged by:** [@DarkFighterLuke](https://github.com/DarkFighterLuke) **Base:** `master` ← **Head:** `workaround-series-crash` --- ### 📝 Commits (1) - [`2fee654`](https://github.com/tubearchivist/tubearchivist-jf-plugin/commit/2fee654efee09eac192babc57a0d9224fa61d3ee) chore: wrap itemYTId stuff in try catch ### 📊 Changes **1 file changed** (+16 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Plugin.TubeArchivistMetadata/Plugin.cs` (+16 -8) </details> ### 📄 Description This just wraps the problematic code (as noted in #25) in a try catch to stop the server from crashing and logs a line for the item. The end result is to still show the error but just continue as seen in the logs here: ``` [2024-10-21 17:11:40.376 +02:00] [ERR] Error while processing item path: "null" System.ArgumentNullException: Value cannot be null. (Parameter 'source') at System.Linq.ThrowHelper.ThrowArgumentNullException(ExceptionArgument argument) at System.Linq.Enumerable.Count[TSource](IEnumerable`1 source, Func`2 predicate) at Jellyfin.Plugin.TubeArchivistMetadata.Utilities.Utils.DetectDirectorySeparator(String path) at Jellyfin.Plugin.TubeArchivistMetadata.Utilities.Utils.GetVideoNameFromPath(String path) at Jellyfin.Plugin.TubeArchivistMetadata.Plugin.OnWatchedStatusChange(Object sender, UserDataSaveEventArgs eventArgs) ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>