DLTemplate

Complexity

Total Complexity

105

Size/Duplication

Total Lines	526
Duplicated Lines	0 %

Importance

Changes	11
Bugs	4	Features	0

20 Methods

Rating	Name	Duplication	Size	Complexity
A	getInstance()	0	8	2
A	__construct()	0	3	1
A	setTemplateExtension()	0	12	2
A	setTemplatePath()	0	15	3
A	__wakeup()	0	2	1
A	__clone()	0	2	1
C	parseDocumentSource()	0	25	8
A	getTemplateData()	0	7	1
A	cleanPHx()	0	8	2
A	renameKeyArr()	0	3	1
A	toPlaceholders()	0	10	2
A	setTemplateData()	0	7	2
C	parseChunk()	0	31	15
A	setPHxPlaceholders()	0	10	4
D	renderDoc()	0	34	10
F	getChunk()	0	93	36
A	createPHx()	0	7	2
A	getTemplate()	0	12	3
B	getTwig()	0	17	5
A	getBlade()	0	16	4

<?php

include_once(MODX_BASE_PATH . 'assets/lib/APIHelpers.class.php');

use EvolutionCMS\Core as DocumentParser;

The type EvolutionCMS\Core was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 * Class DLTemplate
 */
class DLTemplate
{
    /**
     * Объект DocumentParser - основной класс MODX
     * @var DocumentParser $modx
     * @access protected
     */
    protected $modx = null;

    /**
     * @var DLTemplate cached reference to singleton instance
     */
    protected static $instance;

    protected $templatePath = 'assets/templates/';

    protected $templateExtension = 'html';

    /**
     * @var null|Twig_Environment twig object

The type Twig_Environment was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    protected $twig = null;

    /**
     * @var null|Jenssegers\Blade\Blade blade object

The type Jenssegers\Blade\Blade was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    protected $blade = null;

    protected $twigEnabled = false;

    protected $bladeEnabled = false;

    protected $templateData = array();

    public $phx = null;

    /**
     * gets the instance via lazy initialization (created on first usage)
     *
     * @return self
     */
    public static function getInstance(DocumentParser $modx)
    {

        if (null === self::$instance) {
            self::$instance = new self($modx);
        }

        return self::$instance;
    }

    /**
     * is not allowed to call from outside: private!
     *
     */
    private function __construct(DocumentParser $modx)
    {
        $this->modx = $modx;
    }

    /**
     * prevent the instance from being cloned
     *
     * @return void
     */
    private function __clone()
    {
    }

    /**
     * prevent from being unserialized
     *
     * @return void
     */
    private function __wakeup()
    {
    }

    /**
     * Задает относительный путь к папке с шаблонами
     *
     * @param $path
     * @param bool $supRoot
     * @return $this
     */
    public function setTemplatePath($path, $supRoot = false)
    {
        $path = trim($path);
        if ($supRoot === false) {
            $path = preg_replace(array(
                '/\.*[\/|\\\]/i',
                '/[\/|\\\]+/i'
            ), array('/', '/'), $path);
        }

        if (!empty($path)) {

There must be a single space after a NOT operator; 0 found

            $this->templatePath = $path;
        }

        return $this;
    }

    /**
     * Задает расширение файла с шаблоном
     *
     * @param $ext
     * @return $this
     */
    public function setTemplateExtension($ext)
    {
        $ext = preg_replace(array(
            '/\.*[\/|\\\]/i',
            '/[\/|\\\]+/i'
        ), array('/', '/'), trim($ext, ". \t\n\r\0\x0B"));

        if (!empty($ext)) {

There must be a single space after a NOT operator; 0 found

            $this->templateExtension = $ext;
        }

        return $this;
    }

    /**
     * Additional data for external templates
     *
     * @param array $data
     * @return $this
     */
    public function setTemplateData($data = array())
    {
        if (is_array($data)) {

The condition is_array($data) is always true.

            $this->templateData = $data;
        }

        return $this;
    }

    /**
     * @param array $data
     * @return array
     */
    public function getTemplateData($data = array())
    {
        $plh = $this->templateData;
        $plh['data'] = $data;
        $plh['modx'] = $this->modx;

        return $plh;
    }

    /**
     * Сохранение данных в массив плейсхолдеров
     *
     * @param mixed $data данные
     * @param int $set устанавливать ли глобальнй плейсхолдер MODX
     * @param string $key ключ локального плейсхолдера
     * @param string $prefix префикс для ключей массива
     * @return string
     */
    public function toPlaceholders($data, $set = 0, $key = 'contentPlaceholder', $prefix = '')
    {
        $out = '';
        if ($set != 0) {
            $this->modx->toPlaceholder($key, $data, $prefix);
        } else {
            $out = $data;
        }

        return $out;
    }

