Symplify\PHP7_Sculpin\Bundle\PaginationBundle\SculpinPaginationExtension

Complexity

Total Complexity

1

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	loadConfiguration()	0	7	1

<?php

/*
 * This file is a part of Sculpin.
 *
 * (c) Dragonfly Development Inc.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Symplify\PHP7_Sculpin\Bundle\PaginationBundle;

use Nette\DI\Compiler;
use Nette\DI\CompilerExtension;
use Symplify\PHP7_Sculpin\Bundle\PaginationBundle\DependencyInjection\Configuration;
use Symfony\Component\Config\Definition\ConfigurationInterface;

final class SculpinPaginationExtension extends CompilerExtension
{
    public function loadConfiguration()
    {
        Compiler::loadDefinitions(
            $this->getContainerBuilder(),
            $this->loadFromFile(__DIR__.'/Resources/config/services.yml')

It seems like $this->loadFromFile(__DIR__ . '/Resources/config/services.yml') targeting Nette\DI\CompilerExtension::loadFromFile() can also be of type string; however, Nette\DI\Compiler::loadDefinitions() does only seem to accept array, maybe add an additional type check?

        );
    }

    // todo: set configuration parameter alias

//    public function getConfiguration() : ConfigurationInterface

45% of this comment could be valid code. Did you maybe forget this after debugging?

//    {
//        return new Configuration();
//    }
//
//    protected function getParametersAliases() : array
//    {
//        return [
//            'max_per_page' => 'sculpin_pagination.max_per_page',
//        ];
//    }
}