[stable31] fix: the account files filter should also work in trashbin

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) {

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()]

dist/files_sharing-init.js.license

@@ -21,6 +21,7 @@ SPDX-FileCopyrightText: Matt Zabriskie
 SPDX-FileCopyrightText: Joyent
 SPDX-FileCopyrightText: Jonas Schade <derzade@gmail.com>
 SPDX-FileCopyrightText: Jacob Clevenger<https://github.com/wheatjs>
+SPDX-FileCopyrightText: Iskren Ivov Chernev <iskren.chernev@gmail.com> (https://github.com/ichernev)
 SPDX-FileCopyrightText: Guillaume Chau <guillaume.b.chau@gmail.com>
 SPDX-FileCopyrightText: GitHub Inc.
 SPDX-FileCopyrightText: Feross Aboukhadijeh
@@ -77,6 +78,9 @@ This file is generated from multiple sources. Included packages:
 - @nextcloud/logger
 	- version: 3.0.2
 	- license: GPL-3.0-or-later
+- @nextcloud/moment
+	- version: 1.3.5
+	- license: GPL-3.0-or-later
 - @nextcloud/paths
 	- version: 2.2.1
 	- license: GPL-3.0-or-later
@@ -140,6 +144,9 @@ This file is generated from multiple sources. Included packages:
 - md5
 	- version: 2.3.0
 	- license: BSD-3-Clause
+- moment
+	- version: 2.30.1
+	- license: MIT
 - buffer
 	- version: 6.0.3
 	- license: MIT
@@ -185,6 +192,9 @@ This file is generated from multiple sources. Included packages:
 - vue-loader
 	- version: 15.11.1
 	- license: MIT
+- vue-router
+	- version: 3.6.5
+	- license: MIT
 - vue
 	- version: 2.7.16
 	- license: MIT