From 292240df46836f784a4007dc1077608510f6b24e Mon Sep 17 00:00:00 2001 From: Bill Thornton Date: Tue, 21 May 2024 16:18:42 -0400 Subject: [PATCH] Fix missing policy check for download all --- src/components/itemContextMenu.js | 45 ++++++++++++++++--------------- 1 file changed, 24 insertions(+), 21 deletions(-) diff --git a/src/components/itemContextMenu.js b/src/components/itemContextMenu.js index 263a5571642..fe0f063682d 100644 --- a/src/components/itemContextMenu.js +++ b/src/components/itemContextMenu.js @@ -169,12 +169,30 @@ export function getCommands(options) { }); } - if (item.Type === 'Season' || item.Type == 'Series') { - commands.push({ - name: globalize.translate('DownloadAll'), - id: 'downloadall', - icon: 'file_download' - }); + if (appHost.supports('filedownload')) { + // CanDownload should probably be updated to return true for these items? + if (user.Policy.EnableContentDownloading && (item.Type === 'Season' || item.Type == 'Series')) { + commands.push({ + name: globalize.translate('DownloadAll'), + id: 'downloadall', + icon: 'file_download' + }); + } + + // Books are promoted to major download Button and therefor excluded in the context menu + if (item.CanDownload && item.Type !== 'Book') { + commands.push({ + name: globalize.translate('Download'), + id: 'download', + icon: 'file_download' + }); + + commands.push({ + name: globalize.translate('CopyStreamURL'), + id: 'copy-stream', + icon: 'content_copy' + }); + } } if (item.CanDelete && options.deleteItem !== false) { @@ -193,21 +211,6 @@ export function getCommands(options) { } } - // Books are promoted to major download Button and therefor excluded in the context menu - if ((item.CanDownload && appHost.supports('filedownload')) && item.Type !== 'Book') { - commands.push({ - name: globalize.translate('Download'), - id: 'download', - icon: 'file_download' - }); - - commands.push({ - name: globalize.translate('CopyStreamURL'), - id: 'copy-stream', - icon: 'content_copy' - }); - } - if (commands.length) { commands.push({ divider: true