Odiseo\SyliusMailchimpPlugin\Handler\ProductRegisterHandler

Complexity

Total Complexity

22

Size/Duplication

Total Lines	189
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	2

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getVariantPrice()	0	11	2
A	getProductUrl()	0	22	3
A	__construct()	0	12	1
C	register()	0	70	12
A	unregister()	0	20	3
A	getImageUrl()	0	6	1

<?php

declare(strict_types=1);

namespace Odiseo\SyliusMailchimpPlugin\Handler;

use Liip\ImagineBundle\Imagine\Cache\CacheManager;
use Odiseo\SyliusMailchimpPlugin\Api\EcommerceInterface;
use Sylius\Component\Core\Model\ChannelInterface;
use Sylius\Component\Core\Model\ChannelPricingInterface;
use Sylius\Component\Core\Model\ProductImageInterface;
use Sylius\Component\Core\Model\ProductInterface;
use Sylius\Component\Core\Model\ProductVariant;
use Symfony\Component\EventDispatcher\GenericEvent;
use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
use Symfony\Component\Routing\RouterInterface;
use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;

final class ProductRegisterHandler implements ProductRegisterHandlerInterface
{
    /** @var EcommerceInterface */
    private $ecommerceApi;

    /** @var RouterInterface */
    private $router;

    /** @var CacheManager */
    private $cacheManager;

    /** @var EventDispatcherInterface */
    private $eventDispatcher;

    /** @var bool */
    private $enabled;

    public function __construct(
        EcommerceInterface $ecommerceApi,
        RouterInterface $router,
        CacheManager $cacheManager,
        EventDispatcherInterface $eventDispatcher,
        bool $enabled
    ) {
        $this->ecommerceApi = $ecommerceApi;
        $this->router = $router;
        $this->cacheManager = $cacheManager;
        $this->eventDispatcher = $eventDispatcher;
        $this->enabled = $enabled;
    }

    /**
     * {@inheritdoc}
     */
    public function register(ProductInterface $product, ChannelInterface $channel, bool $createOnly = false)
    {
        if (!$this->enabled) {
            return false;
        }

        $productId = (string) $product->getId();
        $storeId = $channel->getCode();

        $response = $this->ecommerceApi->getProduct($storeId, $productId);

It seems like $storeId can also be of type null; however, parameter $storeId of Odiseo\SyliusMailchimpPlugin\Api\EcommerceInterface::getProduct() does only seem to accept string, maybe add an additional type check?

        $isNew = !isset($response['id']);

        // Do nothing if the product exists
        if (false === $isNew && true === $createOnly) {
            return false;
        }

        $variants = [];
        /** @var ProductVariant $productVariant */
        foreach ($product->getVariants() as $productVariant) {
            $variant = [
                'id' => (string) $productVariant->getId(),
                'title' => $productVariant->getName() ? $productVariant->getName() : $product->getName(),
                'inventory_quantity' => $productVariant->isTracked() ? $productVariant->getOnHand() : 1,
            ];

            if ($variantPrice = $this->getVariantPrice($productVariant, $channel)) {
                $variant['price'] = $variantPrice / 100;
            }

            $variants[] = $variant;
        }

        $productImages = [];
        /** @var ProductImageInterface $image */
        foreach ($product->getImages() as $image) {
            $productImages[] = [
                'id' => (string) $image->getId(),
                'url' => $this->getImageUrl($image, $channel),
            ];
        }

        $data = [
            'id' => $productId,
            'title' => $product->getName(),
            'url' => $this->getProductUrl($product, $channel),
            'description' => $product->getDescription() ?: '',
            'images' => $productImages,
            'variants' => $variants,
        ];

        if (count($productImages) > 0) {
            $data['image_url'] = $productImages[0]['url'];
        }

        if ($isNew) {
            $event = new GenericEvent($product, ['data' => $data, 'channel' => $channel]);
            $this->eventDispatcher->dispatch($event, 'mailchimp.product.pre_add');

The call to Symfony\Contracts\EventDispatcher\EventDispatcherInterface::dispatch() has too many arguments starting with 'mailchimp.product.pre_add'.

            $data = $event->getArgument('data');

            $response = $this->ecommerceApi->addProduct($storeId, $data);

It seems like $storeId can also be of type null; however, parameter $storeId of Odiseo\SyliusMailchimpPlugin\Api\EcommerceInterface::addProduct() does only seem to accept string, maybe add an additional type check?

        } else {
            $event = new GenericEvent($product, ['data' => $data, 'channel' => $channel]);
            $this->eventDispatcher->dispatch($event, 'mailchimp.product.pre_update');
            $data = $event->getArgument('data');

            $response = $this->ecommerceApi->updateProduct($storeId, $productId, $data);

It seems like $storeId can also be of type null; however, parameter $storeId of Odiseo\SyliusMailchimpPlugin\Api\EcommerceInterface::updateProduct() does only seem to accept string, maybe add an additional type check?

        }

        return $response;
    }

