pjpawel\Magis\Template

Complexity

Total Complexity

8

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getTemplateContent()	0	6	2
A	__construct()	0	3	1
A	getTemplatePath()	0	3	1
A	loadTemplateContent()	0	10	3
A	resolveTemplatePath()	0	3	1

<?php

namespace pjpawel\Magis;

use pjpawel\Magis\Exception\TemplateException;

/**
 * @author Paweł Podgórski <pawel.jan.podgorski@gmail.com>
 */
class Template
{

    private string $templatePath;
    private ?string $content = null;

    public function __construct(string $templatePath)
    {
        $this->templatePath = $templatePath;
    }

    /**
     * @return string
     */
    public function getTemplatePath(): string
    {
        return $this->templatePath;
    }

    /**
     * @return string
     * @throws TemplateException
     */
    public function getTemplateContent(): string
    {
        if ($this->content === null) {
            $this->loadTemplateContent();
        }
        return $this->content;

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

    }

    /**
     * @return void
     * @throws TemplateException
     */
    private function loadTemplateContent(): void
    {
        if (!is_file($this->templatePath)) {
            throw new TemplateException('There is no template with given name');
        }
        $content = file_get_contents($this->templatePath);
        if ($content === false) {
            throw new TemplateException('Cannot get content of template');
        }
        $this->content = $content;
    }

    public static function resolveTemplatePath(string $directory, string $name): string
    {
        return $directory . $name;
    }

}