BitBag\SyliusCmsPlugin\Controller\ResourceDataProcessingTrait

Complexity

Total Complexity

14

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	setFileContentsAsMediaPath()	0	5	1
A	setResourceMediaPath()	0	11	4
A	getRequestConfiguration()	0	3	1
A	getResourceInterface()	0	5	3
A	setPathForImageMediaType()	0	10	2
A	getFormForResource()	0	3	1
A	getMediaPathIfNotNull()	0	6	1
A	setPathForNonImageMediaType()	0	7	1

<?php

declare(strict_types=1);

namespace BitBag\SyliusCmsPlugin\Controller;

use BitBag\SyliusCmsPlugin\Entity\MediaInterface;
use Sylius\Bundle\ResourceBundle\Controller\RequestConfiguration;
use Sylius\Component\Resource\Model\ResourceInterface;
use Symfony\Component\Form\FormInterface;
use Symfony\Component\HttpFoundation\File\File;
use Symfony\Component\HttpFoundation\Request;
use Webmozart\Assert\Assert;

trait ResourceDataProcessingTrait
{
    private function getResourceInterface(Request $request): object
    {
        return null !== $request->get('id') && $this->repository->find($request->get('id')) ?
            $this->repository->find($request->get('id')) :
            $this->factory->createNew();
    }

    private function getFormForResource(RequestConfiguration $configuration, ResourceInterface $resource): FormInterface
    {
        return $this->resourceFormFactory->create($configuration, $resource);
    }

    private function getRequestConfiguration(Request $request): RequestConfiguration
    {
        return $this->requestConfigurationFactory->create($this->metadata, $request);
    }

    private function setResourceMediaPath(MediaInterface $media): void
    {
        if (null === $media->getPath()) {
            return;
        }
        Assert::notNull($media->getMimeType());
        Assert::notNull($media->getType());
        if (1 === preg_match("/image\//", $media->getMimeType()) && 'image' === $media->getType()) {

It seems like $media->getMimeType() can also be of type null; however, parameter $subject of preg_match() does only seem to accept string, maybe add an additional type check?

            $this->setPathForImageMediaType($media);
        } else {
            $this->setPathForNonImageMediaType($media);
        }
    }

    private function setPathForImageMediaType(MediaInterface $media): void
    {
        Assert::string($media->getPath());
        if (!$this->cacheManager->isStored($media->getPath(), $this::FILTER)) {

The constant BitBag\SyliusCmsPlugin\Controller\ResourceDataProcessingTrait::FILTER was not found. Maybe you did not declare it correctly or list all dependencies?

            $this->cacheManager->store($this->dataManager->find($this::FILTER, $media->getPath()), $media->getPath(), $this::FILTER);
        }
        $resolvedPath = $this->cacheManager->resolve($media->getPath(), $this::FILTER);
        $fileContents = file_get_contents($resolvedPath);
        Assert::string($fileContents);
        $this->setFileContentsAsMediaPath($media, $fileContents);
    }

    private function setPathForNonImageMediaType(MediaInterface $media): void
    {
        $mediaPath = $this->getMediaPathIfNotNull($media);
        $file = new File($mediaPath);
        $fileContents = file_get_contents($file->getPathname());
        Assert::string($fileContents);
        $this->setFileContentsAsMediaPath($media, $fileContents);
    }

    private function setFileContentsAsMediaPath(MediaInterface $media, string $fileContents): void
    {
        $base64Content = base64_encode($fileContents);
        $path = 'data:' . $media->getMimeType() . ';base64, ' . $base64Content;
        $media->setPath($path);
    }

    private function getMediaPathIfNotNull(MediaInterface $media): string
    {
        Assert::string($media->getPath());
        Assert::string($this->getParameter('sylius_core.public_dir'));

It seems like getParameter() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $this->getParameter('sylius_core.public_dir') . $media->getPath();
    }
}