Merge pull request #56738 from nextcloud/backport/56060/stable32

[stable32] fix(files_sharing): Hide 'Open locally' action

apps/files/src/actions/openLocallyAction.ts

@@ -5,13 +5,14 @@
 import { encodePath } from '@nextcloud/paths'
 import { generateOcsUrl } from '@nextcloud/router'
 import { getCurrentUser } from '@nextcloud/auth'
-import { FileAction, Permission, type Node } from '@nextcloud/files'
+import { FileAction, type Node } from '@nextcloud/files'
 import { showError, DialogBuilder } from '@nextcloud/dialogs'
 import { translate as t } from '@nextcloud/l10n'
 import axios from '@nextcloud/axios'
 import LaptopSvg from '@mdi/svg/svg/laptop.svg?raw'
 import IconWeb from '@mdi/svg/svg/web.svg?raw'
 import { isPublicShare } from '@nextcloud/sharing/public'
+import { isSyncable } from '../utils/permissions.ts'
 
 export const action = new FileAction({
 	id: 'edit-locally',
@@ -30,7 +31,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

@@ -35,3 +35,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)
+}