fix(trash): Fix deleting items from public share with write access

When no user in session, use an empty string as deleted_by, which shows
 as "Unknown" in the UI.

Signed-off-by: Côme Chilliet <[email protected]>

Author: come-nc

lib/Trash/TrashBackend.php

@@ -237,9 +237,7 @@ public function moveToTrash(IStorage $storage, string $internalPath): bool {
 			$fileEntry = $storage->getCache()->get($internalPath);
 			$folderId = $storage->getFolderId();
 			$user = $this->userSession->getUser();
-			if (!$user) {
-				throw new \Exception('file moved to trash with no user in context');
-			}
+
 			// ensure the folder exists
 			$this->getTrashFolder($folderId);
 
@@ -261,7 +259,7 @@ public function moveToTrash(IStorage $storage, string $internalPath): bool {
 					$originalLocation = $storage->getWrapperStorage()->getUnjailedPath($originalLocation);
 				}
 
-				$this->trashManager->addTrashItem($folderId, $name, $time, $originalLocation, $fileEntry->getId(), $user->getUID());
+				$this->trashManager->addTrashItem($folderId, $name, $time, $originalLocation, $fileEntry->getId(), $user?->getUID() ?? '');
 
 				// some storage backends (object/encryption) can either already move the cache item or cause the target to be scanned
 				// so we only conditionally do the cache move here