fix(files_sharing): Drop permissions on unmounted pending and deleted shares

Pending and deleted shares are not mounted into the user's filesystem, so
generic file operations like delete or download produced a misleading
"file is not available" error.

These shares now carry no permissions, so every permission-aware action
hides itself automatically, without the files app having to special-case
each view. Conversion additionally requires read permission, matching the
server-side readability check.

Signed-off-by: nfebe <fenn25.fn@gmail.com>

Author: nfebe

apps/files/src/actions/convertAction.ts

@@ -41,6 +41,11 @@ export const registerConvertActions = () => {
 					}
 				}
 
+				// Conversion reads the source file, so it requires read permission
+				if (nodes.some((node) => (node.permissions & Permission.READ) === 0)) {
+					return false
+				}
+
 				// Check that all nodes have the same mime type
 				return nodes.every(node => from === node.mime)
 			},

apps/files_sharing/src/services/SharingService.spec.ts

@@ -533,6 +533,34 @@ describe('SharingService share to Node mapping', () => {
 		expect(file.attributes.favorite).toBe(0)
 	})
 
+	test('Pending share has no permissions', async () => {
+		axios.get
+			.mockReturnValueOnce(Promise.resolve({
+				data: { ocs: { data: [shareFile] } },
+			}))
+			.mockReturnValueOnce(Promise.resolve({
+				data: { ocs: { data: [] } },
+			}))
+
+		const shares = await getContents(false, false, true, false)
+
+		expect(axios.get).toHaveBeenCalledTimes(2)
+		expect(shares.contents).toHaveLength(1)
+		expect(shares.contents[0].permissions).toBe(0)
+	})
+
+	test('Deleted share has no permissions', async () => {
+		axios.get.mockReturnValueOnce(Promise.resolve({
+			data: { ocs: { data: [shareFolder] } },
+		}))
+
+		const shares = await getContents(false, false, false, true)
+
+		expect(axios.get).toHaveBeenCalledTimes(1)
+		expect(shares.contents).toHaveLength(1)
+		expect(shares.contents[0].permissions).toBe(0)
+	})
+
 	test('Empty', async () => {
 		vi.spyOn(logger, 'error').mockImplementationOnce(() => {})
 		axios.get.mockReturnValueOnce(Promise.resolve({

apps/files_sharing/src/services/SharingService.ts

@@ -21,7 +21,7 @@ const headers = {
 	'Content-Type': 'application/json',
 }
 
-const ocsEntryToNode = async function(ocsEntry: any): Promise<Folder | File | null> {
+const ocsEntryToNode = async function(ocsEntry: any, unmounted = false): Promise<Folder | File | null> {
 	try {
 		// Federated share handling
 		if (ocsEntry?.remote_id !== undefined) {
@@ -53,6 +53,13 @@ const ocsEntryToNode = async function(ocsEntry: any): Promise<Folder | File | nu
 			ocsEntry.displayname_owner = ocsEntry.owner
 		}
 
+		// Pending and deleted shares are not mounted into the user's filesystem,
+		// so no file operation can act on them until they are accepted or restored.
+		if (unmounted) {
+			ocsEntry.item_permissions = Permission.NONE
+			ocsEntry.permissions = Permission.NONE
+		}
+
 		const isFolder = ocsEntry?.item_type === 'folder'
 		const hasPreview = ocsEntry?.has_preview === true
 		const Node = isFolder ? Folder : File
@@ -202,24 +209,25 @@ const groupBy = function(nodes: (Folder | File)[], key: string) {
 }
 
 export const getContents = async (sharedWithYou = true, sharedWithOthers = true, pendingShares = false, deletedshares = false, filterTypes: number[] = []): Promise<ContentsWithRoot> => {
-	const promises = [] as AxiosPromise<OCSResponse<any>>[]
+	const requests = [] as { promise: AxiosPromise<OCSResponse<any>>, unmounted: boolean }[]
 
 	if (sharedWithYou) {
-		promises.push(getSharedWithYou(), getRemoteShares())
+		requests.push({ promise: getSharedWithYou(), unmounted: false }, { promise: getRemoteShares(), unmounted: false })
 	}
 	if (sharedWithOthers) {
-		promises.push(getSharedWithOthers())
+		requests.push({ promise: getSharedWithOthers(), unmounted: false })
 	}
 	if (pendingShares) {
-		promises.push(getPendingShares(), getRemotePendingShares())
+		requests.push({ promise: getPendingShares(), unmounted: true }, { promise: getRemotePendingShares(), unmounted: true })
 	}
 	if (deletedshares) {
-		promises.push(getDeletedShares())
+		requests.push({ promise: getDeletedShares(), unmounted: true })
 	}
 
-	const responses = await Promise.all(promises)
-	const data = responses.map((response) => response.data.ocs.data).flat()
-	let contents = (await Promise.all(data.map(ocsEntryToNode)))
+	const responses = await Promise.all(requests.map(({ promise }) => promise))
+	const data = responses.flatMap((response, index) => response.data.ocs.data
+		.map((entry) => ({ entry, unmounted: requests[index].unmounted })))
+	let contents = (await Promise.all(data.map(({ entry, unmounted }) => ocsEntryToNode(entry, unmounted))))
 		.filter((node) => node !== null) as (Folder | File)[]
 
 	if (filterTypes.length > 0) {