Fusible\ViewProvider\Config

Complexity

Total Complexity

5

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	15	3
A	view()	0	4	1
A	helper()	0	4	1

<?php
/**
 * Aura\View Provider for Aura\Di
 *
 * PHP version 5
 *
 * Copyright (C) 2016 Jake Johns
 *
 * This software may be modified and distributed under the terms
 * of the MIT license.  See the LICENSE file for details.
 *
 * @category  Config
 * @package   Fusible\ViewProvider
 * @author    Jake Johns <jake@jakejohns.net>
 * @copyright 2016 Jake Johns
 * @license   http://jnj.mit-license.org/2016 MIT License
 * @link      https://github.com/fusible/fusible.view-provider
 */

namespace Fusible\ViewProvider;

use Aura\Di\Container;
use Aura\Di\ConfigCollection;

/**
 * Config
 *
 * @category Config
 * @package  Fusible\ViewProvider
 * @author   Jake Johns <jake@jakejohns.net>
 * @license  http://jnj.mit-license.org/2016 MIT License
 * @link     https://github.com/fusible/fusible.view-provider
 *
 * @see ContainerConfig
 */
class Config extends ConfigCollection
{
    protected $configs = [];

    /**
     * __construct
     *
     * @param mixed $templates DESCRIPTION
     * @param array $helpers   DESCRIPTION
     *
     * @return mixed

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     *
     * @access public
     */
    public function __construct($templates = null, array $helpers = [])
    {
        $this->configs = [
            ViewConfig::class => new ViewConfig,
            ViewHelperConfig::class => new ViewHelperConfig
        ];

        if ($templates) {
            $this->view()->addTemplatePath($templates);
        }

        if ($helpers) {

The expression $helpers of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $this->helper()->addHelpers($helpers);
        }
    }

    /**
     * View
     *
     * @return mixed
     *
     * @access public
     */
    public function view()
    {
        return $this->configs[ViewConfig::class];
    }

    /**
     * Helper
     *
     * @return mixed
     * @throws exceptionclass [description]
     *
     * @access public
     */
    public function helper()
    {
        return $this->configs[ViewHelperConfig::class];
    }
}