PHPMV\VueJSComponent

Complexity

Total Complexity

16

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Test Coverage

Coverage

75%

Importance

Changes	2
Bugs	0	Features	0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	2
A	generateObject()	0	4	1
A	extends()	0	2	1
A	addProps()	0	2	1
A	generateGlobalScript()	0	5	1
A	getName()	0	2	1
A	importTemplate()	0	2	1
A	onDeactivated()	0	2	1
A	addTemplate()	0	2	1
A	generateFile()	0	10	4
A	onActivated()	0	2	1
A	getVarName()	0	2	1

<?php

namespace PHPMV;

use PHPMV\js\JavascriptUtils;
use PHPMV\utils\JsUtils;

class VueJSComponent extends AbstractVueJS {
    protected string $name;
    protected array $props;
    protected array $template;
    protected array $extends;
    protected ?string $varName;

    public function __construct(string $name,string $varName = null) {
        parent::__construct();
        $this->name = $name;
        $this->props = [];
        $this->extends = [];
        $this->template = [];
        if(!$varName){
            $varName = JsUtils::kebabToPascal($name);
        }
        $this->varName = $varName;
    }

    public function extends(VueJSComponent $component):void {
        $this->extends['extends'] = $component->getVarName();
    }

    public function generateObject():string {
        $script = JavascriptUtils::arrayToJsObject($this->props + $this->components + $this->filters + $this->extends + $this->mixins + $this->data + $this->computeds + $this->watchers + $this->hooks + $this->methods + $this->template);
        $script = JsUtils::cleanJSONFunctions($script);
        return $script;
    }

    public function generateGlobalScript():string {
        $script = "Vue.component('".$this->name."',";
        $script .= $this->generateObject();
        $script .= ")";
        return $script;
    }

    public function generateFile(bool $inVariable = false, bool $global = false):void {
        $script = ($inVariable) ? JsUtils::declareVariable("const", $this->varName, $this->generateGlobalScript(), false) : $this->generateGlobalScript().";";

It seems like $this->varName can also be of type null; however, parameter $name of PHPMV\utils\JsUtils::declareVariable() does only seem to accept string, maybe add an additional type check?

        if (!$global){
            \file_put_contents($this->name.".js",$script);
        }
        elseif(file_exists("components.js")){
            \file_put_contents("components.js",PHP_EOL . $script,FILE_APPEND);
        }
        else{
            \file_put_contents("components.js",$script);
        }
    }

    public function addProps(string ...$props):void {
        $this->props["props"] = $props;
    }

    public function addTemplate(string $template):void {
        $this->template["template"] = $template;
    }

    public function importTemplate(string $template):void {
        $this->template["template"] = "'".\str_replace(["\n","\r","\t"]," ",(\file_get_contents($template.'.html',true))."'");
    }

    public function onActivated(string $body):void {
        $this->addHook("activated", $body);
    }

    public function onDeactivated(string $body):void {
        $this->addHook("deactivated", $body);
    }

    public function getName():string {
        return $this->name;
    }

    public function getVarName():string {
        return $this->varName;

The expression return $this->varName could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }
}