mirror of
https://github.com/jellyfin/jellyfin.git
synced 2025-04-24 05:57:20 -04:00
Merge pull request #2903 from randrey/dlna-albumart-fix
Fix DLNA clients displaying wrong album art.
(cherry picked from commit 1cc5d6745a
)
Signed-off-by: Joshua M. Boniface <joshua@boniface.me>
This commit is contained in:
parent
3044dfc114
commit
d8f865e93c
1 changed files with 46 additions and 7 deletions
|
@ -1006,19 +1006,58 @@ namespace Emby.Dlna.Didl
|
|||
}
|
||||
}
|
||||
|
||||
item = item.GetParents().FirstOrDefault(i => i.HasImage(ImageType.Primary));
|
||||
|
||||
if (item != null)
|
||||
// For audio tracks without art use album art if available.
|
||||
if (item is Audio audioItem)
|
||||
{
|
||||
if (item.HasImage(ImageType.Primary))
|
||||
{
|
||||
return GetImageInfo(item, ImageType.Primary);
|
||||
}
|
||||
var album = audioItem.AlbumEntity;
|
||||
return album != null && album.HasImage(ImageType.Primary)
|
||||
? GetImageInfo(album, ImageType.Primary)
|
||||
: null;
|
||||
}
|
||||
|
||||
// Don't look beyond album/playlist level. Metadata service may assign an image from a different album/show to the parent folder.
|
||||
if (item is MusicAlbum || item is Playlist)
|
||||
{
|
||||
return null;
|
||||
}
|
||||
|
||||
// For other item types check parents, but be aware that image retrieved from a parent may be not suitable for this media item.
|
||||
var parentWithImage = GetFirstParentWithImageBelowUserRoot(item);
|
||||
if (parentWithImage != null)
|
||||
{
|
||||
return GetImageInfo(parentWithImage, ImageType.Primary);
|
||||
}
|
||||
|
||||
return null;
|
||||
}
|
||||
|
||||
private BaseItem GetFirstParentWithImageBelowUserRoot(BaseItem item)
|
||||
{
|
||||
if (item == null)
|
||||
{
|
||||
return null;
|
||||
}
|
||||
|
||||
if (item.HasImage(ImageType.Primary))
|
||||
{
|
||||
return item;
|
||||
}
|
||||
|
||||
var parent = item.GetParent();
|
||||
if (parent is UserRootFolder)
|
||||
{
|
||||
return null;
|
||||
}
|
||||
|
||||
// terminate in case we went past user root folder (unlikely?)
|
||||
if (parent is Folder folder && folder.IsRoot)
|
||||
{
|
||||
return null;
|
||||
}
|
||||
|
||||
return GetFirstParentWithImageBelowUserRoot(parent);
|
||||
}
|
||||
|
||||
private ImageDownloadInfo GetImageInfo(BaseItem item, ImageType type)
|
||||
{
|
||||
var imageInfo = item.GetImageInfo(type, 0);
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue