chore: remove legacy node attributes deprecation

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>

Author: skjnldsv

__tests__/files/node.spec.ts

@@ -820,26 +820,9 @@ describe('Attributes update', () => {
 		})
 
 		expect(file.attributes?.etag).toBe('5678')
-		expect(file.attributes?.size).toBe(9999)
-		expect(file.attributes?.owner).toBe('emma')
-		expect(file.attributes?.fileid).toBeUndefined()
-	})
-
-	test('Deprecated access to toplevel attributes', () => {
-		const spy = vi.spyOn(window.console, 'warn')
-		const file = new File({
-			source: 'https://cloud.domain.com/remote.php/dav/files/emma/Photos/picture.jpg',
-			mime: 'image/jpeg',
-			owner: 'emma',
-			size: 9999,
-			attributes: {
-				etag: '1234',
-				size: 9999,
-			},
-		})
-
-		expect(file.attributes.size).toBe(9999)
-		expect(spy).toBeCalledTimes(1)
+		expect(file?.size).toBe(9999)
+		expect(file?.owner).toBe('emma')
+		expect(file?.fileid).toBeUndefined()
 	})
 
 	test('Changing a protected attributes is not possible', () => {
@@ -853,9 +836,10 @@ describe('Attributes update', () => {
 		})
 
 		// We can not update the owner
-		expect(() => { file.attributes.owner = 'admin' }).toThrowError()
+		// @ts-expect-error owner is a read-only property
+		expect(() => { file.owner = 'admin' }).toThrowError()
 		// The owner is still the original one
-		expect(file.attributes?.owner).toBe('emma')
+		expect(file?.owner).toBe('emma')
 	})
 
 })

lib/node/node.ts

@@ -51,12 +51,7 @@ export abstract class Node {
 			// Apply original changes
 			return Reflect.deleteProperty(target, prop)
 		},
-		// TODO: This is deprecated and only needed for files v3
 		get: (target: Attribute, prop: string, receiver) => {
-			if (this.readonlyAttributes.includes(prop)) {
-				logger.warn(`Accessing "Node.attributes.${prop}" is deprecated, access it directly on the Node instance.`)
-				return Reflect.get(this, prop)
-			}
 			return Reflect.get(target, prop, receiver)
 		},
 	} as ProxyHandler<Attribute>
@@ -74,8 +69,6 @@ export abstract class Node {
 		validateData(data, davService)
 
 		this._data = {
-			// TODO: Remove with next major release, this is just for compatibility
-			displayname: data.attributes?.displayname,
 			...data,
 			attributes: {},
 		}