Eccube\DependencyInjection\Compiler\PaymentMethodPass

Complexity

Total Complexity

3

Size/Duplication

Total Lines	19
Duplicated Lines	47.37 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

77.78%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	process()	9	14	3

<?php

/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) LOCKON CO.,LTD. All Rights Reserved.
 *
 * http://www.lockon.co.jp/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Eccube\DependencyInjection\Compiler;

use Eccube\Service\Payment\PaymentMethodInterface;
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;

class PaymentMethodPass implements CompilerPassInterface

Missing class doc comment

{
    const PAYMENT_METHOD_TAG = 'eccube.payment.method';

    public function process(ContainerBuilder $container)
    {
        $ids = $container->findTaggedServiceIds(self::PAYMENT_METHOD_TAG);

        foreach ($ids as $id => $tags) {
            $def = $container->getDefinition($id);
            $class = $container->getParameterBag()->resolveValue($def->getClass());
            if (!is_subclass_of($class, PaymentMethodInterface::class)) {

Due to PHP Bug #53727, is_subclass_of might return inconsistent results on some PHP versions if \Eccube\Service\Payment\PaymentMethodInterface::class can be an interface. If so, you could instead use ReflectionClass::implementsInterface.

                throw new \InvalidArgumentException(
                    sprintf('Service "%s" must implement interface "%s".', $id, PaymentMethodInterface::class));
            }
            $def->setPublic(true);
        }
    }
}