BitBag\SyliusWishlistPlugin\CommandHandler\Wishlist\RemoveWishlistHandler

Complexity

Total Complexity

2

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	5	1
A	__construct()	0	7	1

<?php

declare(strict_types=1);

namespace BitBag\SyliusWishlistPlugin\CommandHandler\Wishlist;

use BitBag\SyliusWishlistPlugin\Command\Wishlist\RemoveWishlist;
use BitBag\SyliusWishlistPlugin\Repository\WishlistRepositoryInterface;
use BitBag\SyliusWishlistPlugin\Updater\WishlistUpdaterInterface;
use Symfony\Component\Messenger\Handler\MessageHandlerInterface;

final class RemoveWishlistHandler implements MessageHandlerInterface
{
    private WishlistRepositoryInterface $wishlistRepository;

    private WishlistUpdaterInterface $wishlistUpdater;

    public function __construct(
        WishlistRepositoryInterface $wishlistRepository,
        WishlistUpdaterInterface $wishlistUpdater
    )
    {
        $this->wishlistRepository = $wishlistRepository;
        $this->wishlistUpdater = $wishlistUpdater;
    }

    public function __invoke(RemoveWishlist $removeWishlist)
    {
        $wishlist = $this->wishlistRepository->findByToken($removeWishlist->getWishlistTokenValue());

        $this->wishlistUpdater->removeWishlist($wishlist);

It seems like $wishlist can also be of type null; however, parameter $wishlist of BitBag\SyliusWishlistPlugin\Updater\WishlistUpdaterInterface::removeWishlist() does only seem to accept BitBag\SyliusWishlistPlugin\Entity\WishlistInterface, maybe add an additional type check?

    }
}