fix(files_sharing): Only clear permissions of pending federated shares

Signed-off-by: provokateurin <[email protected]>

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

@@ -392,7 +392,7 @@ describe('SharingService share to Node mapping', () => {
 		expect(file.mtime?.getTime()).toBe(remoteFile.mtime * 1000)
 		// not available for remote shares
 		expect(file.size).toBe(undefined)
-		expect(file.permissions).toBe(0)
+		expect(file.permissions).toBe(19)
 		expect(file.root).toBe('/files/test')
 		expect(file.attributes).toBeInstanceOf(Object)
 		expect(file.attributes.favorite).toBe(0)

apps/files_sharing/src/services/SharingService.ts

@@ -40,9 +40,12 @@ const ocsEntryToNode = async function(ocsEntry: any): Promise<Folder | File | nu
 				ocsEntry.file_target = ocsEntry.name
 			}
 
-			// Need to set permissions to NONE for federated shares
-			ocsEntry.item_permissions = Permission.NONE
-			ocsEntry.permissions = Permission.NONE
+			// If the share is not accepted yet we don't know which permissions it will have
+			if (!ocsEntry.accepted) {
+				// Need to set permissions to NONE for federated shares
+				ocsEntry.item_permissions = Permission.NONE
+				ocsEntry.permissions = Permission.NONE
+			}
 
 			ocsEntry.uid_owner = ocsEntry.owner
 			// TODO: have the real display name stored somewhere