BitBag\SyliusMolliePlugin\Validator\Constraints\CurrencyValidator::validate()

Complexity

Conditions	6
Paths	4

Size

Total Lines	30
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function validate($paymentMethod, Constraint $constraint): void
    {
        Assert::isInstanceOf($paymentMethod, PaymentMethodInterface::class);

        Assert::isInstanceOf($constraint, Currency::class);

        $gatewayConfig = $paymentMethod->getGatewayConfig();

        if (
            null === $gatewayConfig ||
            (
                $gatewayConfig->getFactoryName() !== MollieSubscriptionGatewayFactory::FACTORY_NAME
            )
        ) {
            return;
        }

        /** @var ChannelInterface $channel */
        foreach ($paymentMethod->getChannels() as $channel) {
            if (
                null === $channel->getBaseCurrency() ||
                false === in_array(strtoupper($channel->getBaseCurrency()->getCode()), MollieSubscriptionGatewayFactory::CURRENCIES_AVAILABLE)
            ) {
                $message = $constraint->message ?? null;

                $this->context->buildViolation($message, [

It seems like $message can also be of type null; however, parameter $message of Symfony\Component\Validator\Context\ExecutionContextInterface::buildViolation() does only seem to accept string, maybe add an additional type check?

                    '{{ currencies }}' => implode(', ', MollieSubscriptionGatewayFactory::CURRENCIES_AVAILABLE),
                ])->atPath('channels')->addViolation();

                return;
            }