Assets::makeScript() - Code Metrics - Inspection of "feat: nova classe Assets e novos metodos insert e..." - moisespontes/view - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 04e35b...6957a7 )

by Moises

created 2024-04-09 19:42 UTC

Assets::makeScript() A

↳ Parent: Assets

Complexity

Conditions	5
Paths	8

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	6
c	1
b	0
f	0
dl	0
loc	12
rs	9.6111
cc	5
nc	8
nop	1

<?php

namespace DevPontes\View;

/**
 * Class DevPontes Assets
 *
 * @author Moises Pontes <[email protected]>
 * @package DevPontes\View
 */
class Assets
{
    /** @var string */
    private $styles;

    /** @var string */
    private $scripts;

    /** @var string */
    private $stylePath;

    /** @var string */
    private $scriptPath;

    /** @var string */
    private $source;

    /** @var bool */
    private $cache;

    /**
     * Assets constructor.
     *
     * @param string $src path to resources folder
     * @param boolean $cache [optional] cache on/off, defaults true
     * @example - new Assets('assets', false);
     */
    public function __construct(string $src, bool $cache = true)
    {
        $this->source = $src;
        $this->cache = $cache;
    }

    /**
     * @return string
     */
    public function getStyles(): string
    {
        return $this->styles;
    }

    /**
     * @return string
     */
    public function getScripts(): string
    {
        return $this->scripts;
    }

    /**
     * Set path for styles
     *
     * @param string $stylePath
     * @return View
     */
    public function setStylePath(string $stylePath): Assets
    {
        $this->stylePath = $stylePath;
        return $this;

    }

    /**
     * Set path for scripts
     *
     * @param string $scriptPath
     * @return View
     */
    public function setScriptPath(string $scriptPath): Assets
    {
        $this->scriptPath = $scriptPath;
        return $this;

    }

    /**
     * Make CSS styles
     *
     * @param array $css array of styles
     * @return Assets
     * @example - $a->makeStyle($css['global', ...]);
     */
    public function makeStyle(array $css): Assets
    {
        $version = $this->cache ? "" : "?v=" . time();
        $stylePath = empty($this->stylePath) ? "css" : $this->stylePath;

        if ($css) {

            foreach ($css as $style) {
                $this->styles .= "<link href='{$this->source}/{$stylePath}/{$style}.css{$version}' rel='stylesheet'>\n    ";
            }
        }

        return $this;
    }

    /**
     * Make JS scripts
     *
     * @param array $js array of scripts
     * @return Assets
     * @example - $a->makeScript($js['global', ...]);
     */
    public function makeScript(array $js): Assets
    {
        $version = $this->cache ? "" : "?v=" . time();
        $scriptPath = empty($this->scriptPath) ? "js" : $this->scriptPath;

        if ($js) {

            foreach ($js as $script) {
                $this->scripts .= "<script src='{$this->source}/{$scriptPath}/{$script}.js{$version}'></script>\n    ";
            }
        }

        return $this;
    }
}


1			<?php
2
3			namespace DevPontes\View;
4
5			/**
6			* Class DevPontes Assets
7			*
8			* @author Moises Pontes <[email protected]>
9			* @package DevPontes\View
10			*/
11			class Assets
12			{
13			/** @var string */
14			private $styles;
15
16			/** @var string */
17			private $scripts;
18
19			/** @var string */
20			private $stylePath;
21
22			/** @var string */
23			private $scriptPath;
24
25			/** @var string */
26			private $source;
27
28			/** @var bool */
29			private $cache;
30
31			/**
32			* Assets constructor.
33			*
34			* @param string $src path to resources folder
35			* @param boolean $cache [optional] cache on/off, defaults true
36			* @example - new Assets('assets', false);
37			*/
38			public function __construct(string $src, bool $cache = true)
39			{
40			$this->source = $src;
41			$this->cache = $cache;
42			}
43
44			/**
45			* @return string
46			*/
47			public function getStyles(): string
48			{
49			return $this->styles;
50			}
51
52			/**
53			* @return string
54			*/
55			public function getScripts(): string
56			{
57			return $this->scripts;
58			}
59
60			/**
61			* Set path for styles
62			*
63			* @param string $stylePath
64			* @return View
65			*/
66			public function setStylePath(string $stylePath): Assets
67			{
68			$this->stylePath = $stylePath;
69			return $this;
			0 ignored issues – show Bug Best Practice introduced 2024-04-09 19:50 UTC by Report Bug Copy Issue Report The expression `return $this` returns the type `DevPontes\View\Assets` which is incompatible with the documented return type `DevPontes\View\View`. Loading history...
70			}
71
72			/**
73			* Set path for scripts
74			*
75			* @param string $scriptPath
76			* @return View
77			*/
78			public function setScriptPath(string $scriptPath): Assets
79			{
80			$this->scriptPath = $scriptPath;
81			return $this;
			0 ignored issues – show Bug Best Practice introduced 2024-04-09 19:50 UTC by Report Bug Copy Issue Report The expression `return $this` returns the type `DevPontes\View\Assets` which is incompatible with the documented return type `DevPontes\View\View`. Loading history...
82			}
83
84			/**
85			* Make CSS styles
86			*
87			* @param array $css array of styles
88			* @return Assets
89			* @example - $a->makeStyle($css['global', ...]);
90			*/
91			public function makeStyle(array $css): Assets
92			{
93			$version = $this->cache ? "" : "?v=" . time();
94			$stylePath = empty($this->stylePath) ? "css" : $this->stylePath;
95
96			if ($css) {
			0 ignored issues – show Bug Best Practice introduced 2024-04-09 19:50 UTC by Report Bug Copy Issue Report The expression `$css` 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 check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
97			foreach ($css as $style) {
98			$this->styles .= "<link href='{$this->source}/{$stylePath}/{$style}.css{$version}' rel='stylesheet'>\n ";
99			}
100			}
101
102			return $this;
103			}
104
105			/**
106			* Make JS scripts
107			*
108			* @param array $js array of scripts
109			* @return Assets
110			* @example - $a->makeScript($js['global', ...]);
111			*/
112			public function makeScript(array $js): Assets
113			{
114			$version = $this->cache ? "" : "?v=" . time();
115			$scriptPath = empty($this->scriptPath) ? "js" : $this->scriptPath;
116
117			if ($js) {
			0 ignored issues – show Bug Best Practice introduced 2024-04-09 19:50 UTC by Report Bug Copy Issue Report The expression `$js` 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 check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
118			foreach ($js as $script) {
119			$this->scripts .= "<script src='{$this->source}/{$scriptPath}/{$script}.js{$version}'></script>\n ";
120			}
121			}
122
123			return $this;
124			}
125			}
126

moisespontes / view

Push — main ( 04e35b...6957a7 )

Assets::makeScript() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like