View::set() - Code Metrics - Inspection of "Removed obsolete code." - canax/view - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 1e45b1...b422dd )

by Mikael

created 2018-08-09 12:50 UTC

View::set() B

↳ Parent: View

Complexity

Conditions	7
Paths	7

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	24.4119

Importance

Changes

Metric	Value
cc	7
nc	7
nop	4
dl	0
loc	40
ccs	7
cts	24
cp	0.2917
crap	24.4119
rs	8.3466
c	0
b	0
f	0

<?php

namespace Anax\View;

use Anax\View\Exception;
use Anax\View\ViewRenderFile;
use Psr\Container\ContainerInterface;

/**
 * A view connected to a template file, supporting Anax DI.
 */
class View
{
    /**
     * @var $template     Template file or array
     * @var $templateData Data to send to template file
     * @var $sortOrder    For sorting views
     * @var $type         Type of view
     */
    private $template;
    private $templateData = [];
    private $sortOrder;
    private $type;



    /**
     * Set values for the view.
     *
     * @param array|string $template the template file, or array
     * @param array        $data     variables to make available to the
     *                               view, default is empty
     * @param integer      $sort     which order to display the views,
     *                               if suitable
     * @param string       $type     which type of view
     *
     * @return self
     */
    public function set($template, $data = [], $sort = 0, $type = "file")
    {
        if (is_array($template)) {
            if (isset($template["callback"])) {
                $type = "callback";
                $this->template = $template;
            } else {
                $this->template = $template["template"];
            }

            $this->sortOrder = $template["sort"] ?? $sort;
            $this->type = $template["type"] ?? $type;

            // Merge data array
            $data1 = $template["data"] ?? [];
            if (empty($data)) {
                $this->templateData = $data1;
            } else if (empty($data1)) {
                $this->templateData = $data;
            } else {
                foreach ($data as $key => $val) {
                    if (is_array($val)) {
                        $data1[$key] = array_merge($data1[$key], $val);
                    } else {
                        $data1[$key] = $val;
                    }
                    $this->templateData = $data1;
                }
            }

            return;
        }

        $this->template     = $template;
        $this->templateData = $data;
        $this->sortOrder    = $sort;
        $this->type         = $type;

        return $this;
    }



    /**
     * Render the view by its type.
     *
     * @param object $di optional with access to the framework resources.
     *
     * @return void
     */
    public function render(ContainerInterface $di = null)
    {
        switch ($this->type) {
            case "file":
                if ($di->has("viewRenderFile")) {
                    $viewRender = $di->get("viewRenderFile");
function someFunction(A $objectMaybe = null)
{
    if ($objectMaybe instanceof A) {
        $objectMaybe->doSomething();
    }
}
                } else {
                    $viewRender = new ViewRenderFile($di);

                    $viewRender->setDI($di);
function notNullable(stdClass $x) { }

// Unsafe
function withoutCheck(stdClass $x = null) {
    notNullable($x);
}

// Safe - Alternative 1: Adding Additional Type-Check
function withCheck(stdClass $x = null) {
    if ($x instanceof stdClass) {
        notNullable($x);
    }
}

// Safe - Alternative 2: Changing Parameter
function withNonNullableParam(stdClass $x) {
    notNullable($x);
}
                }
                $viewRender->render($this->template, $this->templateData);
                break;

            case "callback":
                if (!isset($this->template["callback"]) || !is_callable($this->template["callback"])) {
                    throw new Exception("View missing callback.");
                }

                echo call_user_func($this->template["callback"]);

                break;

            case "string":
                echo $this->template;

                break;

            default:
                throw new Exception("Not a valid template type: {$this->type}");
        }
    }



    /**
     * Give the sort order for this view.
     *
     * @return int
     */
    public function sortOrder()
    {
        return $this->sortOrder;
    }
}


