Passed
Push — develop ( dc5bbb...4fca3e )
by Andrew
18:41
created

VitePluginService   A

Complexity

Total Complexity 6

Size/Duplication

Total Lines 50
Duplicated Lines 0 %

Importance

Changes 10
Bugs 1 Features 0
Metric Value
eloc 20
c 10
b 1
f 0
dl 0
loc 50
rs 10
wmc 6

1 Method

Rating   Name   Duplication   Size   Complexity  
A init() 0 26 6
1
<?php
2
/**
3
 * Vite plugin for Craft CMS 3.x
4
 *
5
 * Allows the use of the Vite.js next generation frontend tooling with Craft CMS
6
 *
7
 * @link      https://nystudio107.com
0 ignored issues
show
Coding Style introduced by
The tag in position 1 should be the @copyright tag
Loading history...
8
 * @copyright Copyright (c) 2021 nystudio107
0 ignored issues
show
Coding Style introduced by
@copyright tag must contain a year and the name of the copyright holder
Loading history...
9
 */
0 ignored issues
show
Coding Style introduced by
PHP version not specified
Loading history...
Coding Style introduced by
Missing @category tag in file comment
Loading history...
Coding Style introduced by
Missing @package tag in file comment
Loading history...
Coding Style introduced by
Missing @author tag in file comment
Loading history...
Coding Style introduced by
Missing @license tag in file comment
Loading history...
10
11
namespace nystudio107\pluginvite\services;
12
13
use Craft;
14
use craft\helpers\App;
15
use nystudio107\pluginvite\helpers\FileHelper;
16
17
/**
0 ignored issues
show
Coding Style introduced by
Missing short description in doc comment
Loading history...
18
 * @author    nystudio107
0 ignored issues
show
Coding Style introduced by
The tag in position 1 should be the @package tag
Loading history...
Coding Style introduced by
Content of the @author tag must be in the form "Display Name <[email protected]>"
Loading history...
Coding Style introduced by
Tag value for @author tag indented incorrectly; expected 2 spaces but found 4
Loading history...
19
 * @package   Vite
0 ignored issues
show
Coding Style introduced by
Tag value for @package tag indented incorrectly; expected 1 spaces but found 3
Loading history...
20
 * @since     1.0.0
0 ignored issues
show
Coding Style introduced by
The tag in position 3 should be the @author tag
Loading history...
Coding Style introduced by
Tag value for @since tag indented incorrectly; expected 3 spaces but found 5
Loading history...
21
 */
0 ignored issues
show
Coding Style introduced by
Missing @link tag in class comment
Loading history...
Coding Style introduced by
Missing @license tag in class comment
Loading history...
Coding Style introduced by
Missing @category tag in class comment
Loading history...
22
class VitePluginService extends ViteService
23
{
24
    // Constants
25
    // =========================================================================
26
27
    const MANIFEST_FILE_NAME = 'manifest.json';
28
29
    // Public Properties
30
    // =========================================================================
31
32
    /**
0 ignored issues
show
Coding Style introduced by
Missing short description in doc comment
Loading history...
33
     * @var string AssetBundle class name to get the published URLs from
34
     */
35
    public $assetClass;
36
37
    /**
0 ignored issues
show
Coding Style introduced by
Missing short description in doc comment
Loading history...
38
     * @var string The environment variable to look for in order to enable the devServer; the value doesn't matter,
39
     *              it just needs to exist
40
     */
41
    public $pluginDevServerEnvVar = 'VITE_PLUGIN_DEVSERVER';
42
43
    /**
0 ignored issues
show
Coding Style introduced by
Missing short description in doc comment
Loading history...
44
     * @inheritDoc
45
     */
0 ignored issues
show
Coding Style introduced by
Missing @return tag in function comment
Loading history...
46
    public function init()
47
    {
48
        // See if the $pluginDevServerEnvVar env var exists, and if not, don't run off of the dev server
49
        $useDevServer = (bool)App::env($this->pluginDevServerEnvVar);
50
        if ($useDevServer === false) {
51
            $this->useDevServer = false;
52
        }
53
        parent::init();
54
        // If we're in a plugin, make sure the caches are unique
55
        if ($this->assetClass) {
56
            $this->cacheKeySuffix = $this->assetClass;
57
        }
58
        $this->invalidateCaches();
59
        // If we have no asset bundle class, or the dev server is running, don't swap in our `/cpresources/` paths
60
        if (!$this->assetClass || $this->devServerRunning()) {
61
            return;
62
        }
63
        // Map the $manifestPath and $serverPublic to the hashed `/cpresources/` path & URL for our AssetBundle
64
        $bundle = new $this->assetClass();
65
        $baseAssetsUrl = Craft::$app->assetManager->getPublishedUrl(
66
            $bundle->sourcePath,
67
            true
0 ignored issues
show
Unused Code introduced by
The call to yii\web\AssetManager::getPublishedUrl() has too many arguments starting with true. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

67
        /** @scrutinizer ignore-call */ 
68
        $baseAssetsUrl = Craft::$app->assetManager->getPublishedUrl(

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. Please note the @ignore annotation hint above.

Loading history...
68
        );
69
        $this->manifestPath = FileHelper::createUrl($bundle->sourcePath, self::MANIFEST_FILE_NAME);
70
        if ($baseAssetsUrl !== false) {
71
            $this->serverPublic = $baseAssetsUrl;
72
        }
73
    }
74
}
75