    /**
     * refactor $modx->getChunk();
     *
     * @param string $name Template: chunk name || @CODE: template || @FILE: file with template
     * @return string html template with placeholders without data
     */
    public function getChunk($name)

Function's nesting level (5) exceeds 3; consider refactoring the function

    {
        $tpl = '';
        $this->twigEnabled = (0 === strpos($name, '@T_'));
        $this->bladeEnabled = (0 === strpos($name, '@B_'));
        if ($name != '' && !isset($this->modx->chunkCache[$name])) {

There must be a single space after a NOT operator; 0 found

            $mode = (preg_match(
                '/^((@[A-Z_]+)[:]{0,1})(.*)/Asu',
                trim($name),
                $tmp
            ) && isset($tmp[2], $tmp[3])) ? $tmp[2] : false;
            $subTmp = (isset($tmp[3])) ? trim($tmp[3]) : null;
            if ($this->twigEnabled) {
                $mode = '@' . substr($mode, 3);

It seems like $mode can also be of type false; however, parameter $string of substr() does only seem to accept string, maybe add an additional type check?

            } elseif ($this->bladeEnabled) {
                $mode = '@' . substr($mode, 3);
            }
            switch ($mode) {
                case '@FILE':
                    if ($subTmp != '') {
                        $real = realpath(MODX_BASE_PATH . $this->templatePath);
                        $path = realpath(MODX_BASE_PATH . $this->templatePath . preg_replace(array(
                                '/\.*[\/|\\\]/i',
                                '/[\/|\\\]+/i'
                            ), array('/', '/'), $subTmp) . '.' . $this->templateExtension);
                        if (basename($path, '.' . $this->templateExtension) !== '' &&
                            0 === strpos($path, $real) &&
                            file_exists($path)
                        ) {
                            $tpl = file_get_contents($path);
                        }
                    }
                    break;
                case '@CHUNK':
                    if ($subTmp != '') {
                        $tpl = $this->modx->getChunk($subTmp);
                    }
                    break;
                case '@INLINE':
                case '@TPL':
                case '@CODE':
                    $tpl = $tmp[3]; //without trim
                    break;
                case '@DOCUMENT':
                case '@DOC':
                    switch (true) {
                        case ((int)$subTmp > 0):
                            $tpl = $this->modx->getPageInfo((int)$subTmp, 0, "content");
                            $tpl = isset($tpl['content']) ? $tpl['content'] : '';
                            break;
                        case ((int)$subTmp == 0):
                            $tpl = $this->modx->documentObject['content'];
                            break;
                    }
                    break;
                case '@PLH':
                case '@PLACEHOLDER':
                    if ($subTmp != '') {
                        $tpl = $this->modx->getPlaceholder($subTmp);
                    }
                    break;
                case '@CFG':
                case '@CONFIG':
                case '@OPTIONS':
                    if ($subTmp != '') {
                        $tpl = $this->modx->getConfig($subTmp);
                    }
                    break;
                case '@SNIPPET':
                    if ($subTmp != '') {
                        $tpl = $this->modx->runSnippet($subTmp, $this->modx->event->params);
                    }
                    break;
                case '@RENDERPAGE':
                    $tpl = $this->renderDoc($subTmp, false);

It seems like $subTmp can also be of type string; however, parameter $id of DLTemplate::renderDoc() does only seem to accept integer, maybe add an additional type check?

                    break;
                case '@LOADPAGE':
                    $tpl = $this->renderDoc($subTmp, true);
                    break;
                case '@TEMPLATE':
                    $tpl = $this->getTemplate($subTmp);

It seems like $subTmp can also be of type string; however, parameter $id of DLTemplate::getTemplate() does only seem to accept integer, maybe add an additional type check?

                    break;
                default:
                    $tpl = $this->modx->getChunk($name);
            }
            $this->modx->chunkCache[$name] = $tpl;
        } else {
            if ($name != '') {
                $tpl = $this->modx->getChunk($name);
            }
        }

        return $tpl;
    }

