Tvi\MonitorBundle\Check\rabbitmq\QueueSize\Plugin::checkFactoryConf()

Complexity

Conditions	4
Paths	1

Size

Total Lines	24
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	4

Importance

Changes

0

<?php
    public function checkFactoryConf(TreeBuilder $builder): ArrayNodeDefinition

Missing doc comment for function checkFactoryConf()

    {
        /* @var ArrayNodeDefinition $node */
        $node = parent::checkFactoryConf($builder);

        $node = $node
            ->beforeNormalization()
                ->ifArray()

Object operator not indented correctly; expected 12 spaces but found 16

                ->then(static function ($value) {

Object operator not indented correctly; expected 12 spaces but found 16

The opening parenthesis of a multi-line function call should be the last content on the line.

                    if (isset($value['dsn'])) {
                        foreach ($value['items'] as &$v) {
                            if (!array_key_exists('dsn', $v['check'])) {
                                $v['check']['dsn'] = $value['dsn'];
                            }
                        }
                    }

                    return $value;
                })

For multi-line function calls, the closing parenthesis should be on a new line.

            ->end();

        $node->children()->scalarNode('dsn')->defaultNull()->end();

        return $node;

The expression return $node returns the type Symfony\Component\Config\Definition\Builder\NodeDefinition which includes types incompatible with the type-hinted return Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition.

    }