SyncFS\Configuration\Configuration

Complexity

Total Complexity

2

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getConfigTreeBuilder()	0	9	1
A	getConfigNode()	0	23	1

<?php

namespace SyncFS\Configuration;

use Symfony\Component\Config\Definition\Builder\NodeDefinition;
use Symfony\Component\Config\Definition\Builder\TreeBuilder;
use Symfony\Component\Config\Definition\ConfigurationInterface;

/**
 * Class Configuration
 *
 * @package SyncFS\Configuration
 * @author  Matej Velikonja <matej@velikonja.si>
 */
class Configuration implements ConfigurationInterface
{
    /**
     * Generates the configuration tree builder.
     *
     * @return \Symfony\Component\Config\Definition\Builder\TreeBuilder The tree builder
     */
    public function getConfigTreeBuilder()
    {
        $treeBuilder = new TreeBuilder();
        $rootNode    = $treeBuilder->root('filesystem');

        $this->getConfigNode($rootNode);

        return $treeBuilder;
    }

    /**
     * @param NodeDefinition $node
     *
     * @return \Symfony\Component\Config\Definition\Builder\NodeDefinition
     */
    public function getConfigNode(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('default_client')->defaultValue('rsync')->end()
                ->arrayNode('maps')
                    ->isRequired()
                    ->requiresAtLeastOneElement()
                    ->useAttributeAsKey('name')
                    ->prototype('array')
                        ->children()
                            ->scalarNode('src')->isRequired()->end()
                            ->scalarNode('dst')->isRequired()->end()
                            ->scalarNode('client')->defaultNull()->end()
                        ->end()
                    ->end()
                ->end()
                ->scalarNode('timeout')->defaultValue(15 * 60)->end()
            ->end()
        ->end();

        return $node;
    }
}