Drone\Mvc\ModuleFactory

Complexity

Total Complexity

6

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	create()	0	26	6

<?php
/**
 * DronePHP (http://www.dronephp.com)
 *
 * @link      http://github.com/Pleets/DronePHP
 * @copyright Copyright (c) 2016-2018 Pleets. (http://www.pleets.org)
 * @license   http://www.dronephp.com/license
 * @author    Darío Rivera <fermius.us@gmail.com>
 */

namespace Drone\Mvc;

use Drone\Mvc\Exception;

/**
 * ModuleFactory Class
 *
 * This class creates a module instance
 */
class ModuleFactory
{
    /**
     * Creates the module instance
     *
     * @param string             $module
     * @param array              $module_settings
     *
     * @return null
     */
    public static function create($module, Array $module_settings = [])
    {
        if (empty($module))
            throw new \RuntimeException("The module name must be specified");

        if (!is_string($module))

The condition is_string($module) is always true.

            throw new \InvalidArgumentException("Invalid type given. string expected");

        /*
         * Module class instantiation
         *
         * Each module must have a class called Module in her namespace. This class
         * is initilized here and can change the behavior of a controller using
         * allowExecution() or disallowExecution().
         */
        $fqn_module = "\\" . $module . "\\Module";

        if (!class_exists($fqn_module))
            throw new Exception\ModuleNotFoundException("The module class '$fqn_module' does not exists!");

        $module = new $fqn_module($module);

        if (array_key_exists('config', $module_settings) && !is_null($module_settings["config"]))
            $module->setConfigFile($module_settings["config"]);

        return $module;

The expression return $module returns the type object which is incompatible with the documented return type null.

    }
}