JoppeDc\SyliusBetterSeoPlugin\EventListener\SeoTranslationImagesUploadListener

Complexity

Total Complexity

6

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	uploadMedia()	0	14	5
A	__construct()	0	3	1

<?php

declare(strict_types=1);

namespace JoppeDc\SyliusBetterSeoPlugin\EventListener;

use JoppeDc\SyliusBetterSeoPlugin\Entity\SeoImageInterface;
use JoppeDc\SyliusBetterSeoPlugin\Entity\SeoInterface;
use JoppeDc\SyliusBetterSeoPlugin\Entity\SeoTranslation;
use Sylius\Component\Core\Uploader\ImageUploaderInterface;
use Symfony\Component\EventDispatcher\GenericEvent;
use Webmozart\Assert\Assert;

class SeoTranslationImagesUploadListener
{
    /** @var ImageUploaderInterface */
    private $uploader;

    public function __construct(ImageUploaderInterface $uploader)
    {
        $this->uploader = $uploader;
    }

    public function uploadMedia(GenericEvent $event): void
    {
        $subject = $event->getSubject();
        Assert::isInstanceOf($subject, SeoInterface::class);
        /** @var SeoTranslation $translation */
        foreach ($subject->getSeo()->getTranslations() as $translation) {
            $image = $translation->getImage();
            if ($image instanceof SeoImageInterface) {
                if ($image->hasFile()) {

The method hasFile() does not exist on JoppeDc\SyliusBetterSeoPlugin\Entity\SeoImageInterface. Since it exists in all sub-types, consider adding an abstract or default implementation to JoppeDc\SyliusBetterSeoPlugin\Entity\SeoImageInterface.

                    $this->uploader->upload($image);
                }
                // Upload failed? Let's remove that image.
                if (null === $image->getPath()) {

The method getPath() does not exist on JoppeDc\SyliusBetterSeoPlugin\Entity\SeoImageInterface. Since it exists in all sub-types, consider adding an abstract or default implementation to JoppeDc\SyliusBetterSeoPlugin\Entity\SeoImageInterface.

                    $translation->setImage(null);
                }
            }
        }
    }
}