[PR #4612] [MERGED] Enable nullable for MediaBrowser.XbmcMetadata project #10191

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

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4612
Author: @martinek-stepan
Created: 11/28/2020
Status: Merged
Merged: 12/31/2020
Merged by: @Bond-009

Base: masterHead: nullable-xbmc


📝 Commits (3)

  • 52dc902 Enable nullable for MediaBrowser.XbmcMetadata project
  • 8e5b532 Missed errors.
  • 4e13959 Update MediaBrowser.XbmcMetadata/Savers/BaseNfoSaver.cs

📊 Changes

13 files changed (+43 additions, -27 deletions)

View changed files

📝 Jellyfin.Data/Jellyfin.Data.csproj (+2 -2)
📝 MediaBrowser.Model/MediaBrowser.Model.csproj (+1 -1)
📝 MediaBrowser.Model/Providers/ExternalIdInfo.cs (+18 -3)
📝 MediaBrowser.Providers/Manager/ProviderManager.cs (+5 -7)
📝 MediaBrowser.XbmcMetadata/EntryPoint.cs (+1 -1)
📝 MediaBrowser.XbmcMetadata/MediaBrowser.XbmcMetadata.csproj (+1 -0)
📝 MediaBrowser.XbmcMetadata/Parsers/BaseNfoParser.cs (+5 -4)
📝 MediaBrowser.XbmcMetadata/Parsers/MovieNfoParser.cs (+2 -2)
📝 MediaBrowser.XbmcMetadata/Parsers/SeriesNfoParser.cs (+3 -3)
📝 MediaBrowser.XbmcMetadata/Providers/BaseNfoProvider.cs (+1 -1)
📝 MediaBrowser.XbmcMetadata/Providers/BaseVideoNfoProvider.cs (+1 -1)
📝 MediaBrowser.XbmcMetadata/Savers/BaseNfoSaver.cs (+2 -1)
📝 MediaBrowser.XbmcMetadata/Savers/MovieNfoSaver.cs (+1 -1)

📄 Description

Changes

Enable nullable on the project level and reflect changes in code.

Issues

Part of issue #2768


🔄 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/4612 **Author:** [@martinek-stepan](https://github.com/martinek-stepan) **Created:** 11/28/2020 **Status:** ✅ Merged **Merged:** 12/31/2020 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `nullable-xbmc` --- ### 📝 Commits (3) - [`52dc902`](https://github.com/jellyfin/jellyfin/commit/52dc902a3b4ec2d0b94f2dff86cc117ed26c1f42) Enable nullable for MediaBrowser.XbmcMetadata project - [`8e5b532`](https://github.com/jellyfin/jellyfin/commit/8e5b532d3272c8df64377aa12a0ace1dbea017a1) Missed errors. - [`4e13959`](https://github.com/jellyfin/jellyfin/commit/4e1395930297d25814345dd9e97709dc817865c2) Update MediaBrowser.XbmcMetadata/Savers/BaseNfoSaver.cs ### 📊 Changes **13 files changed** (+43 additions, -27 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Data/Jellyfin.Data.csproj` (+2 -2) 📝 `MediaBrowser.Model/MediaBrowser.Model.csproj` (+1 -1) 📝 `MediaBrowser.Model/Providers/ExternalIdInfo.cs` (+18 -3) 📝 `MediaBrowser.Providers/Manager/ProviderManager.cs` (+5 -7) 📝 `MediaBrowser.XbmcMetadata/EntryPoint.cs` (+1 -1) 📝 `MediaBrowser.XbmcMetadata/MediaBrowser.XbmcMetadata.csproj` (+1 -0) 📝 `MediaBrowser.XbmcMetadata/Parsers/BaseNfoParser.cs` (+5 -4) 📝 `MediaBrowser.XbmcMetadata/Parsers/MovieNfoParser.cs` (+2 -2) 📝 `MediaBrowser.XbmcMetadata/Parsers/SeriesNfoParser.cs` (+3 -3) 📝 `MediaBrowser.XbmcMetadata/Providers/BaseNfoProvider.cs` (+1 -1) 📝 `MediaBrowser.XbmcMetadata/Providers/BaseVideoNfoProvider.cs` (+1 -1) 📝 `MediaBrowser.XbmcMetadata/Savers/BaseNfoSaver.cs` (+2 -1) 📝 `MediaBrowser.XbmcMetadata/Savers/MovieNfoSaver.cs` (+1 -1) </details> ### 📄 Description <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** <!-- Describe your changes here in 1-5 sentences. --> Enable nullable on the project level and reflect changes in code. **Issues** <!-- Tag any issues that this PR solves here. ex. Fixes # --> Part of issue #2768 --- <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:15:41 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10191