LearnZF2Themes\Factory\Controller\IndexControllerFactory

Complexity

Total Complexity

1

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes	4
Bugs	1	Features	2

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	9	1

<?php

/**
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
 * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
 * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
 * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
 * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 *
 * This software consists of voluntary contributions made by many individuals
 * and is licensed under the MIT license.
 */
namespace LearnZF2Themes\Factory\Controller;

use LearnZF2Themes\Controller\IndexController;
use Zend\Mvc\Controller\ControllerManager;

/**
 * @author Stanimir Dimitrov <stanimirdim92@gmail.com>
 */
final class IndexControllerFactory
{
    /**
     * {@inheritdoc}
     */
    public function __invoke(ControllerManager $controllerManager)
    {
        $serviceLocator = $controllerManager->getServiceLocator();
        $themesConfig = $serviceLocator->get('getThemesFromDir');
        $reloadService = $serviceLocator->get('reloadService');
        $controller = new IndexController($themesConfig, $reloadService);

It seems like $themesConfig defined by $serviceLocator->get('getThemesFromDir') on line 35 can also be of type object; however, LearnZF2Themes\Controller\IndexController::__construct() does only seem to accept array, maybe add an additional type check?

        return $controller;
    }
}