JoppeDc\SyliusBetterSeoPlugin\EventListener\SeoTranslationImagesUploadListener::uploadMedia()

Complexity

Conditions	5
Paths	6

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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);
                }