Merge pull request #56060 from nextcloud/fix/hide-open-locally-action

apps/files/src/actions/openLocallyAction.ts

@@ -10,12 +10,13 @@ import IconWeb from '@mdi/svg/svg/web.svg?raw'
 import { getCurrentUser } from '@nextcloud/auth'
 import axios from '@nextcloud/axios'
 import { DialogBuilder, showError } from '@nextcloud/dialogs'
-import { FileAction, Permission } from '@nextcloud/files'
+import { FileAction } from '@nextcloud/files'
 import { translate as t } from '@nextcloud/l10n'
 import { encodePath } from '@nextcloud/paths'
 import { generateOcsUrl } from '@nextcloud/router'
 import { isPublicShare } from '@nextcloud/sharing/public'
 import logger from '../logger.ts'
+import { isSyncable } from '../utils/permissions.ts'
 
 export const action = new FileAction({
 	id: 'edit-locally',
@@ -34,7 +35,7 @@ export const action = new FileAction({
 			return false
 		}
 
-		return (nodes[0].permissions & Permission.UPDATE) !== 0
+		return isSyncable(nodes[0])
 	},
 
 	async exec(node: Node) {

apps/files/src/utils/permissions.ts

@@ -36,3 +36,22 @@ export function isDownloadable(node: Node): boolean {
 
 	return true
 }
+
+/**
+ * Check permissions on the node if it can be synced/open locally
+ *
+ * @param node The node to check
+ * @return True if syncable, false otherwise
+ */
+export function isSyncable(node: Node): boolean {
+	if (!node.isDavResource) {
+		return false
+	}
+
+	if ((node.permissions & Permission.UPDATE) === 0) {
+		return false
+	}
+
+	// Syncable has the same permissions as downloadable for now
+	return isDownloadable(node)
+}