[PR #4220] [MERGED] Fix invalid operation exception in TvdbEpisodeImageProvider.GetImages #9953

Closed
opened 2026-02-07 06:11:26 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4220
Author: @Spacetech
Created: 9/28/2020
Status: Merged
Merged: 9/28/2020
Merged by: @dkanada

Base: masterHead: TvdbEpisodeImageProvider_nullable_check


📝 Commits (1)

  • 12275e5 Fix invalid operation exception in TvdbEpisodeImageProvider.GetImages

📊 Changes

1 file changed (+17 additions, -10 deletions)

View changed files

📝 MediaBrowser.Providers/Plugins/TheTvdb/TvdbEpisodeImageProvider.cs (+17 -10)

📄 Description

Depending on the TVDB data for a series / episode, this fixes the following error during library scans:

[2020-09-27 15:54:47.517 -07:00] [ERR] [10] MediaBrowser.Providers.Manager.ProviderManager: "TvdbEpisodeImageProvider" failed in GetImageInfos for type "Episode" at "%xyz%"
System.InvalidOperationException: Nullable object must have a value.
   at System.Nullable`1.get_Value()
   at MediaBrowser.Providers.Plugins.TheTvdb.TvdbEpisodeImageProvider.GetImages(BaseItem item, CancellationToken cancellationToken) in C:\Programming\jellyfin\MediaBrowser.Providers\Plugins\TheTvdb\TvdbEpisodeImageProvider.cs:line 60
   at MediaBrowser.Providers.Manager.ProviderManager.GetImages(BaseItem item, IRemoteImageProvider provider, IReadOnlyCollection`1 preferredLanguages, CancellationToken cancellationToken, Nullable`1 type) in C:\Programming\jellyfin\MediaBrowser.Providers\Manager\ProviderManager.cs:line 263

Changes

  • Confirm IndexNumber & ParentIndexNumber have a value before using them
  • You'll now see a log line showing that one of those numbers was null instead of the exception:
    [2020-09-27 16:05:50.015 -07:00] [ERR] [101] MediaBrowser.Providers.Plugins.TheTvdb.TvdbEpisodeImageProvider: Episode 1xnull not found for series "%xyz%"

Issues

  • The same error was mentioned in a comment for #2355, but this change is unrelated to the main issue.

🔄 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/jellyfin/jellyfin/pull/4220 **Author:** [@Spacetech](https://github.com/Spacetech) **Created:** 9/28/2020 **Status:** ✅ Merged **Merged:** 9/28/2020 **Merged by:** [@dkanada](https://github.com/dkanada) **Base:** `master` ← **Head:** `TvdbEpisodeImageProvider_nullable_check` --- ### 📝 Commits (1) - [`12275e5`](https://github.com/jellyfin/jellyfin/commit/12275e5e7bfe46e407606a977f1ca3a54fb9e62c) Fix invalid operation exception in TvdbEpisodeImageProvider.GetImages ### 📊 Changes **1 file changed** (+17 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Providers/Plugins/TheTvdb/TvdbEpisodeImageProvider.cs` (+17 -10) </details> ### 📄 Description Depending on the TVDB data for a series / episode, this fixes the following error during library scans: ``` [2020-09-27 15:54:47.517 -07:00] [ERR] [10] MediaBrowser.Providers.Manager.ProviderManager: "TvdbEpisodeImageProvider" failed in GetImageInfos for type "Episode" at "%xyz%" System.InvalidOperationException: Nullable object must have a value. at System.Nullable`1.get_Value() at MediaBrowser.Providers.Plugins.TheTvdb.TvdbEpisodeImageProvider.GetImages(BaseItem item, CancellationToken cancellationToken) in C:\Programming\jellyfin\MediaBrowser.Providers\Plugins\TheTvdb\TvdbEpisodeImageProvider.cs:line 60 at MediaBrowser.Providers.Manager.ProviderManager.GetImages(BaseItem item, IRemoteImageProvider provider, IReadOnlyCollection`1 preferredLanguages, CancellationToken cancellationToken, Nullable`1 type) in C:\Programming\jellyfin\MediaBrowser.Providers\Manager\ProviderManager.cs:line 263 ``` **Changes** - Confirm `IndexNumber` & `ParentIndexNumber` have a value before using them - You'll now see a log line showing that one of those numbers was null instead of the exception: ```[2020-09-27 16:05:50.015 -07:00] [ERR] [101] MediaBrowser.Providers.Plugins.TheTvdb.TvdbEpisodeImageProvider: Episode 1xnull not found for series "%xyz%"``` **Issues** - The same error was mentioned in a comment for #2355, but this change is unrelated to the main issue. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:11:26 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9953