Fix sidebar opening for different dav root and non-dav files

Signed-off-by: John Molakvoæ <[email protected]>

Author: skjnldsv

cypress/e2e/non-dav-files.cy.js

@@ -73,4 +73,11 @@ describe('Open non-dav files in viewer', function() {
 		cy.get('body > .viewer button.prev').should('not.be.visible')
 		cy.get('body > .viewer button.next').should('not.be.visible')
 	})
+
+	it('Does not see the sidebar button', function() {
+		// open the menu
+		cy.get('body > .viewer .modal-header button.action-item__menutoggle').click()
+		cy.get('.action-button__icon.icon-menu-sidebar').should('not.be.visible')
+	})
+
 })

src/components/Images.vue

@@ -110,7 +110,7 @@ export default {
 				return this.src
 			}
 
-			// If there is no preview and we have a drirect source
+			// If there is no preview and we have a direct source
 			// load it instead
 			if (this.source && !this.hasPreview) {
 				return this.source

src/mixins/Mime.js

@@ -50,12 +50,12 @@ export default {
 		// file path relative to user folder
 		hasPreview: {
 			type: Boolean,
-			required: true,
+			default: false,
 		},
 		// unique file id
 		fileid: {
 			type: [Number, String],
-			required: true,
+			required: false,
 		},
 		// list of all the visible files
 		fileList: {

src/mixins/PreviewUrl.js

@@ -21,7 +21,7 @@
  *
  */
 import { generateUrl } from '@nextcloud/router'
-import { getToken, isPublic } from '../utils/davUtils.js'
+import { getToken, getUserRoot, isPublic } from '../utils/davUtils.js'
 import { encodeFilePath, getDavPath } from '../utils/fileUtils.js'
 
 export default {

src/utils/davUtils.js

@@ -23,20 +23,23 @@
 import { generateRemoteUrl } from '@nextcloud/router'
 import { getCurrentUser } from '@nextcloud/auth'
 
-const getRootPath = function() {
+export const getRootPath = function() {
 	if (getCurrentUser()) {
-		return generateRemoteUrl(`dav/files/${getCurrentUser().uid}`)
+		return generateRemoteUrl(`dav${getUserRoot()}`)
 	} else {
 		return generateRemoteUrl('webdav').replace('/remote.php', '/public.php')
 	}
 }
 
-const isPublic = function() {
+export const getUserRoot = function() {
+	return '/files/' + getCurrentUser().uid
+}
+
+export const isPublic = function() {
 	return !getCurrentUser()
 }
 
-const getToken = function() {
+export const getToken = function() {
 	return document.getElementById('sharingToken') && document.getElementById('sharingToken').value
 }
 
-export { getRootPath, getToken, isPublic }

src/utils/fileUtils.js

@@ -21,9 +21,9 @@
  */
 import { dirname } from '@nextcloud/paths'
 import { generateUrl } from '@nextcloud/router'
-
 import camelcase from 'camelcase'
-import { getRootPath, getToken, isPublic } from './davUtils.js'
+
+import { getRootPath, getToken, getUserRoot, isPublic } from './davUtils.js'
 import { isNumber } from './numberUtil.js'
 
 /**
@@ -129,9 +129,22 @@ const genFileInfo = function(obj) {
  * @param {object} fileInfo The fileInfo
  * @param {string} fileInfo.filename the file full path
  * @param {string} fileInfo.basename the file name
+ * @param {string} fileInfo.source the file source if any
  * @return {string}
  */
-const getDavPath = function({ filename, basename }) {
+const getDavPath = function({ filename, basename, source = '' }) {
+	const prefixUser = getUserRoot()
+
+	// If we have a source but we're not a dav resource, return null
+	if (source && !source.includes(prefixUser)) {
+		return null
+	}
+
+	// Workaround for files with different root like /remote.php/dav
+	if (filename.startsWith(prefixUser)) {
+		filename = filename.slice(prefixUser.length)
+	}
+
 	// TODO: allow proper dav access without the need of basic auth
 	// https://github.com/nextcloud/server/issues/19700
 	if (isPublic()) {

src/views/Viewer.vue

@@ -78,7 +78,7 @@
 				{{ t('viewer', 'Edit') }}
 			</NcActionButton>
 			<!-- Menu items -->
-			<NcActionButton v-if="Sidebar && !isSidebarShown"
+			<NcActionButton v-if="Sidebar && sidebarOpenFilePath && !isSidebarShown"
 				:close-after-click="true"
 				icon="icon-menu-sidebar"
 				@click="showSidebar">
@@ -167,7 +167,7 @@ import isFullscreen from '@nextcloud/vue/dist/Mixins/isFullscreen.js'
 import isMobile from '@nextcloud/vue/dist/Mixins/isMobile'
 
 import { extractFilePaths, sortCompare } from '../utils/fileUtils.js'
-import { getRootPath } from '../utils/davUtils.js'
+import { getRootPath, getUserRoot } from '../utils/davUtils.js'
 import canDownload from '../utils/canDownload.js'
 import cancelableRequest from '../utils/CancelableRequest.js'
 import Error from '../components/Error.vue'
@@ -290,6 +290,9 @@ export default {
 		sidebarFile() {
 			return this.Sidebar && this.Sidebar.file
 		},
+		sidebarOpenFilePath() {
+			return this.currentFile?.davPath?.split(getUserRoot())[1]
+		},
 
 		/**
 		 * Is the current user allowed to delete the file?
@@ -906,7 +909,7 @@ export default {
 			// TODO: also hide figure, needs a proper method for it in server Sidebar
 
 			if (OCA?.Files?.Sidebar) {
-				await OCA.Files.Sidebar.open(this.currentFile.filename)
+				await OCA.Files.Sidebar.open(this.sidebarOpenFilePath)
 			}
 		},