Sketch\Tpl\InheritanceTag

Complexity

Total Complexity

8

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	extends()	0	5	2
A	handle()	0	6	3
A	replace()	0	9	2

<?php

namespace Sketch\Tpl;

class InheritanceTag extends Tag

The type Sketch\Tpl\Tag was not found. Maybe you did not declare it correctly or list all dependencies?

{
    private $blocks = [];
    private $patternBlock = '/{\s?block \'?"?([\w]+)"?\'?\s?}(.*?){\s?\/block\s?}/is';
    private $patternExtends = '/{\s?extends \'?"?(.*?)"?\'?\s?}/is';

    public function __construct()
    {
        parent::__construct();
        $this->extends();
    }

    private function extends(): void
    {
        if (preg_match($this->patternExtends, self::$content, $match)) {
            self::$content = Content::getContent($match[1], self::$config);

The property content does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            $this->replace();
        }
    }

    private function replace(): void
    {
        foreach ($this->blocks as $key => $value) {
            $pattern = '/{\s?block \'?"?' . $key . '"?\'?\s?}(.*?){\s?\/block\s?}/is';
            self::$content = preg_replace($pattern, $value, self::$content);

The property content does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        }

        self::$content = preg_replace('/{\s?block \'?"?[\w]+"?\'?\s?}/is', '', self::$content);
        self::$content = preg_replace('/{\s?\/block\s?}/is', '', self::$content);
    }

    public function handle(): void
    {
        if (preg_match_all($this->patternBlock, self::$content, $matches, PREG_SET_ORDER)) {
            for ($i = 0; $i < count($matches); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

                $this->blocks[$matches[$i][1]] = $matches[$i][2];
                self::$content = str_replace($this->blocks[$matches[$i][1]], '', self::$content);

The property content does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            };
        }
    }
}