diff options
author | Grigorii K. Shartsev <me@shgk.me> | 2024-10-09 12:37:33 +0200 |
---|---|---|
committer | backportbot[bot] <backportbot[bot]@users.noreply.github.com> | 2024-10-25 12:28:16 +0000 |
commit | 9563f19651bd648305ecc713fdfd90013f8f9066 (patch) | |
tree | 98f67baac481f1b2a1586f5f99d5b11dbd51e9b8 /apps | |
parent | 2f7eeab191a4b77bb4497a1075fec232d3d688d4 (diff) | |
download | nextcloud-server-9563f19651bd648305ecc713fdfd90013f8f9066.tar.gz nextcloud-server-9563f19651bd648305ecc713fdfd90013f8f9066.zip |
fix(files): handle empty view with error
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Diffstat (limited to 'apps')
-rw-r--r-- | apps/files/src/utils/davUtils.ts | 36 | ||||
-rw-r--r-- | apps/files/src/views/FilesList.vue | 33 |
2 files changed, 63 insertions, 6 deletions
diff --git a/apps/files/src/utils/davUtils.ts b/apps/files/src/utils/davUtils.ts index 38137a04d16..cf49df5b278 100644 --- a/apps/files/src/utils/davUtils.ts +++ b/apps/files/src/utils/davUtils.ts @@ -4,6 +4,8 @@ */ import { getCurrentUser } from '@nextcloud/auth' +import { t } from '@nextcloud/l10n' +import type { WebDAVClientError } from 'webdav' /** * Check whether this is a public share @@ -21,3 +23,37 @@ export function getToken() { const tokenElement = document.getElementById('sharingToken') as (HTMLInputElement | null) return tokenElement?.value } + +/** + * Whether error is a WebDAVClientError + * @param error - Any exception + * @return {boolean} - Whether error is a WebDAVClientError + */ +function isWebDAVClientError(error: unknown): error is WebDAVClientError { + return error instanceof Error && 'status' in error && 'response' in error +} + +/** + * Get a localized error message from webdav request + * @param error - An exception from webdav request + * @return {string} Localized error message for end user + */ +export function humanizeWebDAVError(error: unknown) { + if (error instanceof Error) { + if (isWebDAVClientError(error)) { + const status = error.status || error.response?.status || 0 + if ([400, 404, 405].includes(status)) { + return t('files', 'Folder not found') + } else if (status === 403) { + return t('files', 'This operation is forbidden') + } else if (status === 500) { + return t('files', 'This directory is unavailable, please check the logs or contact the administrator') + } else if (status === 503) { + return t('files', 'Storage is temporarily not available') + } + } + return t('files', 'Unexpected error: {error}', { error: error.message }) + } + + return t('files', 'Unknown error') +} diff --git a/apps/files/src/views/FilesList.vue b/apps/files/src/views/FilesList.vue index 7700cf92382..1fc6c190733 100644 --- a/apps/files/src/views/FilesList.vue +++ b/apps/files/src/views/FilesList.vue @@ -75,16 +75,32 @@ <!-- Empty content placeholder --> <template v-else-if="!loading && isEmptyDir"> - <div v-if="currentView?.emptyView" class="files-list__empty-view-wrapper"> + <!-- Empty due to error --> + <NcEmptyContent v-if="error" :name="error" data-cy-files-content-error> + <template #action> + <NcButton type="secondary" @click="fetchContent"> + <template #icon> + <IconReload :size="20" /> + </template> + {{ t('files', 'Retry') }} + </NcButton> + </template> + <template #icon> + <IconAlertCircleOutline /> + </template> + </NcEmptyContent> + <!-- Custom empty view --> + <div v-else-if="currentView?.emptyView" class="files-list__empty-view-wrapper"> <div ref="customEmptyView" /> </div> + <!-- Default empty directory view --> <NcEmptyContent v-else :name="currentView?.emptyTitle || t('files', 'No files in here')" :description="currentView?.emptyCaption || t('files', 'Upload some content or sync with your devices!')" data-cy-files-content-empty> <template v-if="directory !== '/'" #action> <!-- Uploader --> - <UploadPicker v-if="currentFolder && canUpload && !isQuotaExceeded" + <UploadPicker v-if="canUpload && !isQuotaExceeded" allow-folders class="files-list__header-upload-button" :content="getContent" @@ -93,10 +109,7 @@ multiple @failed="onUploadFail" @uploaded="onUpload" /> - <NcButton v-else - :aria-label="t('files', 'Go to the previous folder')" - :to="toPreviousDir" - type="primary"> + <NcButton v-else :to="toPreviousDir" type="primary"> {{ t('files', 'Go back') }} </NcButton> </template> @@ -134,6 +147,8 @@ import { UploadPicker, UploadStatus } from '@nextcloud/upload' import { loadState } from '@nextcloud/initial-state' import { defineComponent } from 'vue' +import IconAlertCircleOutline from 'vue-material-design-icons/AlertCircleOutline.vue' +import IconReload from 'vue-material-design-icons/Reload.vue' import LinkIcon from 'vue-material-design-icons/Link.vue' import ListViewIcon from 'vue-material-design-icons/FormatListBulletedSquare.vue' import NcAppContent from '@nextcloud/vue/dist/Components/NcAppContent.js' @@ -161,6 +176,7 @@ import filesListWidthMixin from '../mixins/filesListWidth.ts' import filesSortingMixin from '../mixins/filesSorting.ts' import logger from '../logger.ts' import DragAndDropNotice from '../components/DragAndDropNotice.vue' +import { humanizeWebDAVError } from '../utils/davUtils.ts' const isSharingEnabled = (getCapabilities() as { files_sharing?: boolean })?.files_sharing !== undefined @@ -182,6 +198,8 @@ export default defineComponent({ AccountPlusIcon, UploadPicker, ViewGridIcon, + IconAlertCircleOutline, + IconReload, }, mixins: [ @@ -227,6 +245,7 @@ export default defineComponent({ data() { return { loading: true, + error: null as string | null, promise: null as CancelablePromise<ContentsWithRoot> | Promise<ContentsWithRoot> | null, dirContentsFiltered: [] as INode[], @@ -482,6 +501,7 @@ export default defineComponent({ methods: { async fetchContent() { this.loading = true + this.error = null const dir = this.directory const currentView = this.currentView @@ -530,6 +550,7 @@ export default defineComponent({ }) } catch (error) { logger.error('Error while fetching content', { error }) + this.error = humanizeWebDAVError(error) } finally { this.loading = false } |