From 86c2a5c3e80ad6070ce868179fdd742ec2649747 Mon Sep 17 00:00:00 2001 From: Pol Dellaiera Date: Wed, 29 Sep 2021 15:07:27 +0200 Subject: [PATCH] fix: Update `ChillDocumentManager`. Fix statuscode and return types. --- .../src/Service/Wopi/ChillDocumentManager.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/Bundle/ChillWopiBundle/src/Service/Wopi/ChillDocumentManager.php b/src/Bundle/ChillWopiBundle/src/Service/Wopi/ChillDocumentManager.php index bfa3ee8b0..1924edd01 100644 --- a/src/Bundle/ChillWopiBundle/src/Service/Wopi/ChillDocumentManager.php +++ b/src/Bundle/ChillWopiBundle/src/Service/Wopi/ChillDocumentManager.php @@ -21,6 +21,7 @@ use Error; use loophp\psr17\Psr17Interface; use Psr\Http\Message\RequestInterface; use Psr\Http\Message\StreamInterface; +use Ramsey\Uuid\Uuid; use Symfony\Bridge\PsrHttpMessage\HttpMessageFactoryInterface; use Symfony\Component\HttpFoundation\RequestStack; use Symfony\Component\Mime\MimeTypes; @@ -103,7 +104,7 @@ final class ChillDocumentManager implements DocumentManagerInterface public function findByDocumentId(string $documentId): ?Document { return $this->storedObjectRepository->findOneBy( [ - 'uuid' => $documentId, + 'uuid' => Uuid::fromString($documentId), ] ); } @@ -198,7 +199,7 @@ final class ChillDocumentManager implements DocumentManagerInterface ->createStream($this->getContent($document)); } - private function deleteContent(StoredObject $storedObject): string + private function deleteContent(StoredObject $storedObject): void { /** @var StdClass $object */ $object = $this->tempUrlGenerator->generate('DELETE', $storedObject->getFilename()); @@ -234,7 +235,7 @@ final class ChillDocumentManager implements DocumentManagerInterface $response = $this->httpClient->request('PUT', $object->url, ['body' => $content]); - if (200 !== $response->getStatusCode()) + if (201 !== $response->getStatusCode()) { throw new Error('Unable to save stored object.'); }