    /**
     * Рендер документа с подстановкой плейсхолдеров и выполнением сниппетов
     *
     * @param int $id ID документа
     * @param bool $events Во время рендера документа стоит ли вызывать события OnLoadWebDocument и OnLoadDocumentObject (внутри метода getDocumentObject).
     * @param mixed $tpl Шаблон с которым необходимо отрендерить документ. Возможные значения:
     *                       null - Использовать шаблон который назначен документу
     *                       int(0-n) - Получить шаблон из базы данных с указанным ID и применить его к документу
     *                       string - Применить шаблон указанный в строке к документу
     * @return string
     *
     * Событие OnLoadWebDocument дополнительно передает параметры:
     *       - с источиком от куда произошел вызов события
     *       - оригинальный экземпляр класса DocumentParser
     */
    public function renderDoc($id, $events = false, $tpl = null)
    {
        $id = (int)$id;
        if ($id <= 0) {
            return '';
        }

        $m = clone $this->modx; //Чтобы была возможность вызывать события
        $m->documentIdentifier = $id;
        $m->documentObject = $m->getDocumentObject('id', (int)$id, $events ? 'prepareResponse' : null);
        if ($m->documentObject['type'] == "reference") {
            if (is_numeric($m->documentObject['content']) && $m->documentObject['content'] > 0) {
                $m->documentObject['content'] = $this->renderDoc($m->documentObject['content'], $events);
            }
        }
        switch (true) {
            case is_integer($tpl):
                $tpl = $this->getTemplate($tpl);
                break;
            case is_string($tpl):
                break;
            case is_null($tpl):
            default:
                $tpl = $this->getTemplate($m->documentObject['template']);
        }
        $m->documentContent = $tpl;
        if ($events) {
            $m->invokeEvent("OnLoadWebDocument", array(
                'source'   => 'DLTemplate',
                'mainModx' => $this->modx,
            ));
        }

        return $this->parseDocumentSource($m->documentContent, $m);
    }

    /**
     * Получить содержимое шаблона с определенным номером
     * @param int $id Номер шаблона
     * @return string HTML код шаблона
     */
    public function getTemplate($id)
    {
        $tpl = null;
        $id = (int)$id;
        if ($id > 0) {
            $tpl = $this->modx->getDatabase()->getValue("SELECT `content` FROM {$this->modx->getDatabase()->getFullTableName("site_templates")} WHERE `id` = {$id}");
        }
        if (is_null($tpl)) {
            $tpl = '[*content*]';
        }

        return $tpl;
    }

    /**
     * refactor $modx->parseChunk();
     *
     * @param string $name Template: chunk name || @CODE: template || @FILE: file with template
     * @param array $data paceholder
     * @param bool $parseDocumentSource render html template via DocumentParser::parseDocumentSource()
     * @return string html template with data without placeholders
     */
    public function parseChunk($name, $data = array(), $parseDocumentSource = false, $disablePHx = false)

Function's nesting level (4) exceeds 3; consider refactoring the function

    {
        $out = $this->getChunk($name);
        switch (true) {
            case $this->twigEnabled && $out !== '' && ($twig = $this->getTwig($name, $out)):

Are you sure the assignment to $twig is correct as $this->getTwig($name, $out) targeting DLTemplate::getTwig() seems to always return null.

                $out = $twig->render(md5($name), $this->getTemplateData($data));

The variable $twig does not seem to be defined for all execution paths leading up to this point.

                break;
            case $this->bladeEnabled && $out !== '' && ($blade = $this->getBlade($name, $out)):

Are you sure the assignment to $blade is correct as $this->getBlade($name, $out) targeting DLTemplate::getBlade() seems to always return null.

                $out = $blade->with($this->getTemplateData($data))->render();

The variable $blade does not seem to be defined for all execution paths leading up to this point.

                break;
            case is_array($data) && ($out != ''):
                if (preg_match("/\[\+[A-Z0-9\.\_\-]+\+\]/is", $out)) {
                    $item = $this->renameKeyArr($data, '[', ']', '+');
                    $out = str_replace(array_keys($item), array_values($item), $out);
                }
                if (!$disablePHx && preg_match("/:([^:=]+)(?:=`(.*?)`(?=:[^:=]+|$))?/is", $out)) {

There must be a single space after a NOT operator; 0 found

                    if (is_null($this->phx) || !($this->phx instanceof DLphx)) {

There must be a single space after a NOT operator; 0 found

                        $this->phx = $this->createPHx(0, 1000);
                    }
                    $this->phx->placeholders = array();
                    $this->setPHxPlaceholders($data);
                    $out = $this->phx->Parse($out);
                    $out = $this->cleanPHx($out);

It seems like $out can also be of type mixed; however, parameter $string of DLTemplate::cleanPHx() does only seem to accept string, maybe add an additional type check?

                }
                break;
        }
        if ($parseDocumentSource) {
            $out = $this->parseDocumentSource($out);
        }

        return $out;
    }

