Issue 69: Drop the media id variable, just call $media->id().

CollaboraOnline · Dec 11, 2024 · f1b4d15 · f1b4d15
1 parent 1a9be86
commit f1b4d15
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/Controller/ViewerController.php b/src/Controller/ViewerController.php
@@ -144,12 +144,10 @@ protected function getViewerRender(MediaInterface $media, string $wopi_client, b
     $wopi_base = $cool_settings['wopi_base'];
     $allowfullscreen = $cool_settings['allowfullscreen'] ?? FALSE;
 
-    $id = $media->id();
-
     $expire_timestamp = $this->getExpireTimestamp();
     $access_token = $this->jwtTranscoder->encode(
       [
-        'fid' => $id,
+        'fid' => $media->id(),
         'uid' => $this->currentUser->id(),
         'wri' => $can_write,
       ],
@@ -158,7 +156,7 @@ protected function getViewerRender(MediaInterface $media, string $wopi_client, b
 
     $render_array = [
       '#wopiClient' => $wopi_client,
-      '#wopiSrc' => urlencode($wopi_base . '/cool/wopi/files/' . $id),
+      '#wopiSrc' => urlencode($wopi_base . '/cool/wopi/files/' . $media->id()),
       '#accessToken' => $access_token,
       // Convert to milliseconds.
       '#accessTokenTtl' => $expire_timestamp * 1000,