Sylius\Bundle\CoreBundle\Validator\Constraints\OrderShippingMethodEligibilityValidator

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	validate()	0	18	4

<?php

/*
 * This file is part of the Sylius package.
 *
 * (c) Paweł Jędrzejewski
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sylius\Bundle\CoreBundle\Validator\Constraints;

use Sylius\Component\Core\Model\OrderInterface;
use Sylius\Component\Shipping\Checker\ShippingMethodEligibilityCheckerInterface;
use Symfony\Component\Validator\Constraint;
use Symfony\Component\Validator\ConstraintValidator;
use Webmozart\Assert\Assert;

/**
 * @author Arkadiusz Krakowiak <arkadiusz.krakowiak@lakion.com>
 */
final class OrderShippingMethodEligibilityValidator extends ConstraintValidator
{
    /**
     * @var ShippingMethodEligibilityCheckerInterface
     */
    private $methodEligibilityChecker;

    /**
     * @param ShippingMethodEligibilityCheckerInterface $methodEligibilityChecker
     */
    public function __construct(ShippingMethodEligibilityCheckerInterface $methodEligibilityChecker)

The variable name $methodEligibilityChecker exceeds the maximum configured length of 20.

    {
        $this->methodEligibilityChecker = $methodEligibilityChecker;
    }

    /**
     * @param OrderInterface $value

There is no parameter named $value. Was it maybe removed?

     *
     * {@inheritdoc}
     */
    public function validate($order, Constraint $constraint)
    {
        Assert::isInstanceOf($order, OrderInterface::class);

        $shipments = $order->getShipments();
        if ($shipments->isEmpty()) {
            return;
        }

        foreach ($shipments as $shipment) {
            if (!$this->methodEligibilityChecker->isEligible($shipment, $shipment->getMethod())) {
                $this->context->addViolation(
                    $constraint->message,
                    ['%shippingMethodName%' => $shipment->getMethod()->getName()]
                );
            }
        }
    }
}