diff --git a/server/controllers/RSSFeedController.js b/server/controllers/RSSFeedController.js index 9732368d..b9a08e11 100644 --- a/server/controllers/RSSFeedController.js +++ b/server/controllers/RSSFeedController.js @@ -18,7 +18,7 @@ class RSSFeedController { } // Check request body options exist - if (!options.serverAddress || !options.slug || options.metadataDetails == null) { + if (!options.serverAddress || !options.slug) { Logger.error(`[RSSFeedController] Invalid request body to open RSS feed`) return res.status(400).send('Invalid request body') } @@ -49,7 +49,7 @@ class RSSFeedController { if (!collection) return res.sendStatus(404) // Check request body options exist - if (!options.serverAddress || !options.slug || options.metadataDetails == null) { + if (!options.serverAddress || !options.slug) { Logger.error(`[RSSFeedController] Invalid request body to open RSS feed`) return res.status(400).send('Invalid request body') } @@ -83,7 +83,7 @@ class RSSFeedController { if (!series) return res.sendStatus(404) // Check request body options exist - if (!options.serverAddress || !options.slug || options.metadataDetails == null) { + if (!options.serverAddress || !options.slug) { Logger.error(`[RSSFeedController] Invalid request body to open RSS feed`) return res.status(400).send('Invalid request body') } diff --git a/server/managers/RssFeedManager.js b/server/managers/RssFeedManager.js index f5f74313..9325a7a2 100644 --- a/server/managers/RssFeedManager.js +++ b/server/managers/RssFeedManager.js @@ -188,9 +188,9 @@ class RssFeedManager { async openFeedForItem(user, libraryItem, options) { const serverAddress = options.serverAddress const slug = options.slug - const preventIndexing = options.metadataDetails.preventIndexing - const ownerName = options.metadataDetails.ownerName - const ownerEmail = options.metadataDetails.ownerEmail + const preventIndexing = options.metadataDetails?.preventIndexing ?? true + const ownerName = options.metadataDetails?.ownerName + const ownerEmail = options.metadataDetails?.ownerEmail const feed = new Feed() feed.setFromItem(user.id, slug, libraryItem, serverAddress, preventIndexing, ownerName, ownerEmail) @@ -205,9 +205,9 @@ class RssFeedManager { async openFeedForCollection(user, collectionExpanded, options) { const serverAddress = options.serverAddress const slug = options.slug - const preventIndexing = options.metadataDetails.preventIndexing - const ownerName = options.metadataDetails.ownerName - const ownerEmail = options.metadataDetails.ownerEmail + const preventIndexing = options.metadataDetails?.preventIndexing ?? true + const ownerName = options.metadataDetails?.ownerName + const ownerEmail = options.metadataDetails?.ownerEmail const feed = new Feed() feed.setFromCollection(user.id, slug, collectionExpanded, serverAddress, preventIndexing, ownerName, ownerEmail) @@ -222,9 +222,9 @@ class RssFeedManager { async openFeedForSeries(user, seriesExpanded, options) { const serverAddress = options.serverAddress const slug = options.slug - const preventIndexing = options.metadataDetails.preventIndexing - const ownerName = options.metadataDetails.ownerName - const ownerEmail = options.metadataDetails.ownerEmail + const preventIndexing = options.metadataDetails?.preventIndexing ?? true + const ownerName = options.metadataDetails?.ownerName + const ownerEmail = options.metadataDetails?.ownerEmail const feed = new Feed() feed.setFromSeries(user.id, slug, seriesExpanded, serverAddress, preventIndexing, ownerName, ownerEmail)