fix: Bring back actually setting the guest name

Signed-off-by: Julius Härtl <[email protected]>
nextcloud · May 8, 2024 · a3f56d3 · a3f56d3
1 parent 02f5e2f
commit a3f56d3
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/lib/Controller/DocumentController.php b/lib/Controller/DocumentController.php
@@ -383,6 +383,8 @@ public function token(int $fileId, ?string $shareToken = null, ?string $path = n
 
 			$wopi = $this->getToken($file, $share);
 
+			$this->tokenManager->setGuestName($wopi, $guestName);
+
 			return new DataResponse(array_merge(
 				[ 'urlSrc' => $this->tokenManager->getUrlSrc($file) ],
 				$wopi->jsonSerialize(),

diff --git a/lib/TokenManager.php b/lib/TokenManager.php
@@ -313,6 +313,15 @@ public function updateGuestName(string $accessToken, string $guestName) {
 		$this->wopiMapper->update($wopi);
 	}
 
+	public function setGuestName(Wopi $wopi, ?string $guestName = null): Wopi {
+		if ($wopi->getTokenType() !== Wopi::TOKEN_TYPE_GUEST && $wopi->getTokenType() !== Wopi::TOKEN_TYPE_REMOTE_GUEST) {
+			return $wopi;
+		}
+
+		$wopi->setGuestDisplayname($this->prepareGuestName($guestName));
+		return $this->wopiMapper->update($wopi);
+	}
+
 	public function getUrlSrc(File $file): string {
 		return $this->wopiParser->getUrlSrc($file->getMimeType())['urlsrc'];
 	}