BitBag\SyliusMolliePlugin\Resolver\PaymentMethodConfigResolver

Complexity

Total Complexity

2

Size/Duplication

Total Lines	17
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getConfigFromMethodId()	0	7	1

<?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\Resolver;

use BitBag\SyliusMolliePlugin\Entity\MollieGatewayConfigInterface;
use Sylius\Component\Resource\Repository\RepositoryInterface;
use Webmozart\Assert\Assert;

final class PaymentMethodConfigResolver implements PaymentMethodConfigResolverInterface
{
    /** @var RepositoryInterface */
    private $mollieMethodRepository;

    public function __construct(RepositoryInterface $mollieMethodRepository)
    {
        $this->mollieMethodRepository = $mollieMethodRepository;
    }

    public function getConfigFromMethodId(string $methodId): MollieGatewayConfigInterface
    {
        $paymentMethod = $this->mollieMethodRepository->findOneBy(['methodId' => $methodId]);

        Assert::notNull($paymentMethod, sprintf('Cannot find payment method config with method id %s', $methodId));

        return $paymentMethod;

The expression return $paymentMethod could return the type null which is incompatible with the type-hinted return BitBag\SyliusMolliePlugin\Entity\MollieGatewayConfigInterface. Consider adding an additional type-check to rule them out.

    }
}