VueManager::addVue() - Code Metrics - Inspection of "feat VueManager & kebabToPascal, declareVariable i..." - phpMv/php-vuejs - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — main ( d7f1b4...976a1c )

by Guillaume

created 2021-05-31 21:41 UTC

VueManager::addVue() A

↳ Parent: VueManager

Complexity

Conditions	1
Paths	1

Size

Total Lines	2
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	1
c	1
b	0
f	0
nc	1
nop	1
dl	0
loc	2
rs	10

<?php
namespace PHPMV;

use PHPMV\js\JavascriptUtils;

class VueManager{

    private static ?VueManager $instance = null;
    protected array $imports;
    protected array $vues;

    protected function __construct() {
        $this->imports = [];
        $this->vues = [];
    }

    public static function getInstance():VueManager {
        if (!isset(self::$instance)) {
            self::$instance = new VueManager();
        }
        return self::$instance;

    }

    public function addImport($import):void {
        $this->imports[] = $import;
    }

    public function addVue(VueJS $vue):void {
        $this->vues[] = $vue;
    }

    public function __toString():string {
        $script = implode("\n",$this->imports);
        $script .= implode("\n",$this->vues);
        return JavascriptUtils::wrapScript($script);
    }
}

1			<?php
2			namespace PHPMV;
3
4			use PHPMV\js\JavascriptUtils;
5
6			class VueManager{
7
8			private static ?VueManager $instance = null;
9			protected array $imports;
10			protected array $vues;
11
12			protected function __construct() {
13			$this->imports = [];
14			$this->vues = [];
15			}
16
17			public static function getInstance():VueManager {
18			if (!isset(self::$instance)) {
19			self::$instance = new VueManager();
20			}
21			return self::$instance;
			0 ignored issues – show Bug Best Practice introduced 2021-05-31 21:44 UTC by Report Bug Copy Issue Report The expression `return self::instance` could return the type `null` which is incompatible with the type-hinted return `PHPMV\VueManager`. Consider adding an additional type-check to rule them out. Loading history...
22			}
23
24			public function addImport($import):void {
25			$this->imports[] = $import;
26			}
27
28			public function addVue(VueJS $vue):void {
29			$this->vues[] = $vue;
30			}
31
32			public function __toString():string {
33			$script = implode("\n",$this->imports);
34			$script .= implode("\n",$this->vues);
35			return JavascriptUtils::wrapScript($script);
36			}
37			}

phpMv / php-vuejs

Push — main ( d7f1b4...976a1c )

VueManager::addVue() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like