From 5191596b760979fbb4f9f02d1bb28d6450bfb0aa Mon Sep 17 00:00:00 2001
From: provokateurin <kate@provokateurin.de>
Date: Mon, 14 Oct 2024 17:51:14 +0200
Subject: [PATCH] fix(Trash): Fix wrong original location

Signed-off-by: provokateurin <kate@provokateurin.de>
---
 tests/Trash/TrashBackendTest.php | 65 ++++++++++++++++++++++++++++++++
 1 file changed, 65 insertions(+)

diff --git a/tests/Trash/TrashBackendTest.php b/tests/Trash/TrashBackendTest.php
index 119065783..e6bb0213b 100644
--- a/tests/Trash/TrashBackendTest.php
+++ b/tests/Trash/TrashBackendTest.php
@@ -19,9 +19,12 @@
 use OCA\GroupFolders\Folder\FolderManager;
 use OCA\GroupFolders\Mount\GroupFolderStorage;
 use OCA\GroupFolders\Trash\TrashBackend;
+use OCP\Constants;
 use OCP\Files\Folder;
 use OCP\Files\IRootFolder;
 use OCP\IUser;
+use OCP\Server;
+use OCP\Share;
 use Test\TestCase;
 use Test\Traits\UserTrait;
 
@@ -211,4 +214,66 @@ public function testHideDeletedTrashItemInDeletedParentFolderAcl(): void {
 
 		$this->logout();
 	}
+
+	public function testWrongOriginalLocation(): void {
+		$shareManager = Server::get(Share\IManager::class);
+
+		$userA = $this->createUser('A', 'test');
+		$userAFolder = Server::get(IRootFolder::class)->getUserFolder('A');
+
+		$userB = $this->createUser('B', 'test');
+		$userBFolder = Server::get(IRootFolder::class)->getUserFolder('B');
+
+		$groupBackend = Server::get(Database::class);
+		$groupBackend->createGroup('A');
+		$groupBackend->addToGroup('A', 'A');
+		$groupBackend->addToGroup('B', 'A');
+		$this->assertCount(2, $groupBackend->usersInGroup('A'));
+
+		$groupFolderId = $this->folderManager->createFolder('A');
+		$this->folderManager->setFolderACL($groupFolderId, true);
+		$this->folderManager->addApplicableGroup($groupFolderId, 'A');
+		$this->folderManager->setGroupPermissions($groupFolderId, 'A', Constants::PERMISSION_ALL);
+		$this->assertInstanceOf(Folder::class, $userAFolder->get('A'));
+
+		$this->loginAsUser('A');
+		$userAFolder->newFolder('A/B/C');
+		$userAFolder->newFile('A/B/C/D', 'foo');
+
+		$this->ruleManager->saveRule(new Rule(new UserMapping('group', 'A'), $userAFolder->get('A/B')->getId(), Constants::PERMISSION_READ, 0));
+		$this->ruleManager->saveRule(new Rule(new UserMapping('user', 'A'), $userAFolder->get('A/B')->getId(), Constants::PERMISSION_ALL, Constants::PERMISSION_READ | Constants::PERMISSION_UPDATE | Constants::PERMISSION_CREATE));
+		$this->ruleManager->saveRule(new Rule(new UserMapping('user', 'A'), $userAFolder->get('A/B/C')->getId(), Constants::PERMISSION_ALL, Constants::PERMISSION_ALL));
+
+		$folderShare = $shareManager->newShare();
+		$folderShare->setShareType(Share\IShare::TYPE_USER);
+		$folderShare->setSharedWith('B');
+		$folderShare->setSharedBy('A');
+		$folderShare->setPermissions(Constants::PERMISSION_ALL);
+		$folderShare->setNode($userAFolder->get('A/B/C'));
+		$folderShare = $shareManager->createShare($folderShare);
+		$this->assertNotEmpty($folderShare->getId());
+
+		$fileShare = $shareManager->newShare();
+		$fileShare->setShareType(Share\IShare::TYPE_USER);
+		$fileShare->setSharedWith('B');
+		$fileShare->setSharedBy('A');
+		$fileShare->setPermissions(19);
+		$fileShare->setNode($userAFolder->get('A/B/C/D'));
+		$fileShare = $shareManager->createShare($fileShare);
+		$this->assertNotEmpty($fileShare->getId());
+
+		$this->loginAsUser('B');
+		$this->assertTrue($userBFolder->get('C/D')->isDeletable());
+		$userBFolder->get('C/D')->delete();
+
+		$trashedOfUserA = $this->trashBackend->listTrashRoot($userA);
+		$this->assertCount(1, $trashedOfUserA);
+		// TODO: Bug original location is wrong
+		$this->assertSame('A/B/C/D', $trashedOfUserA[0]->getOriginalLocation());
+
+		$trashedOfUserB = $this->trashBackend->listTrashRoot($userB);
+		$this->assertCount(1, $trashedOfUserB);
+		// TODO: Bug original location is wrong
+		$this->assertSame('C/D', $trashedOfUserB[0]->getOriginalLocation());
+	}
 }