[PR #14567] [MERGED] Don't fail image saving on missing BaseItem #14032

Closed
opened 2026-02-07 07:23:15 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/14567
Author: @Shadowghost
Created: 8/2/2025
Status: Merged
Merged: 8/3/2025
Merged by: @nielsvanvelzen

Base: masterHead: skip-image-failure


📝 Commits (1)

  • 803e87c Don't fail image saving on missing BaseItem

📊 Changes

1 file changed (+7 additions, -0 deletions)

View changed files

📝 Jellyfin.Server.Implementations/Item/BaseItemRepository.cs (+7 -0)

📄 Description

I know it's a hack but this currently prevents my scans from completing and I was unable to find the root cause. This at least doesn't break the scans


🔄 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/14567 **Author:** [@Shadowghost](https://github.com/Shadowghost) **Created:** 8/2/2025 **Status:** ✅ Merged **Merged:** 8/3/2025 **Merged by:** [@nielsvanvelzen](https://github.com/nielsvanvelzen) **Base:** `master` ← **Head:** `skip-image-failure` --- ### 📝 Commits (1) - [`803e87c`](https://github.com/jellyfin/jellyfin/commit/803e87ca5fa3aac02bf327384b7f02a182a4fa40) Don't fail image saving on missing BaseItem ### 📊 Changes **1 file changed** (+7 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Server.Implementations/Item/BaseItemRepository.cs` (+7 -0) </details> ### 📄 Description I know it's a hack but this currently prevents my scans from completing and I was unable to find the root cause. This at least doesn't break the scans --- <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 07:23:15 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#14032