OAuth2Framework\Bundle\DependencyInjection\Component\Endpoint\ClientRegistrationSource

Complexity

Total Complexity

5

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	continueLoading()	0	9	2
A	name()	0	4	1
A	getNodeDefinition()	0	8	1

<?php

declare(strict_types=1);

/*
 * The MIT License (MIT)
 *
 * Copyright (c) 2014-2018 Spomky-Labs
 *
 * This software may be modified and distributed under the terms
 * of the MIT license.  See the LICENSE file for details.
 */

namespace OAuth2Framework\Bundle\DependencyInjection\Component\Endpoint;

use Fluent\PhpConfigFileLoader;
use OAuth2Framework\Bundle\DependencyInjection\Component\Component;
use Symfony\Component\Config\Definition\Builder\NodeDefinition;
use Symfony\Component\Config\FileLocator;
use Symfony\Component\DependencyInjection\ContainerBuilder;

final class ClientRegistrationSource implements Component

There is at least one abstract method in this class. Maybe declare it as abstract, or implement the remaining methods: load, prepend

{
    /**
     * ClientRegistrationSource constructor.
     */
    public function __construct()
    {
        $this->addSubSource(new ClientRegistrationInitialAccessTokenSource());

The method addSubSource() does not seem to exist on object<OAuth2Framework\Bundle\DependencyInjection\Component\Endpoint\ClientRegistrationSource>.

        $this->addSubSource(new ClientRegistrationSoftwareStatementSource());

The method addSubSource() does not seem to exist on object<OAuth2Framework\Bundle\DependencyInjection\Component\Endpoint\ClientRegistrationSource>.

    }

    /**
     * {@inheritdoc}
     */
    protected function continueLoading(string $path, ContainerBuilder $container, array $config)
    {
        foreach ($config as $k => $v) {
            $container->setParameter($path.'.'.$k, $v);
        }

        $loader = new PhpConfigFileLoader($container, new FileLocator(__DIR__.'/../../../Resources/config/endpoint'));
        $loader->load('client_registration.php');
    }

    /**
     * {@inheritdoc}
     */
    public function name(): string
    {
        return 'client_registration';
    }

    /**
     * {@inheritdoc}
     */
    public function getNodeDefinition(NodeDefinition $node)
    {

        $node

It seems like you code against a specific sub-type and not the parent class Symfony\Component\Config\Definition\Builder\NodeDefinition as the method children() does only exist in the following sub-classes of Symfony\Component\Config\Definition\Builder\NodeDefinition: Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition. Maybe you want to instanceof check for one of these explicitly?

            ->children()
                ->scalarNode('path')->defaultValue('/client/management')->end()
            ->end();
    }
}