BladeXCompiler::parseComponentInnerHtml() - Code Metrics - Inspection of "commit" - spatie/laravel-blade-x - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ec7182...2263ae )

by Freek

created 2018-09-28 20:59 UTC

BladeXCompiler::parseComponentInnerHtml() A

↳ Parent: BladeXCompiler

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Spatie\BladeX;

use SimpleXMLElement;

class BladeXCompiler
{
    /** @var \Spatie\BladeX\BladeX */
    protected $bladeX;

    public function __construct(BladeX $bladeX)
    {
        return $this->bladeX = $bladeX;

    }

    public function compile(string $viewContents): string
    {
        foreach ($this->bladeX->getRegisteredComponents() as $componentName => $viewPath) {
            $pattern = '/<\s*' . $componentName . '[^>]*>((.|\n)*?)<\s*\/\s*' . $componentName . '>/m';

            $viewContents = preg_replace_callback($pattern, function (array $regexResult) use ($viewPath) {
                [$componentHtml, $componentInnerHtml] = $regexResult;


                return "@component('{$viewPath}', [{$this->getComponentAttributes($componentHtml)}])"
                    . $this->parseComponentInnerHtml($componentInnerHtml)
                    . '@endcomponent';
            }, $viewContents);
        }

        return $viewContents;
    }

    protected function getComponentAttributes(string $componentHtml): string
    {
        $componentXml = new SimpleXMLElement($componentHtml);

        return collect($componentXml->attributes())
            ->map(function ($value, $attribute) {
                $value = str_replace("'", "\\'", $value);

                return "'{$attribute}' => '{$value}',";
            })->implode('');
    }

    protected function parseComponentInnerHtml(string $componentInnerHtml): string
    {
        $pattern = '/<\s*slot[^>]*name=[\'"](.*)[\'"][^>]*>((.|\n)*?)<\s*\/\s*slot>/m';

        return preg_replace_callback($pattern, function ($result) {
            [$slot, $name, $contents] = $result;


            return "@slot('{$name}'){$contents}@endslot";
        }, $componentInnerHtml);
    }
}


1			<?php
2
3			namespace Spatie\BladeX;
4
5			use SimpleXMLElement;
6
7			class BladeXCompiler
8			{
9			/** @var \Spatie\BladeX\BladeX */
10			protected $bladeX;
11
12			public function __construct(BladeX $bladeX)
13			{
14			return $this->bladeX = $bladeX;
			0 ignored issues – show Bug introduced 2018-09-28 20:26 UTC by Report Bug Copy Issue Report Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct? Loading history...
15			}
16
17			public function compile(string $viewContents): string
18			{
19			foreach ($this->bladeX->getRegisteredComponents() as $componentName => $viewPath) {
20			$pattern = '/<\s' . $componentName . '[^>]>((.\|\n)?)<\s\/\s*' . $componentName . '>/m';
21
22			$viewContents = preg_replace_callback($pattern, function (array $regexResult) use ($viewPath) {
23			[$componentHtml, $componentInnerHtml] = $regexResult;
			0 ignored issues – show Bug introduced 2018-09-28 20:55 UTC by Report Bug Copy Issue Report The variable `$componentHtml` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history... Bug introduced 2018-09-28 21:00 UTC by Report Bug Copy Issue Report The variable `$componentInnerHtml` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
24
25			return "@component('{$viewPath}', [{$this->getComponentAttributes($componentHtml)}])"
26			. $this->parseComponentInnerHtml($componentInnerHtml)
27			. '@endcomponent';
28			}, $viewContents);
29			}
30
31			return $viewContents;
32			}
33
34			protected function getComponentAttributes(string $componentHtml): string
35			{
36			$componentXml = new SimpleXMLElement($componentHtml);
37
38			return collect($componentXml->attributes())
39			->map(function ($value, $attribute) {
40			$value = str_replace("'", "\\'", $value);
41
42			return "'{$attribute}' => '{$value}',";
43			})->implode('');
44			}
45
46			protected function parseComponentInnerHtml(string $componentInnerHtml): string
47			{
48			$pattern = '/<\sslot[^>]name=[\'"](.)[\'"][^>]>((.\|\n)?)<\s\/\s*slot>/m';
49
50			return preg_replace_callback($pattern, function ($result) {
51			[$slot, $name, $contents] = $result;
			0 ignored issues – show Bug introduced 2018-09-28 20:15 UTC by Report Bug Copy Issue Report The variable `$slot` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history... Bug introduced 2018-09-28 20:15 UTC by Report Bug Copy Issue Report The variable `$name` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history... Bug introduced 2018-09-28 20:15 UTC by Report Bug Copy Issue Report The variable `$contents` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
52
53			return "@slot('{$name}'){$contents}@endslot";
54			}, $componentInnerHtml);
55			}
56			}
57

spatie / laravel-blade-x

Push — master ( ec7182...2263ae )

BladeXCompiler::parseComponentInnerHtml() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like