    /**
     *
     * @param string|array $value
     * @param string $key
     * @param string $path
     */
    public function setPHxPlaceholders($value = '', $key = '', $path = '')

This method is not in camel caps format.

    {
        $keypath = !empty($path) ? $path . "." . $key : $key;

There must be a single space after a NOT operator; 0 found

        $this->phx->curPass = 0;
        if (is_array($value)) {
            foreach ($value as $subkey => $subval) {
                $this->setPHxPlaceholders($subval, $subkey, $keypath);
            }
        } else {
            $this->phx->setPHxVariable($keypath, $value);
        }
    }

    /**
     * Return clone of twig
     *
     * @param string $name
     * @param string $tpl
     * @return null
     */
    protected function getTwig($name, $tpl)
    {
        if (is_null($this->twig) && isset($this->modx->twig)) {
            $twig = clone $this->modx->twig;
            $this->twig = $twig;
        } else {
            $twig = $this->twig;
        }
        if ($twig && class_exists('Twig_Loader_Array')) {
            $twig->getLoader()->addLoader(
                new Twig_Loader_Array(array(

The type Twig_Loader_Array was not found. Maybe you did not declare it correctly or list all dependencies?

                    md5($name) => $tpl
                ))
            );
        }

        return $twig;

The expression return $twig also could return the type object|Twig_Environment which is incompatible with the documented return type null.

    }

    /**
     * Return clone of blade
     *
     * @param string $name
     * @param string $tpl
     * @return null
     */
    protected function getBlade($name, $tpl)
    {
        if ($this->blade === null && isset($this->modx->blade)) {
            $blade = clone $this->modx->blade;
            $this->blade = $blade;
        } else {
            $blade = $this->blade;
        }

        $cache = md5($name). '-'. sha1($tpl);

Concat operator must be surrounded by a single space

        $path = $blade->viewPaths . '/~cache/' . $cache . '.blade.php';
        if (! file_exists($path)) {
            file_put_contents($path, $tpl);
        }

        return $blade->make('~cache/' . $cache);

The method make() does not exist on null.

    }

    /**
     *
     * @param string $string
     * @return string
     */
    public function cleanPHx($string)

This method is not in camel caps format.

    {
        preg_match_all('~\[(\+|\*|\()([^:\+\[\]]+)([^\[\]]*?)(\1|\))\]~s', $string, $matches);
        if ($matches[0]) {
            $string = str_replace($matches[0], '', $string);
        }

        return $string;
    }

    /**
     * @param int $debug
     * @param int $maxpass
     * @return DLphx
     */
    public function createPHx($debug = 0, $maxpass = 50)

This method is not in camel caps format.

    {
        if (!class_exists('DLphx')) {

There must be a single space after a NOT operator; 0 found

            include_once(__DIR__ . '/DLphx.class.php');
        }

        return new DLphx($this->modx, $debug, $maxpass);
    }

    /**
     * Переменовывание элементов массива
     *
     * @param array $data массив с данными
     * @param string $prefix префикс ключей
     * @param string $suffix суффикс ключей
     * @param string $sep разделитель суффиксов, префиксов и ключей массива
     * @return array массив с переименованными ключами
     */
    public function renameKeyArr($data, $prefix = '', $suffix = '', $sep = '.')
    {
        return APIhelpers::renameKeyArr($data, $prefix, $suffix, $sep);
    }

    /**
     * @param $out
     * @param DocumentParser|null $modx
     * @return mixed|string
     */
    public function parseDocumentSource($out, $modx = null)
    {
        if (!is_object($modx)) {

There must be a single space after a NOT operator; 0 found

            $modx = $this->modx;
        }
        $minPasses = empty($modx->minParserPasses) ? 2 : $modx->minParserPasses;
        $maxPasses = empty($modx->maxParserPasses) ? 10 : $modx->maxParserPasses;
        $site_status = $modx->getConfig('site_status');
        $modx->config['site_status'] = 0;
        for ($i = 1; $i <= $maxPasses; $i++) {
            $html = $out;
            if (preg_match('/\[\!(.*)\!\]/us', $out)) {
                $out = str_replace(array('[!', '!]'), array('[[', ']]'), $out);
            }
            if ($i <= $minPasses || $out != $html) {
                $out = $modx->parseDocumentSource($out);
            } else {
                break;
            }
        }
        $out = $modx->rewriteUrls($out);
        $out = $this->cleanPHx($out);
        $modx->config['site_status'] = $site_status;

        return $out;
    }
}