[FIX] deliver current revision in WOPI

Author: chfsx

Services/WOPI/classes/Embed/EmbeddedApplication.php

@@ -40,7 +40,7 @@ class EmbeddedApplication
 
     public function __construct(
         protected ResourceIdentification $identification,
-        protected ?Action $action,
+        protected Action $action,
         protected ResourceStakeholder $stakeholder,
         protected URI $back_target,
         protected bool $inline = false,
@@ -55,9 +55,7 @@ public function __construct(
             'resource_id' => $this->identification->serialize(),
             'user_id' => $DIC->user()->getId(),
             'stakeholder' => $this->stakeholder::class,
-            'editable' => $edit ?? ($this->action === null
-                    ? false
-                    : $this->action->getName() === ActionTarget::EDIT->value)
+            'editable' => $edit ?? ($this->action->getName() === ActionTarget::EDIT->value)
         ];
         $this->token = $data_signer->sign($payload, 'wopi', new \DateTimeImmutable("now + $this->ttl seconds"));
         $this->ilias_base_url = $ilias_base_url ?? new URI(ILIAS_HTTP_PATH);

Services/WOPI/classes/Handler/RequestHandler.php

@@ -128,7 +128,11 @@ public function handleRequest(): void
                 $this->http->close();
             }
             $resource = $this->irss->manage()->getResource($resource_id);
-            $current_revision = $resource->getCurrentRevisionIncludingDraft();
+            if ($this->editable) {
+                $current_revision = $resource->getCurrentRevisionIncludingDraft();
+            } else {
+                $current_revision = $resource->getCurrentRevision();
+            }
 
             $method_override = $this->http->request()->getHeader(self::HEADER_X_WOPI_OVERRIDE)[0] ?? null;
             $is_file_convertion = (bool) ($this->http->request()->getHeader(