BitBagCommerce / SyliusMolliePlugin

src/Uploader/PaymentMethodLogoUploader.php

<?php

/*
 * This file has been created by developers from BitBag.
 * Feel free to contact us once you face any issues or want to start
 * You can find more information about us on https://bitbag.io and write us
 * an email on hello@bitbag.io.
 */

declare(strict_types=1);

namespace BitBag\SyliusMolliePlugin\Uploader;

use BitBag\SyliusMolliePlugin\Entity\MollieGatewayConfigInterface;
use Doctrine\Common\Collections\Collection;
use Gaufrette\Filesystem;

final class PaymentMethodLogoUploader implements PaymentMethodLogoUploaderInterface
{
    /** @var Filesystem */
    private $filesystem;

    public function __construct(Filesystem $filesystem)
    {
        $this->filesystem = $filesystem;
    }

    public function upload(Collection $mollieGatewayConfigs): void
    {
        /** @var MollieGatewayConfigInterface $mollieGatewayConfig */
        foreach ($mollieGatewayConfigs as $mollieGatewayConfig) {
            if ($mollieGatewayConfig->getCustomizeMethodImage()->hasFile()) {
                $this->uploadSingle($mollieGatewayConfig);
            }
        }
    }

    public function remove(string $path): bool
    {
        if ($this->filesystem->has($path)) {
            return $this->filesystem->delete($path);
        }

        return false;
    }

    private function uploadSingle(MollieGatewayConfigInterface $mollieGatewayConfig): void
    {
        $customizeImage = $mollieGatewayConfig->getCustomizeMethodImage();
        $file = $customizeImage->getFile();

        if (null !== $customizeImage->getPath() && $this->fileExists($customizeImage->getPath())) {
            $this->remove($customizeImage->getPath());
        }

        do {
            $hash = md5(uniqid((string) mt_rand(), true));
            $path = $this->expandPath($hash . '.' . $file->guessExtension());

The method guessExtension() does not exist on SplFileInfo. It seems like you code against a sub-type of SplFileInfo such as ECSPrefix20210626\Symfony\Component\HttpFoundation\File\File or Symfony\Component\HttpFoundation\File\File.

        } while ($this->isAdBlockingProne($path) || $this->filesystem->has($path));

        $customizeImage->setPath($path);
        $customizeImage->setName($file->getClientOriginalName());

The method getClientOriginalName() does not exist on SplFileInfo. It seems like you code against a sub-type of SplFileInfo such as ECSPrefix20210626\Symfony\Component\HttpFoundation\File\UploadedFile or Symfony\Component\HttpFoundation\File\UploadedFile.

        $this->filesystem->write(
            $customizeImage->getPath(),
            file_get_contents($file->getPathname())
        );
    }

    private function fileExists(string $path): bool
    {
        return $this->filesystem->has($path);
    }

    private function isAdBlockingProne(string $path): bool
    {
        return strpos($path, 'ad') !== false;
    }

    private function expandPath(string $path): string
    {
        return sprintf(
            '%s/%s/%s',
            substr($path, 0, 2),
            substr($path, 2, 2),
            substr($path, 4)
        );
    }
}