mirror of
https://github.com/advplyr/audiobookshelf.git
synced 2026-05-13 06:51:29 +00:00
Prevent loading full podcast when adding episodes to playlist
This commit is contained in:
parent
8b89b27654
commit
2bc6dcf576
2 changed files with 117 additions and 54 deletions
105
server/utils/playlistHelpers.js
Normal file
105
server/utils/playlistHelpers.js
Normal file
|
|
@ -0,0 +1,105 @@
|
|||
const Database = require('../Database')
|
||||
|
||||
async function getPodcastEpisodesWithLibraryItems(episodeIds) {
|
||||
if (!episodeIds.length) return []
|
||||
|
||||
return Database.podcastEpisodeModel.findAll({
|
||||
where: {
|
||||
id: episodeIds
|
||||
},
|
||||
include: [
|
||||
{
|
||||
model: Database.podcastModel,
|
||||
include: [
|
||||
{
|
||||
model: Database.libraryItemModel
|
||||
}
|
||||
]
|
||||
}
|
||||
]
|
||||
})
|
||||
}
|
||||
|
||||
async function resolvePlaylistRequestItems(items, libraryId) {
|
||||
const libraryItemIds = Array.from(new Set(items.map((i) => i.libraryItemId).filter((i) => i)))
|
||||
const episodeIds = Array.from(new Set(items.map((i) => i.episodeId).filter((i) => i)))
|
||||
const bookLibraryItemIds = Array.from(new Set(items.filter((i) => !i.episodeId).map((i) => i.libraryItemId)))
|
||||
|
||||
const libraryItems = await Database.libraryItemModel.findAll({
|
||||
attributes: ['id', 'mediaId', 'mediaType', 'libraryId'],
|
||||
where: {
|
||||
id: libraryItemIds,
|
||||
libraryId
|
||||
}
|
||||
})
|
||||
if (libraryItems.length !== libraryItemIds.length) {
|
||||
return null
|
||||
}
|
||||
|
||||
const libraryItemsById = new Map(libraryItems.map((libraryItem) => [libraryItem.id, libraryItem]))
|
||||
|
||||
// Books still need their fully expanded library item because the playlist response embeds the whole book object.
|
||||
const bookLibraryItems = bookLibraryItemIds.length
|
||||
? await Database.libraryItemModel.findAllExpandedWhere({
|
||||
id: bookLibraryItemIds,
|
||||
libraryId,
|
||||
mediaType: 'book'
|
||||
})
|
||||
: []
|
||||
const bookLibraryItemsById = new Map(bookLibraryItems.map((libraryItem) => [libraryItem.id, libraryItem]))
|
||||
|
||||
// For podcast adds, load only the requested episodes plus their owning podcast/library item.
|
||||
// The old expanded library-item query pulled every episode for the podcast, which is what blew up memory.
|
||||
const podcastEpisodes = await getPodcastEpisodesWithLibraryItems(episodeIds)
|
||||
if (podcastEpisodes.length !== episodeIds.length) {
|
||||
return null
|
||||
}
|
||||
const podcastEpisodesById = new Map(podcastEpisodes.map((episode) => [episode.id, episode]))
|
||||
|
||||
return items.map((item) => {
|
||||
const libraryItem = libraryItemsById.get(item.libraryItemId)
|
||||
if (!libraryItem) return null
|
||||
|
||||
if (item.episodeId) {
|
||||
const episode = podcastEpisodesById.get(item.episodeId)
|
||||
if (libraryItem.mediaType !== 'podcast' || !episode?.podcast?.libraryItem || episode.podcast.libraryItem.id !== item.libraryItemId) {
|
||||
return null
|
||||
}
|
||||
|
||||
const episodeLibraryItem = episode.podcast.libraryItem
|
||||
episodeLibraryItem.media = episode.podcast
|
||||
|
||||
return {
|
||||
item,
|
||||
mediaItemId: item.episodeId,
|
||||
mediaItemType: 'podcastEpisode',
|
||||
jsonItem: {
|
||||
episodeId: item.episodeId,
|
||||
episode: episode.toOldJSONExpanded(episodeLibraryItem.id),
|
||||
libraryItemId: episodeLibraryItem.id,
|
||||
libraryItem: episodeLibraryItem.toOldJSONMinified()
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
const expandedBookLibraryItem = bookLibraryItemsById.get(item.libraryItemId)
|
||||
if (libraryItem.mediaType !== 'book' || !expandedBookLibraryItem) {
|
||||
return null
|
||||
}
|
||||
|
||||
return {
|
||||
item,
|
||||
mediaItemId: expandedBookLibraryItem.media.id,
|
||||
mediaItemType: 'book',
|
||||
jsonItem: {
|
||||
libraryItemId: expandedBookLibraryItem.id,
|
||||
libraryItem: expandedBookLibraryItem.toOldJSONExpanded()
|
||||
}
|
||||
}
|
||||
})
|
||||
}
|
||||
|
||||
module.exports = {
|
||||
getPodcastEpisodesWithLibraryItems,
|
||||
resolvePlaylistRequestItems
|
||||
}
|
||||
Loading…
Add table
Add a link
Reference in a new issue