diff --git a/apps/files/src/components/FileEntry/FileEntryActions.vue b/apps/files/src/components/FileEntry/FileEntryActions.vue index f8fde7842a879..5f4a115b83afe 100644 --- a/apps/files/src/components/FileEntry/FileEntryActions.vue +++ b/apps/files/src/components/FileEntry/FileEntryActions.vue @@ -163,7 +163,14 @@ export default defineComponent({ if (this.filesListWidth < 768 || this.gridMode) { return [] } - return this.enabledFileActions.filter(action => action?.inline?.(this.source, this.currentView)) + return this.enabledFileActions.filter(action => { + try { + return action?.inline?.(this.source, this.currentView) + } catch (e) { + logger.error('Error while checking if action is inline', { action, e }) + return false + } + }) }, // Enabled action that are displayed inline with a custom render function @@ -236,13 +243,18 @@ export default defineComponent({ methods: { actionDisplayName(action: FileAction) { - if ((this.gridMode || (this.filesListWidth < 768 && action.inline)) && typeof action.title === 'function') { - // if an inline action is rendered in the menu for - // lack of space we use the title first if defined - const title = action.title([this.source], this.currentView) - if (title) return title + try { + if ((this.gridMode || (this.filesListWidth < 768 && action.inline)) && typeof action.title === 'function') { + // if an inline action is rendered in the menu for + // lack of space we use the title first if defined + const title = action.title([this.source], this.currentView) + if (title) return title + } + return action.displayName([this.source], this.currentView) + } catch (e) { + logger.error('Error while getting action display name', { action, e }) + return '' } - return action.displayName([this.source], this.currentView) }, async onActionClick(action, isSubmenu = false) { @@ -257,7 +269,13 @@ export default defineComponent({ return } - const displayName = action.displayName([this.source], this.currentView) + let displayName = action.id + try { + displayName = action.displayName([this.source], this.currentView) + } catch (e) { + logger.error('Error while getting action display name', { action, e }) + } + try { // Set the loading marker this.$emit('update:loading', action.id) diff --git a/apps/files/src/components/FileEntryMixin.ts b/apps/files/src/components/FileEntryMixin.ts index 4a4e4f497b5ec..08dfd40267732 100644 --- a/apps/files/src/components/FileEntryMixin.ts +++ b/apps/files/src/components/FileEntryMixin.ts @@ -6,21 +6,21 @@ import type { PropType } from 'vue' import type { FileSource } from '../types.ts' -import { showError } from '@nextcloud/dialogs' +import { extname } from 'path' import { FileType, Permission, Folder, File as NcFile, NodeStatus, Node, getFileActions } from '@nextcloud/files' -import { translate as t } from '@nextcloud/l10n' import { generateUrl } from '@nextcloud/router' import { isPublicShare } from '@nextcloud/sharing/public' +import { showError } from '@nextcloud/dialogs' +import { t } from '@nextcloud/l10n' import { vOnClickOutside } from '@vueuse/components' -import { extname } from 'path' import Vue, { computed, defineComponent } from 'vue' import { action as sidebarAction } from '../actions/sidebarAction.ts' +import { dataTransferToFileTree, onDropExternalFiles, onDropInternalFiles } from '../services/DropService.ts' import { getDragAndDropPreview } from '../utils/dragUtils.ts' import { hashCode } from '../utils/hashUtils.ts' -import { dataTransferToFileTree, onDropExternalFiles, onDropInternalFiles } from '../services/DropService.ts' -import logger from '../logger.ts' import { isDownloadable } from '../utils/permissions.ts' +import logger from '../logger.ts' Vue.directive('onClickOutside', vOnClickOutside) @@ -200,7 +200,20 @@ export default defineComponent({ } return actions - .filter(action => !action.enabled || action.enabled([this.source], this.currentView)) + .filter(action => { + if (!action.enabled) { + return true + } + + // In case something goes wrong, since we don't want to break + // the entire list, we filter out actions that throw an error. + try { + return action.enabled([this.source], this.currentView) + } catch (error) { + logger.error('Error while checking action', { action, error }) + return false + } + }) .sort((a, b) => (a.order || 0) - (b.order || 0)) },