mirror of
https://github.com/jellyfin/jellyfin.git
synced 2025-05-24 02:02:29 -04:00
Ensure existing chapter images are deleted when user disables chapter generation
This commit is contained in:
parent
776294b5f8
commit
8e5bd36064
@ -15,6 +15,7 @@ using MediaBrowser.Controller.Entities;
|
||||
using MediaBrowser.Controller.Library;
|
||||
using MediaBrowser.Controller.MediaEncoding;
|
||||
using MediaBrowser.Controller.Providers;
|
||||
using MediaBrowser.Model.Configuration;
|
||||
using MediaBrowser.Model.Dto;
|
||||
using MediaBrowser.Model.Entities;
|
||||
using MediaBrowser.Model.IO;
|
||||
@ -62,23 +63,16 @@ namespace Emby.Server.Implementations.MediaEncoder
|
||||
/// Determines whether [is eligible for chapter image extraction] [the specified video].
|
||||
/// </summary>
|
||||
/// <param name="video">The video.</param>
|
||||
/// <param name="libraryOptions"></param>
|
||||
/// <returns><c>true</c> if [is eligible for chapter image extraction] [the specified video]; otherwise, <c>false</c>.</returns>
|
||||
private bool IsEligibleForChapterImageExtraction(Video video)
|
||||
private bool IsEligibleForChapterImageExtraction(Video video, LibraryOptions libraryOptions)
|
||||
{
|
||||
if (video.IsPlaceHolder)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
var libraryOptions = _libraryManager.GetLibraryOptions(video);
|
||||
if (libraryOptions is not null)
|
||||
{
|
||||
if (!libraryOptions.EnableChapterImageExtraction)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
}
|
||||
else
|
||||
if (libraryOptions is null || !libraryOptions.EnableChapterImageExtraction)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
@ -99,7 +93,9 @@ namespace Emby.Server.Implementations.MediaEncoder
|
||||
|
||||
public async Task<bool> RefreshChapterImages(Video video, IDirectoryService directoryService, IReadOnlyList<ChapterInfo> chapters, bool extractImages, bool saveChapters, CancellationToken cancellationToken)
|
||||
{
|
||||
if (!IsEligibleForChapterImageExtraction(video))
|
||||
var libraryOptions = _libraryManager.GetLibraryOptions(video);
|
||||
|
||||
if (!IsEligibleForChapterImageExtraction(video, libraryOptions))
|
||||
{
|
||||
extractImages = false;
|
||||
}
|
||||
@ -179,6 +175,12 @@ namespace Emby.Server.Implementations.MediaEncoder
|
||||
chapter.ImageDateModified = _fileSystem.GetLastWriteTimeUtc(path);
|
||||
changesMade = true;
|
||||
}
|
||||
else if (libraryOptions?.EnableChapterImageExtraction != true)
|
||||
{
|
||||
// We have an image for the current chapter but the user has disabled chapter image extraction -> delete this chapter's image
|
||||
chapter.ImagePath = null;
|
||||
changesMade = true;
|
||||
}
|
||||
}
|
||||
|
||||
if (saveChapters && changesMade)
|
||||
|
Loading…
x
Reference in New Issue
Block a user