nystudio107\vite\services\Helper

Complexity

Total Complexity

5

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getCssInlineTags()	0	12	2
A	getCriticalCssTags()	0	26	3

<?php
/**
 * Vite plugin for Craft CMS 3.x
 *
 * Allows the use of the Vite.js next generation frontend tooling with Craft CMS
 *
 * @link      https://nystudio107.com

The tag in position 1 should be the @copyright tag

 * @copyright Copyright (c) 2021 nystudio107

@copyright tag must contain a year and the name of the copyright holder

 */

Missing @author tag in file comment

Missing @license tag in file comment

Missing @package tag in file comment

Missing @category tag in file comment

PHP version not specified

namespace nystudio107\vite\services;

use nystudio107\vite\Vite;
use nystudio107\vite\models\Settings;

use nystudio107\pluginvite\helpers\FileHelper;

use Craft;
use craft\base\Component;
use craft\helpers\Html;

use Twig\Error\LoaderError;

/**

Missing short description in doc comment

 * @author    nystudio107

Tag value for @author tag indented incorrectly; expected 2 spaces but found 4

Content of the @author tag must be in the form "Display Name <username@example.com>"

The tag in position 1 should be the @package tag

 * @package   Vite

Tag value for @package tag indented incorrectly; expected 1 spaces but found 3

 * @since     1.0.5

The tag in position 3 should be the @author tag

Tag value for @since tag indented incorrectly; expected 3 spaces but found 5

 */

Missing @link tag in class comment

Missing @license tag in class comment

Missing @category tag in class comment

class Helper extends Component
{
    /**
     * Returns the Critical CSS file for $template wrapped in <style></style>
     * tags
     *
     * @param null|string $name

Missing parameter comment

     * @param array $attributes additional HTML key/value pair attributes to add to the resulting tag

Expected 7 spaces after parameter type; 1 found

     *
     * @return string
     * @throws LoaderError
     */
    public function getCriticalCssTags($name = null, array $attributes = []): string
    {
        // Resolve the template name
        $template = Craft::$app->getView()->resolveTemplate($name ?? Vite::$templateName ?? '');
        if ($template) {
            /** @var Settings $settings */

Missing short description in doc comment

The close comment tag must be the only content on the line

The open comment tag must be the only content on the line

            $settings = Vite::$plugin->getSettings();
            $name = FileHelper::createUrl(
                pathinfo($template, PATHINFO_DIRNAME),

It seems like pathinfo($template, nystudio107\vite\services\PATHINFO_DIRNAME) can also be of type array; however, parameter $url of nystudio107\pluginvite\helpers\FileHelper::createUrl() does only seem to accept string, maybe add an additional type check?

                pathinfo($template, PATHINFO_FILENAME)

It seems like pathinfo($template, nystudio107\vite\services\PATHINFO_FILENAME) can also be of type array; however, parameter $path of nystudio107\pluginvite\helpers\FileHelper::createUrl() does only seem to accept string, maybe add an additional type check?

            );
            $dirPrefix = 'templates/';
            if (defined('CRAFT_TEMPLATES_PATH')) {
                $dirPrefix = CRAFT_TEMPLATES_PATH;
            }
            $name = strstr($name, $dirPrefix);
            $name = (string)str_replace($dirPrefix, '', $name);
            $path = FileHelper::createUrl(
                    $settings->criticalPath,

This line of the multi-line function call does not seem to be indented correctly. Expected 16 spaces, but found 20.

                    $name

This line of the multi-line function call does not seem to be indented correctly. Expected 16 spaces, but found 20.

                ) . $settings->criticalSuffix;

This line of the multi-line function call does not seem to be indented correctly. Expected 12 spaces, but found 16.

            return $this->getCssInlineTags($path, $attributes);
        }

        return '';
    }

    /**

Missing short description in doc comment

     * @param string $path

Missing parameter comment

     * @param array $attributes additional HTML key/value pair attributes to add to the resulting tag

Expected 2 spaces after parameter type; 1 found

     *
     * @return string
     */
    public function getCssInlineTags(string $path, array $attributes = []): string
    {
        /** @var Settings $settings */

The close comment tag must be the only content on the line

Missing short description in doc comment

The open comment tag must be the only content on the line

        $settings = Vite::$plugin->getSettings();
        $result = FileHelper::fetch($path, null, $settings->cacheKeySuffix);
        if ($result) {
            $config = [];

            return Html::style($result, array_merge($config, $attributes));

It seems like $result can also be of type array; however, parameter $content of yii\helpers\BaseHtml::style() does only seem to accept string, maybe add an additional type check?

        }

        return '';
    }
}