From 88590df43b5a00f9bd3e2c30e7e1c8fc0292d455 Mon Sep 17 00:00:00 2001
From: Matthias Schuhmayer <38959016+mattamon@users.noreply.github.com>
Date: Tue, 14 May 2024 13:55:31 +0200
Subject: [PATCH] Add text controller and encoder (#61)

* Add Text preview controller and encoder

* Apply php-cs-fixer changes

* Add declare strict

* Remove preview

* Remove unused use statements

* Add max file size exception

* Add Encoder Tests

---------

Co-authored-by: mattamon <mattamon@users.noreply.github.com>
---
 config/assets.yaml                            |  6 +-
 src/Asset/Controller/Data/TextController.php  | 79 +++++++++++++++++++
 src/Asset/Encoder/TextEncoder.php             | 44 +++++++++++
 src/Asset/Encoder/TextEncoderInterface.php    | 29 +++++++
 .../MaxFileSizeExceededException.php          | 31 ++++++++
 .../Attributes/Response/Content/DataJson.php  | 42 ++++++++++
 tests/Unit/Asset/Encoder/EncoderTest.php      | 71 +++++++++++++++++
 7 files changed, 301 insertions(+), 1 deletion(-)
 create mode 100644 src/Asset/Controller/Data/TextController.php
 create mode 100644 src/Asset/Encoder/TextEncoder.php
 create mode 100644 src/Asset/Encoder/TextEncoderInterface.php
 create mode 100644 src/Exception/MaxFileSizeExceededException.php
 create mode 100644 src/OpenApi/Attributes/Response/Content/DataJson.php
 create mode 100644 tests/Unit/Asset/Encoder/EncoderTest.php

diff --git a/config/assets.yaml b/config/assets.yaml
index f980bafea..28aee31b2 100644
--- a/config/assets.yaml
+++ b/config/assets.yaml
@@ -13,4 +13,8 @@ services:
 
   # Hydrators
   Pimcore\Bundle\StudioBackendBundle\Asset\Hydrator\CustomSettingsHydratorInterface:
-    class: Pimcore\Bundle\StudioBackendBundle\Asset\Hydrator\CustomSettingsHydrator
\ No newline at end of file
+    class: Pimcore\Bundle\StudioBackendBundle\Asset\Hydrator\CustomSettingsHydrator
+
+  # Encoder
+  Pimcore\Bundle\StudioBackendBundle\Asset\Encoder\TextEncoderInterface:
+      class: Pimcore\Bundle\StudioBackendBundle\Asset\Encoder\TextEncoder
\ No newline at end of file
diff --git a/src/Asset/Controller/Data/TextController.php b/src/Asset/Controller/Data/TextController.php
new file mode 100644
index 000000000..226e2e548
--- /dev/null
+++ b/src/Asset/Controller/Data/TextController.php
@@ -0,0 +1,79 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\StudioBackendBundle\Asset\Controller\Data;
+
+use OpenApi\Attributes\Get;
+use Pimcore\Bundle\StaticResolverBundle\Models\Element\ServiceResolverInterface;
+use Pimcore\Bundle\StudioBackendBundle\Asset\Encoder\TextEncoderInterface;
+use Pimcore\Bundle\StudioBackendBundle\Controller\AbstractApiController;
+use Pimcore\Bundle\StudioBackendBundle\OpenApi\Attributes\Parameters\Path\IdParameter;
+use Pimcore\Bundle\StudioBackendBundle\OpenApi\Attributes\Response\Content\DataJson;
+use Pimcore\Bundle\StudioBackendBundle\OpenApi\Attributes\Response\Error\MethodNotAllowedResponse;
+use Pimcore\Bundle\StudioBackendBundle\OpenApi\Attributes\Response\Error\NotFoundResponse;
+use Pimcore\Bundle\StudioBackendBundle\OpenApi\Attributes\Response\Error\UnauthorizedResponse;
+use Pimcore\Bundle\StudioBackendBundle\OpenApi\Attributes\Response\Error\UnprocessableContentResponse;
+use Pimcore\Bundle\StudioBackendBundle\OpenApi\Attributes\Response\Error\UnsupportedMediaTypeResponse;
+use Pimcore\Bundle\StudioBackendBundle\OpenApi\Attributes\Response\SuccessResponse;
+use Pimcore\Bundle\StudioBackendBundle\OpenApi\Config\Tags;
+use Pimcore\Bundle\StudioBackendBundle\Util\Traits\ElementProviderTrait;
+use Symfony\Component\HttpFoundation\JsonResponse;
+use Symfony\Component\Routing\Attribute\Route;
+use Symfony\Component\Serializer\SerializerInterface;
+
+/**
+ * @internal
+ */
+final class TextController extends AbstractApiController
+{
+    use ElementProviderTrait;
+
+    public function __construct(
+        SerializerInterface $serializer,
+        private readonly ServiceResolverInterface $serviceResolver,
+        private readonly TextEncoderInterface $textEncoder,
+
+    ) {
+        parent::__construct($serializer);
+    }
+
+    #[Route('/assets/{id}/text', name: 'pimcore_studio_api_get_asset_data_text', methods: ['GET'])]
+    //#[IsGranted('STUDIO_API')]
+    //#[IsGranted(UserPermissions::ASSETS->value)]
+    #[Get(
+        path: self::API_PATH . '/assets/{id}/text',
+        operationId: 'getAssetDataTextById',
+        summary: 'Get asset data in text UTF8 representation by id',
+        security: self::SECURITY_SCHEME,
+        tags: [Tags::Assets->name]
+    )]
+    #[IdParameter(type: 'asset')]
+    #[SuccessResponse(
+        description: 'UTF8 encoded text data',
+        content: new DataJson('UTF 8 encoded text data')
+    )]
+    #[UnauthorizedResponse]
+    #[NotFoundResponse]
+    #[MethodNotAllowedResponse]
+    #[UnsupportedMediaTypeResponse]
+    #[UnprocessableContentResponse]
+    public function getTextData(int $id): JsonResponse
+    {
+        $element = $this->getElement($this->serviceResolver, 'asset', $id);
+
+        return $this->jsonResponse(['data' => $this->textEncoder->encodeUTF8($element)]);
+    }
+}
diff --git a/src/Asset/Encoder/TextEncoder.php b/src/Asset/Encoder/TextEncoder.php
new file mode 100644
index 000000000..bc35fb6b6
--- /dev/null
+++ b/src/Asset/Encoder/TextEncoder.php
@@ -0,0 +1,44 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\StudioBackendBundle\Asset\Encoder;
+
+use ForceUTF8\Encoding;
+use Pimcore\Bundle\StudioBackendBundle\Exception\InvalidElementTypeException;
+use Pimcore\Bundle\StudioBackendBundle\Exception\MaxFileSizeExceededException;
+use Pimcore\Model\Asset\Text;
+use Pimcore\Model\Element\ElementInterface;
+
+final class TextEncoder implements TextEncoderInterface
+{
+    private const MAX_FILE_SIZE = 2000000;
+
+    /**
+     * @throws InvalidElementTypeException|MaxFileSizeExceededException
+     */
+    public function encodeUTF8(ElementInterface $element): string
+    {
+        if (!$element instanceof Text) {
+            throw new InvalidElementTypeException('Element must be an instance of Text');
+        }
+
+        if ($element->getFileSize() > self::MAX_FILE_SIZE) {
+            throw new MaxFileSizeExceededException(self::MAX_FILE_SIZE);
+        }
+
+        return Encoding::toUTF8($element->getData());
+    }
+}
diff --git a/src/Asset/Encoder/TextEncoderInterface.php b/src/Asset/Encoder/TextEncoderInterface.php
new file mode 100644
index 000000000..a8aeb5574
--- /dev/null
+++ b/src/Asset/Encoder/TextEncoderInterface.php
@@ -0,0 +1,29 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\StudioBackendBundle\Asset\Encoder;
+
+use Pimcore\Bundle\StudioBackendBundle\Exception\InvalidElementTypeException;
+use Pimcore\Bundle\StudioBackendBundle\Exception\MaxFileSizeExceededException;
+use Pimcore\Model\Element\ElementInterface;
+
+interface TextEncoderInterface
+{
+    /**
+     * @throws InvalidElementTypeException|MaxFileSizeExceededException
+     */
+    public function encodeUTF8(ElementInterface $element): string;
+}
diff --git a/src/Exception/MaxFileSizeExceededException.php b/src/Exception/MaxFileSizeExceededException.php
new file mode 100644
index 000000000..ddb11f3be
--- /dev/null
+++ b/src/Exception/MaxFileSizeExceededException.php
@@ -0,0 +1,31 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\StudioBackendBundle\Exception;
+
+/**
+ * @internal
+ */
+final class MaxFileSizeExceededException extends AbstractApiException
+{
+    public function __construct(int|float $maxFileSize)
+    {
+        parent::__construct(
+            413,
+            sprintf('Max file size of %d bytes exceeded', $maxFileSize)
+        );
+    }
+}
diff --git a/src/OpenApi/Attributes/Response/Content/DataJson.php b/src/OpenApi/Attributes/Response/Content/DataJson.php
new file mode 100644
index 000000000..542f6e32d
--- /dev/null
+++ b/src/OpenApi/Attributes/Response/Content/DataJson.php
@@ -0,0 +1,42 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\StudioBackendBundle\OpenApi\Attributes\Response\Content;
+
+use OpenApi\Attributes\JsonContent;
+use OpenApi\Attributes\Property;
+
+/**
+ * @internal
+ */
+final class DataJson extends JsonContent
+{
+    public function __construct(string $description = '')
+    {
+        parent::__construct(
+            properties: [
+                new Property(
+                    'data',
+                    title: 'data',
+                    description: $description,
+                    type: 'string',
+                    example: 'Test content'
+                ),
+            ],
+            type: 'object',
+        );
+    }
+}
diff --git a/tests/Unit/Asset/Encoder/EncoderTest.php b/tests/Unit/Asset/Encoder/EncoderTest.php
new file mode 100644
index 000000000..24e8cc761
--- /dev/null
+++ b/tests/Unit/Asset/Encoder/EncoderTest.php
@@ -0,0 +1,71 @@
+<?php
+declare(strict_types=1);
+
+/**
+ * Pimcore
+ *
+ * This source file is available under two different licenses:
+ * - GNU General Public License version 3 (GPLv3)
+ * - Pimcore Commercial License (PCL)
+ * Full copyright and license information is available in
+ * LICENSE.md which is distributed with this source code.
+ *
+ *  @copyright  Copyright (c) Pimcore GmbH (http://www.pimcore.org)
+ *  @license    http://www.pimcore.org/license     GPLv3 and PCL
+ */
+
+namespace Pimcore\Bundle\StudioBackendBundle\Tests\Unit\Asset\Encoder;
+
+use Codeception\Test\Unit;
+use Exception;
+use Pimcore\Bundle\StudioBackendBundle\Asset\Encoder\TextEncoder;
+use Pimcore\Bundle\StudioBackendBundle\Asset\Encoder\TextEncoderInterface;
+use Pimcore\Bundle\StudioBackendBundle\Exception\InvalidElementTypeException;
+use Pimcore\Bundle\StudioBackendBundle\Exception\MaxFileSizeExceededException;
+use Pimcore\Model\Asset\Document;
+use Pimcore\Model\Asset\Text;
+
+/**
+ * @internal
+ */
+final class EncoderTest extends Unit
+{
+    private TextEncoderInterface $encoder;
+
+    public function _before(): void
+    {
+        $this->encoder = new TextEncoder();
+    }
+
+    public function testWrongElementType(): void
+    {
+        $element = new Document();
+
+        $this->expectException(InvalidElementTypeException::class);
+
+        $this->encoder->encodeUTF8($element);
+    }
+
+    /**
+     * @throws Exception
+     */
+    public function testFileSizeExceeded(): void
+    {
+        $element = $this->makeEmpty(Text::class, ['getFileSize' => 2000001]);
+
+        $this->expectException(MaxFileSizeExceededException::class);
+
+        $this->encoder->encodeUTF8($element);
+    }
+
+    /**
+     * @throws Exception
+     */
+    public function testUTF8Encoding(): void
+    {
+        $element = $this->makeEmpty(Text::class, ['getData' => 'Héllö, 世界!']);
+        $encodedData = $this->encoder->encodeUTF8($element);
+
+        $this->assertTrue(mb_check_encoding($encodedData, 'UTF-8'));
+    }
+}