1		<?php
2
3		namespace Anax\View;
4
5		use Anax\View\Exception;
6		use Anax\View\ViewRenderFile;
7		use Psr\Container\ContainerInterface;
8
9		/**
10		* A view connected to a template file, supporting Anax DI.
11		*/
12		class View
13		{
14		/**
15		* @var $template Template file or array
16		* @var $templateData Data to send to template file
17		* @var $sortOrder For sorting views
18		* @var $type Type of view
19		*/
20		private $template;
21		private $templateData = [];
22		private $sortOrder;
23		private $type;
24
25
26
27		/**
28		* Set values for the view.
29		*
30		* @param array\|string $template the template file, or array
31		* @param array $data variables to make available to the
32		* view, default is empty
33		* @param integer $sort which order to display the views,
34		* if suitable
35		* @param string $type which type of view
36		*
37		* @return self
38		*/
39	2	public function set($template, $data = [], $sort = 0, $type = "file")
40		{
41	2	if (is_array($template)) {
42		if (isset($template["callback"])) {
43		$type = "callback";
44		$this->template = $template;
45		} else {
46		$this->template = $template["template"];
47		}
48
49		$this->sortOrder = $template["sort"] ?? $sort;
50		$this->type = $template["type"] ?? $type;
51
52		// Merge data array
53		$data1 = $template["data"] ?? [];
54		if (empty($data)) {
55		$this->templateData = $data1;
56		} else if (empty($data1)) {
57		$this->templateData = $data;
58		} else {
59		foreach ($data as $key => $val) {
60		if (is_array($val)) {
61		$data1[$key] = array_merge($data1[$key], $val);
62		} else {
63		$data1[$key] = $val;
64		}
65		$this->templateData = $data1;
66		}
67		}
68
69		return;
70		}
71
72	2	$this->template = $template;
73	2	$this->templateData = $data;
74	2	$this->sortOrder = $sort;
75	2	$this->type = $type;
76
77	2	return $this;
78		}
79
80
81
82		/**
83		* Render the view by its type.
84		*
85		* @param object $di optional with access to the framework resources.
86		*
87		* @return void
88		*/
89	2	public function render(ContainerInterface $di = null)
90		{
91	2	switch ($this->type) {
92		case "file":
93		if ($di->has("viewRenderFile")) {
94		$viewRender = $di->get("viewRenderFile");
		0 ignored issues – show Bug introduced 2017-09-12 21:06 UTC by Report Bug Copy Issue Report It seems like `$di` is not always an object, but can also be of type `null`. Maybe add an additional type check? If a variable is not always an object, we recommend to add an additional type check to ensure your method call is safe: function someFunction(A $objectMaybe = null) { if ($objectMaybe instanceof A) { $objectMaybe->doSomething(); } } Loading history...
95		} else {
96		$viewRender = new ViewRenderFile($di);
		0 ignored issues – show Unused Code introduced 2018-08-09 12:53 UTC by Report Bug Copy Issue Report The call to `ViewRenderFile::__construct()` has too many arguments starting with `$di`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
97		$viewRender->setDI($di);
		0 ignored issues – show Bug introduced 2018-08-09 12:53 UTC by Report Bug Copy Issue Report It seems like `$di` defined by parameter `$di` on line `89` can be `null`; however, `Anax\Commons\ContainerInjectableTrait::setDI()` does not accept `null`, maybe add an additional type check? It seems like you allow that null is being passed for a parameter, however the function which is called does not seem to accept null. We recommend to add an additional type check (or disallow null for the parameter): function notNullable(stdClass $x) { } // Unsafe function withoutCheck(stdClass $x = null) { notNullable($x); } // Safe - Alternative 1: Adding Additional Type-Check function withCheck(stdClass $x = null) { if ($x instanceof stdClass) { notNullable($x); } } // Safe - Alternative 2: Changing Parameter function withNonNullableParam(stdClass $x) { notNullable($x); } Loading history...
98		}
99		$viewRender->render($this->template, $this->templateData);
100		break;
101
102		case "callback":
103		if (!isset($this->template["callback"]) \|\| !is_callable($this->template["callback"])) {
104		throw new Exception("View missing callback.");
105		}
106
107		echo call_user_func($this->template["callback"]);
108
109		break;
110
111		case "string":
112	1	echo $this->template;
113
114	1	break;
115
116		default:
117	1	throw new Exception("Not a valid template type: {$this->type}");
118		}
119	1	}
120
121
122
123		/**
124		* Give the sort order for this view.
125		*
126		* @return int
127		*/
128		public function sortOrder()
129		{
130		return $this->sortOrder;
131		}
132		}
133

canax / view

Push — master ( 1e45b1...b422dd )

View::set() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like