TemplateDispatcher::setContentType() - Code Metrics - Inspection of "refactor" - jetfirephp/routing - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 4c92de...3b4c18 )

by Sinnarasa

created 2017-07-16 12:42 UTC

TemplateDispatcher::setContentType() A

↳ Parent: TemplateDispatcher

Complexity

Conditions	2
Paths	2

Size

Total Lines	6
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	6
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	4
nc	2
nop	1

<?php

namespace JetFire\Routing\Dispatcher;

use JetFire\Routing\Router;

/**
 * Class TemplateDispatcher
 * @package JetFire\Routing\Dispatcher
 */
class TemplateDispatcher implements DispatcherInterface
{

    /**
     * @var Router
     */
    private $router;

    /**
     * @var array
     */
    protected $types = [
        'json' => 'application/json',
        'xml' => 'application/xml',
        'txt' => 'text/plain',
        'html' => 'text/html'
    ];

    /**
     * @param Router $router
     */
    public function __construct(Router $router)
    {
        $this->router = $router;
    }

    /**
     * @description call template file
     */
    public function call()
    {
        $this->setContentType($this->router->route->getTarget('extension'));
        if (isset($this->router->route->getTarget('callback')[$this->router->route->getTarget('extension')])) {
            $this->router->response->setContent(call_user_func_array($this->router->route->getTarget('callback')[$this->router->route->getTarget('extension')], [$this->router->route]));
        } else {
            ob_start();
            if (isset($this->router->route->getTarget()['data'])) extract($this->router->route->getTarget('data'));

            if (isset($this->router->route->getParams()['data'])) extract($this->router->route->getParams()['data']);
            require($this->router->route->getTarget('template'));
            $this->router->response->setContent(ob_get_clean());
        }
    }

    /**
     * @param $extension
     */
    public function setContentType($extension)
    {
        isset($this->types[$extension])
            ? $this->router->response->setHeaders(['Content-Type' => $this->types[$extension]])
            : $this->router->response->setHeaders(['Content-Type' => $this->types['html']]);
    }

}


1			<?php
2
3			namespace JetFire\Routing\Dispatcher;
4
5			use JetFire\Routing\Router;
6
7			/**
8			* Class TemplateDispatcher
9			* @package JetFire\Routing\Dispatcher
10			*/
11			class TemplateDispatcher implements DispatcherInterface
12			{
13
14			/**
15			* @var Router
16			*/
17			private $router;
18
19			/**
20			* @var array
21			*/
22			protected $types = [
23			'json' => 'application/json',
24			'xml' => 'application/xml',
25			'txt' => 'text/plain',
26			'html' => 'text/html'
27			];
28
29			/**
30			* @param Router $router
31			*/
32			public function __construct(Router $router)
33			{
34			$this->router = $router;
35			}
36
37			/**
38			* @description call template file
39			*/
40			public function call()
41			{
42			$this->setContentType($this->router->route->getTarget('extension'));
43			if (isset($this->router->route->getTarget('callback')[$this->router->route->getTarget('extension')])) {
44			$this->router->response->setContent(call_user_func_array($this->router->route->getTarget('callback')[$this->router->route->getTarget('extension')], [$this->router->route]));
45			} else {
46			ob_start();
47			if (isset($this->router->route->getTarget()['data'])) extract($this->router->route->getTarget('data'));
			0 ignored issues – show Bug introduced 2017-07-16 12:46 UTC by Report Bug Copy Issue Report `$this->router->route->getTarget('data')` cannot be passed to `extract()` as the parameter `$var_array` expects a reference. Loading history...
48			if (isset($this->router->route->getParams()['data'])) extract($this->router->route->getParams()['data']);
49			require($this->router->route->getTarget('template'));
50			$this->router->response->setContent(ob_get_clean());
51			}
52			}
53
54			/**
55			* @param $extension
56			*/
57			public function setContentType($extension)
58			{
59			isset($this->types[$extension])
60			? $this->router->response->setHeaders(['Content-Type' => $this->types[$extension]])
61			: $this->router->response->setHeaders(['Content-Type' => $this->types['html']]);
62			}
63
64			}
65

jetfirephp / routing

Push — master ( 4c92de...3b4c18 )

TemplateDispatcher::setContentType() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like