Nip\Controllers\View\ControllerViewHydrator

Complexity

Total Complexity

12

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Test Coverage

Coverage

81.48%

Importance

Changes	2
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	initContentBlocks()	0	9	1
A	initVars()	0	13	3
A	populatePath()	0	8	2
A	detectRequest()	0	12	6

<?php

namespace Nip\Controllers\View;

use Nip\Controllers\Controller;
use Nip\Http\Request;
use Nip\Utility\Container;

/**
 * Class ControllerViewHydrator
 * @package Nip\Controllers\View
 */
class ControllerViewHydrator
{

    /**
     * @param $view
     * @param null $controller

Are you sure the doc-type for parameter $controller is correct as it would always require null to be passed?

     * @return mixed
     */
    public static function initContentBlocks($view, $controller = null)
    {
        $request = static::detectRequest($controller);

The assignment to $request is dead and can be removed.

        $view->setBlock(
            'content',
            $controller->getName() . '/' . $controller->getAction()

The method getName() does not exist on null.

        );

        return $view;
    }

    /**
     * @param $view
     * @param null|Controller $controller
     * @return mixed
     */
    public static function initVars($view, $controller = null)
    {
        $request = static::detectRequest($controller);
        if (method_exists($view, 'setRequest')) {
            if ($request instanceof Request) {
                $view->setRequest($request);
            }
        }

        $view->set('controller', $controller->getName());
        $view->set('action', $controller->getAction());

        return $view;
    }

    /**
     * @param $view
     * @param void|Controller $controller
     * @return mixed
     */
    public static function populatePath($view, $controller = null)
    {
        $path = ViewPathDetector::for($controller);
        if (is_dir($path)) {
            $view->setBasePath($path);
        }

        return $view;
    }

    /**
     * @param null|Controller $controller
     * @return array|Request|\Nip\Request|string|null
     */
    protected static function detectRequest($controller = null)
    {
        if ($controller instanceof Controller && $controller->hasRequest()) {
            return $controller->getRequest();
        }
        if (function_exists('request') && Container::container()->has('request')) {
            $request = request();
            if ($request instanceof Request) {
                return $request;
            }
        }
        return null;
    }
}