From a2c373b7eaaf59b86f20324620cec8145c22cd79 Mon Sep 17 00:00:00 2001
From: Ferdinand Thiessen <opensource@fthiessen.de>
Date: Wed, 3 Sep 2025 17:14:29 +0200
Subject: [PATCH] fix: the account files filter should also work in trashbin

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
---
 .../FileListFilter/FileListFilter.vue         |  2 +-
 .../src/files_filters/AccountFilter.ts        | 29 ++++++++++++++++---
 2 files changed, 26 insertions(+), 5 deletions(-)

diff --git a/apps/files/src/components/FileListFilter/FileListFilter.vue b/apps/files/src/components/FileListFilter/FileListFilter.vue
index bd3ac867ed5..d7f50391a73 100644
--- a/apps/files/src/components/FileListFilter/FileListFilter.vue
+++ b/apps/files/src/components/FileListFilter/FileListFilter.vue
@@ -40,7 +40,7 @@ defineEmits<{
 
 <style scoped>
 .files-list-filter__clear-button :deep(.action-button__text) {
-	color: var(--color-error-text);
+	color: var(--color-text-error, var(--color-error-text));
 }
 
 :deep(.button-vue) {
diff --git a/apps/files_sharing/src/files_filters/AccountFilter.ts b/apps/files_sharing/src/files_filters/AccountFilter.ts
index 4f185d9fd9c..a94b32c6ace 100644
--- a/apps/files_sharing/src/files_filters/AccountFilter.ts
+++ b/apps/files_sharing/src/files_filters/AccountFilter.ts
@@ -11,6 +11,7 @@ import Vue from 'vue'
 
 import FileListFilterAccount from '../components/FileListFilterAccount.vue'
 import { isPublicShare } from '@nextcloud/sharing/public'
+import { TRASHBIN_VIEW_ID } from '../../../files_trashbin/src/files_views/trashbinView'
 
 export interface IAccountData {
 	uid: string
@@ -62,19 +63,30 @@ class AccountFilter extends FileListFilter {
 		const userIds = this.filterAccounts.map(({ uid }) => uid)
 		// Filter if the owner of the node is in the list of filtered accounts
 		return nodes.filter((node) => {
-			const sharees = node.attributes.sharees?.sharee as { id: string }[] | undefined
-			// If the node provides no information lets keep it
-			if (!node.owner && !sharees) {
-				return true
+			if (window.OCP.Files.Router.params.view === TRASHBIN_VIEW_ID) {
+				const deletedBy = node.attributes?.['trashbin-deleted-by-id']
+				if (deletedBy && userIds.includes(deletedBy)) {
+					return true
+				}
+				return false
 			}
+
 			// if the owner matches
 			if (node.owner && userIds.includes(node.owner)) {
 				return true
 			}
+
 			// Or any of the sharees (if only one share this will be an object, otherwise an array. So using `.flat()` to make it always an array)
+			const sharees = node.attributes.sharees?.sharee as { id: string }[] | undefined
 			if (sharees && [sharees].flat().some(({ id }) => userIds.includes(id))) {
 				return true
 			}
+
+			// If the node provides no information lets keep it
+			if (!node.owner && !sharees) {
+				return true
+			}
+
 			// Not a valid node for the current filter
 			return false
 		})
@@ -139,6 +151,15 @@ class AccountFilter extends FileListFilter {
 					})
 				}
 			}
+
+			// lets also handle trashbin
+			const deletedBy = node.attributes?.['trashbin-deleted-by-id']
+			if (deletedBy) {
+				available.set(deletedBy, {
+					uid: deletedBy,
+					displayName: node.attributes?.['trashbin-deleted-by-display-name'] || deletedBy,
+				})
+			}
 		}
 
 		this.availableAccounts = [...available.values()]