Eccube\DependencyInjection\Compiler\PaymentMethodPass::process()

Complexity

Conditions	3
Paths	3

Size

Total Lines

14

Duplication

Lines	9
Ratio	64.29 %

Code Coverage

Tests	7
CRAP Score	3.0987

Importance

Changes	1
Bugs	0	Features	0

<?php
    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);
        }
    }
}