WIP: showcase patch method

config/api_platform/resources/asset/image.yaml

@@ -6,7 +6,7 @@ resources:
             ApiPlatform\Metadata\Get:
                 normalizationContext:
                     groups: ['image:read', 'asset:read', 'asset:item:get', 'dependency:read', 'property:read', 'element:read', 'element:item:get' ]
-            ApiPlatform\Metadata\Put:
+            ApiPlatform\Metadata\Patch:
         properties:
             id:
                 identifier: true

config/services.yaml

@@ -26,7 +26,7 @@ services:
     Pimcore\Bundle\StudioApiBundle\State\VersionProvider: ~
 
     # Processors
-
+    Pimcore\Bundle\StudioApiBundle\State\Asset\Processor: ~
     Pimcore\Bundle\StudioApiBundle\State\ResetPasswordProcessor: ~
     Pimcore\Bundle\StudioApiBundle\State\Token\Create\Processor: ~
     Pimcore\Bundle\StudioApiBundle\State\Token\Refresh\Processor: ~

src/Dto/Asset.php

@@ -25,7 +25,7 @@ public function __construct(
         private readonly string $iconName,
         private readonly bool $hasChildren,
         private readonly string $type,
-        private readonly string $filename,
+        private string $filename,
         private readonly ?string $mimeType,
         private readonly array $metaData,
         private readonly bool $workflowWithPermissions,
@@ -70,11 +70,16 @@ public function hasWorkflowWithPermissions(): bool
         return $this->workflowWithPermissions;
     }
 
-    public function getFilename(): ?string
+    public function getFilename(): string
     {
         return $this->filename;
     }
 
+    public function setFilename(string $filename): void
+    {
+        $this->filename = $filename;
+    }
+
     public function getType(): string
     {
         return $this->type;

src/Dto/Element.php

@@ -23,7 +23,7 @@ class Element
 {
     public function __construct(
         private readonly int $id,
-        private readonly int $parentId,
+        private int $parentId,
         private readonly string $path,
         private readonly int $userOwner,
         private readonly int $userModification,
@@ -45,6 +45,11 @@ public function getParentId(): int
         return $this->parentId;
     }
 
+    public function setParentId(int $parentId): void
+    {
+        $this->parentId = $parentId;
+    }
+
     public function getPath(): string
     {
         return $this->path;

src/Service/AssetService.php

@@ -16,25 +16,63 @@
 
 namespace Pimcore\Bundle\StudioApiBundle\Service;
 
+use Pimcore\Bundle\GenericDataIndexBundle\Service\SearchIndex\IndexQueue\SynchronousProcessingServiceInterface;
 use Pimcore\Bundle\StaticResolverBundle\Models\Asset\AssetResolverInterface;
 use Pimcore\Bundle\StudioApiBundle\Dto\Asset;
-use Pimcore\Model\Asset as CoreAsset;
+use Pimcore\Model\Element\DuplicateFullPathException;
 use Pimcore\Model\Exception\NotFoundException;
+use Pimcore\Model\Asset as CoreAsset;
 
 final readonly class AssetService implements AssetServiceInterface
 {
-    public function __construct(private AssetResolverInterface $assetResolver)
+    public function __construct(
+        private AssetResolverInterface $assetResolver,
+        private SynchronousProcessingServiceInterface $synchronousProcessingService
+    )
     {
     }
 
-    public function processAsset(int $id, Asset $data): CoreAsset
+    /**
+     * @throws DuplicateFullPathException
+     */
+    public function processAsset(Asset $data): CoreAsset
     {
-        $asset = $this->assetResolver->getById($id);
+        $asset = $this->assetResolver->getById($data->getId());
 
         if(!$asset) {
             throw new NotFoundException('Asset not found');
         }
 
+        $differences = $this->compare($asset, $data);
+        foreach($differences as $field => $difference) {
+            $setter = 'set' . $field;
+            $asset->$setter($difference['new']);
+        }
+        $this->synchronousProcessingService->enable();
+        $asset->save();
         return $asset;
     }
+
+
+    private function compare(CoreAsset $current, Asset $data): array
+    {
+        $fieldsToCheck = $this->fieldsToCheck(get_class_methods($data));
+        $differences = [];
+        foreach($fieldsToCheck as $field) {
+            if($current->$field() !== $data->$field()) {
+                $fieldName = str_replace('get', '', $field);
+                $differences[$fieldName] = [
+                    'current' => $current->$field(),
+                    'new' => $data->$field()
+                ];
+            }
+        }
+        return $differences;
+    }
+
+    private function fieldsToCheck(array $methods): array
+    {
+        $setters = array_filter($methods, static fn ($method) => str_starts_with($method, 'set'));
+        return array_map(static fn ($setter) => str_replace('set', 'get', $setter), $setters);
+    }
 }

src/Service/AssetServiceInterface.php

@@ -20,5 +20,5 @@
 
 interface AssetServiceInterface
 {
-    public function processAsset(int $id, Asset $data): CoreAsset;
+    public function processAsset(Asset $data): CoreAsset;
 }

src/Service/CoreData/V1/Hydrator/AssetHydratorService.php

@@ -6,7 +6,7 @@
 use Pimcore\Model\Asset as CoreAsset;
 use Symfony\Contracts\Service\ServiceProviderInterface;
 
-final readonly class AssetHydratorService implements AssetHydratorInterface
+final readonly class AssetHydratorService implements AssetHydratorServiceInterface
 {
     public function __construct(
         private ServiceProviderInterface $assetHydratorLocator,

src/Service/CoreData/V1/Hydrator/AssetHydratorInterface.php → src/Service/CoreData/V1/Hydrator/AssetHydratorServiceInterface.php

@@ -19,7 +19,7 @@
 use Pimcore\Bundle\StudioApiBundle\Dto\Asset;
 use Pimcore\Model\Asset as CoreAsset;
 
-interface AssetHydratorInterface
+interface AssetHydratorServiceInterface
 {
     /**
      * @param CoreAsset $item

src/State/Asset/Processor.php

@@ -4,23 +4,34 @@
 
 use ApiPlatform\Exception\OperationNotFoundException;
 use ApiPlatform\Metadata\Operation;
-use ApiPlatform\Metadata\Put;
+use ApiPlatform\Metadata\Patch;
 use ApiPlatform\State\ProcessorInterface;
 use Pimcore\Bundle\StudioApiBundle\Dto\Asset;
+use Pimcore\Bundle\StudioApiBundle\Service\AssetSearchServiceInterface;
+use Pimcore\Bundle\StudioApiBundle\Service\AssetServiceInterface;
+use Pimcore\Bundle\StudioApiBundle\Service\CoreData\V1\Hydrator\AssetHydratorServiceInterface;
 
 final readonly class Processor implements ProcessorInterface
 {
 
+    public function __construct(
+        private AssetServiceInterface $assetService,
+        private AssetHydratorServiceInterface $assetHydratorService
+    )
+    {
+    }
+
     public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): Asset
     {
         if (
-            !$operation instanceof Put ||
+            !$operation instanceof Patch ||
             !$data instanceof Asset
         ) {
             // wrong operation
             throw new OperationNotFoundException();
         }
+        $asset = $this->assetService->processAsset($data);
 
-
+        return $this->assetHydratorService->hydrate($asset);
     }
 }