Stu\Module\Twig\TwigPage

Complexity

Total Complexity

8

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	render()	0	7	2
A	__construct()	0	4	1
A	setVar()	0	3	1
A	setTemplate()	0	4	1
A	loadTemplate()	0	7	2
A	isTemplateSet()	0	3	1

<?php

declare(strict_types=1);

namespace Stu\Module\Twig;

use RuntimeException;
use Twig\Environment;
use Twig\TemplateWrapper;

//TODO unit tests
final class TwigPage implements TwigPageInterface
{
    private Environment $environment;

    private ?TemplateWrapper $template = null;

    private bool $isTemplateSet = false;

    /** @var array<mixed> */
    private array $variables = [];

    public function __construct(
        Environment $environment
    ) {
        $this->environment = $environment;
    }

    public function setVar(string $var, mixed $value): void
    {
        $this->variables[$var] = $value;
    }

    public function setTemplate(string $file): void
    {
        $this->loadTemplate($file);
        $this->isTemplateSet = true;
    }

    public function isTemplateSet(): bool
    {
        return $this->isTemplateSet;
    }

    public function render(): string
    {
        if ($this->template === null) {
            throw new RuntimeException('can not render before template loaded');
        }

        return $this->template->render($this->variables);
    }

    private function loadTemplate(string $file): TemplateWrapper
    {
        if ($this->template === null) {

            $this->template = $this->environment->load($file);
        }
        return $this->template;

The expression return $this->template could return the type null which is incompatible with the type-hinted return Twig\TemplateWrapper. Consider adding an additional type-check to rule them out.

    }
}