pt
/
jellyfin
mirror of https://github.com/jellyfin/jellyfin.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix multiple NFO backdrops 

and fix screen grabber overwriting local JPGs for video primary images.
Linting: Change to ICollection


Revert changes for different issue fix
This commit is contained in:
SenorSmartyPants 2023-03-28 15:14:26 -05:00
parent 3229d3ba02
commit 4e8f42f457
2 changed files with 18 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
MediaBrowser.Providers/Manager/ItemImageProvider.cs
View File

@ -357,7 +357,12 @@ namespace MediaBrowser.Providers.Manager
return true;
}
private void PruneImages(BaseItem item, IReadOnlyList<ItemImageInfo> images)
/// <summary>
/// Delete images from an item.
/// </summary>
/// <param name="item">The item.</param>
/// <param name="images">The images.</param>
public void PruneImages(BaseItem item, IReadOnlyList<ItemImageInfo> images)
{
for (var i = 0; i < images.Count; i++)
{

12
MediaBrowser.Providers/Manager/MetadataService.cs
View File

@ -688,6 +688,12 @@ namespace MediaBrowser.Providers.Manager
if (localItem.HasMetadata)
{
var oldBackdropImages = Array.Empty<ItemImageInfo>();
if (options.IsReplacingImage(ImageType.Backdrop))
{
oldBackdropImages = item.GetImages(ImageType.Backdrop).ToArray();
}
foreach (var remoteImage in localItem.RemoteImages)
{
try
@ -710,6 +716,12 @@ namespace MediaBrowser.Providers.Manager
}
}
// only delete existing multi-images if new ones were added
if (oldBackdropImages.Length > 0 && oldBackdropImages.Length < item.GetImages(ImageType.Backdrop).Count())
{
imageService.PruneImages(item, oldBackdropImages);
}
if (foundImageTypes.Count > 0)
{
imageService.UpdateReplaceImages(options, foundImageTypes);