    /**
     * {@inheritdoc}
     */
    public function unregister(ProductInterface $product, ChannelInterface $channel)
    {
        if (!$this->enabled) {
            return false;
        }

        $productId = (string) $product->getId();
        $storeId = $channel->getCode();

        $response = $this->ecommerceApi->getProduct($storeId, $productId);

It seems like $storeId can also be of type null; however, parameter $storeId of Odiseo\SyliusMailchimpPlugin\Api\EcommerceInterface::getProduct() does only seem to accept string, maybe add an additional type check?

        $isNew = !isset($response['id']);

        if (!$isNew) {
            $event = new GenericEvent($product, ['channel' => $channel]);
            $this->eventDispatcher->dispatch($event, 'mailchimp.product.pre_remove');

The call to Symfony\Contracts\EventDispatcher\EventDispatcherInterface::dispatch() has too many arguments starting with 'mailchimp.product.pre_remove'.

            return $this->ecommerceApi->removeProduct($storeId, $productId);

It seems like $storeId can also be of type null; however, parameter $storeId of Odiseo\SyliusMailchimpPlugin\Api\EcommerceInterface::removeProduct() does only seem to accept string, maybe add an additional type check?

        }

        return false;
    }

    /**
     * @param ProductVariant $variant
     * @param ChannelInterface $channel
     * @return int|null
     */
    private function getVariantPrice(ProductVariant $variant, ChannelInterface $channel): ?int
    {
        /** @var ChannelPricingInterface $channelPricing */
        $channelPricing = $variant->getChannelPricingForChannel($channel);

        $variantPrice = null;
        if ($channelPricing instanceof ChannelPricingInterface) {

$channelPricing is always a sub-type of Sylius\Component\Core\Model\ChannelPricingInterface.

            $variantPrice = $channelPricing->getPrice();
        }

        return $variantPrice;
    }

    /**
     * @param ProductInterface $product
     * @param ChannelInterface $channel
     * @return string
     */
    private function getProductUrl(ProductInterface $product, ChannelInterface $channel): string
    {
        $context = $this->router->getContext();
        $context->setHost($channel->getHostname());

        $locale = 'en';
        $channelDefaultLocale = $channel->getDefaultLocale();

        if (null !== $channelDefaultLocale) {
            $locale = $channelDefaultLocale->getCode();
        } else {
            if (count($channel->getLocales()) > 0) {
                $locale = $channel->getLocales()->first()->getCode();
            }
        }

        $product->setCurrentLocale($locale);

        return $this->router->generate('sylius_shop_product_show', [
            '_locale' => $locale,
            'slug' => $product->getSlug(),
        ], UrlGeneratorInterface::ABSOLUTE_URL);
    }

    /**
     * @param ProductImageInterface $image
     * @param ChannelInterface $channel
     * @return string
     */
    private function getImageUrl(ProductImageInterface $image, ChannelInterface $channel): string
    {
        $context = $this->router->getContext();
        $context->setHost($channel->getHostname());

        return $this->cacheManager->generateUrl($image->getPath(), 'sylius_large');
    }
}