Kotori\Core\View

Complexity

Total Complexity

9

Size/Duplication

Total Lines	120
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	6

Test Coverage

Coverage

13.51%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	assign()	0	5	1
A	need()	0	10	1
A	__construct()	0	10	2
B	display()	0	29	5

<?php
/**
 * Kotori.php
 *
 * A Tiny Model-View-Controller PHP Framework
 *
 * This content is released under the Apache 2 License
 *
 * Copyright (c) 2015-2017 Kotori Technology. All rights reserved.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

/**
 * View Class
 *
 * @package     Kotori
 * @subpackage  Core
 * @author      Kokororin
 * @link        https://kotori.love
 */
namespace Kotori\Core;

use Kotori\Debug\Hook;
use Kotori\Exception\NotFoundException;
use Kotori\Exception\ResponseException;
use Kotori\Traits\ControllerMethodsTrait;

class View
{
    use ControllerMethodsTrait;
    /**
     * Template Direcory
     *
     * @var string
     */
    protected $tplDir;

    /**
     *
     * Template Path
     *
     * @var string
     */
    protected $viewPath;

    /**
     * Variable List
     *
     * @var array
     */
    protected $data = [];

    /**
     * Variable List for TplInclude
     *
     * @var array
     */
    protected $needData;

    /**
     * Class constructor
     *
     * @param  string $tplDir
     */
    public function __construct($tplDir = null)
    {
        if (null == $tplDir) {

It seems like you are loosely comparing $tplDir of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

            $this->tplDir = Container::get('config')->get('app_full_path') . '/views/';
        } else {
            $this->tplDir = $tplDir;
        }

        Hook::listen(__CLASS__);
    }

    /**
     * Set variables for Template
     *
     * @param  string $key
     * @param  mixed  $value
     * @return \Kotori\Core\View
     */
    public function assign($key, $value)
    {
        $this->data[$key] = $value;
        return $this;
    }

    /**
     * Display Output
     *
     * Processes and sends finalized output data to the browser along
     *
     * @param  string $tpl
     * @return void
     *
     * @throws \Kotori\Exception\NotFoundException
     * @throws \Kotori\Exception\ResponseException
     */
    public function display($tpl = '')
    {
        if (Container::get('request')->isCli()) {
            throw new ResponseException('cannot render template in CLI mode');
        }

        if ('' === $tpl) {
            $tpl = Container::get('route')->getController() . '/'
            . Container::get('route')->getAction();
        }

        $this->viewPath = $this->tplDir . $tpl . '.html';
        if (!Helper::isFile($this->viewPath)) {
            throw new NotFoundException('Template is not existed.');
        }

        unset($tpl);
        ob_start();
        extract($this->data, EXTR_OVERWRITE);
        include $this->viewPath;
        $buffer = ob_get_contents();
        ob_get_clean();
        $output = Helper::comment() . preg_replace('|</body>.*?</html>|is', '', $buffer, -1, $count) . Container::get('trace')->showTrace();
        if ($count > 0) {
            $output .= '</body></html>';
        }

        echo $output;
    }

    /**
     * Include Template
     *
     * @param  string $path
     * @param  array  $data
     * @return void
     */
    public function need($path, array $data = [])
    {
        $this->needData = [
            'path' => Container::get('config')->get('app_full_path') . '/views/' . $path . '.html',
            'data' => $data,
        ];
        unset($path, $data);
        extract($this->needData['data']);
        include $this->needData['path'];
    }
}