BitBag\SyliusWishlistPlugin\CommandHandler\Wishlist\CopySelectedProductsToOtherWishlistHandler

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	9	1
A	__construct()	0	4	1
A	copyWishlistProductsToOtherWishlist()	0	10	2

<?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\SyliusWishlistPlugin\CommandHandler\Wishlist;

use BitBag\SyliusWishlistPlugin\Command\Wishlist\CopySelectedProductsToOtherWishlist;
use BitBag\SyliusWishlistPlugin\Command\Wishlist\WishlistItemInterface;
use BitBag\SyliusWishlistPlugin\Controller\Action\AddProductVariantToWishlistAction;
use Doctrine\Common\Collections\Collection;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\RequestStack;

final class CopySelectedProductsToOtherWishlistHandler
{
    private AddProductVariantToWishlistAction $addProductVariantToWishlistAction;

    private RequestStack $requestStack;

    public function __construct(AddProductVariantToWishlistAction $addProductVariantToWishlistAction, RequestStack $requestStack)
    {
        $this->addProductVariantToWishlistAction = $addProductVariantToWishlistAction;
        $this->requestStack = $requestStack;
    }

    public function __invoke(CopySelectedProductsToOtherWishlist $copySelectedProductsToOtherWishlistCommand): void
    {
        $destinedWishlistId = $copySelectedProductsToOtherWishlistCommand->getDestinedWishlistId();

        $currentRequest = $this->requestStack->getCurrentRequest();

        $this->copyWishlistProductsToOtherWishlist($copySelectedProductsToOtherWishlistCommand->getWishlistProducts(), $currentRequest);

It seems like $currentRequest can also be of type null; however, parameter $request of BitBag\SyliusWishlistPlugin\CommandHandler\Wishlist\CopySelectedProductsToOtherWishlistHandler::copyWishlistProductsToOtherWishlist() does only seem to accept Symfony\Component\HttpFoundation\Request, maybe add an additional type check?

        $this->addProductVariantToWishlistAction->__invoke($destinedWishlistId, $currentRequest);

It seems like $currentRequest can also be of type null; however, parameter $request of BitBag\SyliusWishlistPlugin\Controller\Action\AddProductVariantToWishlistAction::__invoke() does only seem to accept Symfony\Component\HttpFoundation\Request, maybe add an additional type check?

    }

    private function copyWishlistProductsToOtherWishlist(Collection $wishlistProducts, Request $request): void
    {
        $variantIds = [];

        /** @var WishlistItemInterface $wishlistProduct */
        foreach ($wishlistProducts as $wishlistProduct) {
            $variantIds[] = $wishlistProduct->getCartItem()->getCartItem()->getVariant()->getId();
        }

        $request->attributes->set('variantId', $variantIds);
    }
}