MoveJsToFooter::execute() - Code Metrics - hryvinskyi/magento2-deferjs - Measure and Improve Code Quality continuously with Scrutinizer

MoveJsToFooter::execute() B
last analyzed 2020-11-03 19:32 UTC

↳ Parent: MoveJsToFooter

Complexity

Conditions	10
Paths	32

Size

Total Lines	66
Code Lines	41

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	10
eloc	41
c	1
b	0
f	0
nc	32
nop	1
dl	0
loc	66
rs	7.6666

How to fix Long Method Complexity

<?php
/**
 * Copyright (c) 2019. Volodymyr Hryvinskyi.  All rights reserved.
 * @author: <mailto:[email protected]>
 * @github: <https://github.com/hryvinskyi>
 */

declare(strict_types=1);

namespace Hryvinskyi\DeferJs\Model;

use Hryvinskyi\Base\Helper\ArrayHelper;
use Hryvinskyi\Base\Helper\Json;
use Hryvinskyi\DeferJs\Helper\Config;
use Hryvinskyi\DeferJs\Model\Minify\MinifyJsInterface;
use Hryvinskyi\DeferJs\Model\PassesValidator\ValidateSkipper;
use Magento\Framework\App\Response\Http;

/**
 * Class MoveJsToFooter
 */
class MoveJsToFooter implements MoveJsToFooterInterface
{
    /**
     * @var Config
     */
    private $config;

    /**
     * @var MinifyJsInterface
     */
    private $minifyJs;

    /**
     * @var ValidateSkipper
     */
    private $validateSkipper;

    /**
     * MoveJsToFooter constructor.
     *
     * @param Config $config
     * @param MinifyJsInterface $minifyJs
     * @param ValidateSkipper $validateSkipper
     */
    public function __construct(
        Config $config,
        MinifyJsInterface $minifyJs,
        ValidateSkipper $validateSkipper
    ) {
        $this->config = $config;
        $this->minifyJs = $minifyJs;
        $this->validateSkipper = $validateSkipper;
    }

    /**
     * @param Http $http
     *
     * @return void
     */
    public function execute(Http $http)
    {
        $scripts = [];
        $html = $http->getBody();

        $jsons = [];
        $scriptStart = '<script';
        $scriptEnd = '</script>';

        $start = 0;
        $i = 0;
        while (($start = stripos($html, $scriptStart, $start)) !== false) {
            $end = stripos($html, $scriptEnd, $start);

            if ($end === false) {
                break;
            }

            $len = $end + strlen($scriptEnd) - $start;
            $script = substr($html, $start, $len);

            if (
                $this->config->isOptimizeXMagentoInitScripts() &&
                strpos($script, 'text/x-magento-init') !== false
            ) {
                $jsons[] = strip_tags($script);
                $html = str_replace($script, '', $html);
                continue;
            }

            if ($this->validateSkipper->execute($script, $http)) {
                $start++;
                continue;
            }

            $html = str_replace($script, '', $html);
            $scripts[] = $script;

            $i++;
        }

        if ($this->config->isMinifyBodyScript()) {
            $scripts = $this->minifyJs->execute($scripts);
        }

        $merged = [];
        foreach ($jsons as $json) {
            $json = Json::decode($json);
            $merged = ArrayHelper::merge($merged, $json);
        }

        if (count($merged) > 0) {
            $merged = '<script type=text/x-magento-init>' . Json::encode($merged) . '</script>';
        } else {
            $merged = '';
        }

        $scripts = implode('', $scripts);

        if ($endBody = stripos($html, '</body>')) {
            $html = substr($html, 0, $endBody) . $merged . $scripts . substr($html, $endBody);
        } else {
            $html .= $merged . $scripts;
        }

        $http->setBody($html);
    }
}

1			<?php
2			/**
3			* Copyright (c) 2019. Volodymyr Hryvinskyi. All rights reserved.
4			* @author: <mailto:[email protected]>
5			* @github: <https://github.com/hryvinskyi>
6			*/
7
8			declare(strict_types=1);
9
10			namespace Hryvinskyi\DeferJs\Model;
11
12			use Hryvinskyi\Base\Helper\ArrayHelper;
13			use Hryvinskyi\Base\Helper\Json;
14			use Hryvinskyi\DeferJs\Helper\Config;
15			use Hryvinskyi\DeferJs\Model\Minify\MinifyJsInterface;
16			use Hryvinskyi\DeferJs\Model\PassesValidator\ValidateSkipper;
17			use Magento\Framework\App\Response\Http;
18
19			/**
20			* Class MoveJsToFooter
21			*/
22			class MoveJsToFooter implements MoveJsToFooterInterface
23			{
24			/**
25			* @var Config
26			*/
27			private $config;
28
29			/**
30			* @var MinifyJsInterface
31			*/
32			private $minifyJs;
33
34			/**
35			* @var ValidateSkipper
36			*/
37			private $validateSkipper;
38
39			/**
40			* MoveJsToFooter constructor.
41			*
42			* @param Config $config
43			* @param MinifyJsInterface $minifyJs
44			* @param ValidateSkipper $validateSkipper
45			*/
46			public function __construct(
47			Config $config,
48			MinifyJsInterface $minifyJs,
49			ValidateSkipper $validateSkipper
50			) {
51			$this->config = $config;
52			$this->minifyJs = $minifyJs;
53			$this->validateSkipper = $validateSkipper;
54			}
55
56			/**
57			* @param Http $http
58			*
59			* @return void
60			*/
61			public function execute(Http $http)
62			{
63			$scripts = [];
64			$html = $http->getBody();
65
66			$jsons = [];
67			$scriptStart = '<script';
68			$scriptEnd = '</script>';
69
70			$start = 0;
71			$i = 0;
72			while (($start = stripos($html, $scriptStart, $start)) !== false) {
73			$end = stripos($html, $scriptEnd, $start);
74
75			if ($end === false) {
76			break;
77			}
78
79			$len = $end + strlen($scriptEnd) - $start;
80			$script = substr($html, $start, $len);
81
82			if (
83			$this->config->isOptimizeXMagentoInitScripts() &&
84			strpos($script, 'text/x-magento-init') !== false
85			) {
86			$jsons[] = strip_tags($script);
87			$html = str_replace($script, '', $html);
88			continue;
89			}
90
91			if ($this->validateSkipper->execute($script, $http)) {
92			$start++;
93			continue;
94			}
95
96			$html = str_replace($script, '', $html);
97			$scripts[] = $script;
98
99			$i++;
100			}
101
102			if ($this->config->isMinifyBodyScript()) {
103			$scripts = $this->minifyJs->execute($scripts);
104			}
105
106			$merged = [];
107			foreach ($jsons as $json) {
108			$json = Json::decode($json);
109			$merged = ArrayHelper::merge($merged, $json);
110			}
111
112			if (count($merged) > 0) {
113			$merged = '<script type=text/x-magento-init>' . Json::encode($merged) . '</script>';
114			} else {
115			$merged = '';
116			}
117
118			$scripts = implode('', $scripts);
119
120			if ($endBody = stripos($html, '</body>')) {
121			$html = substr($html, 0, $endBody) . $merged . $scripts . substr($html, $endBody);
122			} else {
123			$html .= $merged . $scripts;
124			}
125
126			$http->setBody($html);
127			}
128			}

hryvinskyi / magento2-deferjs

MoveJsToFooter::execute() B last analyzed 2020-11-03 19:32 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

MoveJsToFooter::execute() B
last analyzed 2020-11-03 19:32 UTC