DevPontes\View\Assets

Complexity

Total Complexity

15

Size/Duplication

Total Lines	113
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	makeStyle()	0	12	5
A	setScriptPath()	0	4	1
A	makeScript()	0	12	5
A	__construct()	0	4	1
A	getScripts()	0	3	1
A	getStyles()	0	3	1
A	setStylePath()	0	4	1

<?php

namespace DevPontes\View;

/**
 * Class DevPontes Assets
 *
 * @author Moises Pontes <sesiom_assis@hotmail.com>
 * @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;

The expression return $this returns the type DevPontes\View\Assets which is incompatible with the documented return type DevPontes\View\View.

    }

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

The expression return $this returns the type DevPontes\View\Assets which is incompatible with the documented return type DevPontes\View\View.

    }

    /**
     * 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) {

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.

            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) {

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.

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

        return $this;
    }
}