Новые функции CMS и их производительность относительно версий PHP

На основании опубликованной статьи: Производительность CMS относительно новых и старых версий PHP, предлагаю детально обсудить возможности новой функции шаблонизатора системы.

Старая версия с внутренним кеширование содержания

if (!function_exists("tpl_func")) {
	function tpl_func() {
		global $blockg, $theme, $conf;
		$arg = func_get_args();
		$lan = array();
		static $argc, $cach, $cont;
		if ($argc != $arg[0] || !isset($cach)) {
			$argc = $arg[0];
			$cont = get_theme_file($argc);
			if ($cont) $cach = create_function("\$arg, \$lan", "global \$blockg, \$theme, \$conf; return \"".addslashes(file_get_contents($cont))."\";");
		}
		return ($cont) ? $cach($arg, $lan) : tpl_warn("warn", sprintf(_ERRORTPL, $arg[0]), "", "", "warn");
	}
}

if (!function_exists('setTemplateBasic')) {
	function setTemplateBasic() {
		global $blockg, $theme, $conf;
		$arg = func_get_args();
		$lan = array();
		static $argc, $cach, $cont;
		if ($argc != $arg[0] || !isset($cach)) {
			$argc = $arg[0];
			$cont = getThemeFile($argc);
			if ($cont) $cach = file_get_contents($cont);
		}
		if ($cont) eval("\$rfl = \"".addslashes($cach)."\";");
		return ($cont) ? stripslashes($rfl) : setTemplateWarning('warn', sprintf(_ERRORTPL, $arg[0]), '', '', 'warn');
	}
}

Решил отказаться от языковой конструкций с использованием функции eval().

Основные причины

1. Связана с безопасностью, в случае реализации модулей или дополнений от сторонних разработчиков. Невнимательность которых может быть очень опасна, поскольку позволяет выполнить произвольный код. Таки образом исключим полностью данную проблему, в этом участке.
   

Смена названия функции - это опять по всей системе лазить функции переписывать? )
setTemplateBasic и setTemplateWarning - длинновато как то стало...

Ну внимательность программеров - это уже дело программеров и к системе отношения не имеет. Как бэ сами виноваты!

ЗЫ
С новым годом всех )

Miksar, смена названий функций неизбежна, плюсов всё таки больше чем минусов. Ну а стиль программирования, это как правило, дело привычки. Долго откладывал этот процесс, но думаю что время пришло.

Я не сторонник революций, поэтому планирую постепенный, максимально безболезненный переход (с сохранением совместимости), на стандарты кодирования PSR.

Вот некоторые из правил стандарта PHP-FIG:

• Файлы с PHP кодом необходимо начинать с тэгов <?php

• В файлах с кодом PHP необходимо использовать только UTF-8 без BOM.

• Константам необходимо давать имена в верхнем регистре с символом подчёркивания в качестве разделителя: CMS_PRODUCT

• Методам/функциям необходимо давать имена в стиле setTemplateBasic (так называемый верблюжий стиль, название записывается без пробелов, каждое слово с большой буквы, кроме первого).

• Классам необходимо давать имена в стиле SetTemplateBasic (название записанное без пробелов, каждое слово в этом названии начинается с большой буквы).

Кому интересны детали, в сети достаточно информации, читайте на сайте первоисточника: Basic Coding Standard

Понятно.
Кстати на счет переменных. Можно же не заморачиваться названиями оных, а выводить как сейчас по стилю: {%arg0%}, {%arg1%},....{%argN%}

Да, но в этом и смысл. Переменные типа: {%arg0%}, {%arg0%} менее понятны чем: {%login%}, {%menu%}.

так оно, но теперь придется переделывать шаблоны тем. Причем не просто переписывать переменные, но менять их названия на понятные.
Например в тех же шаблонах вывода новостей мы имеем много входящих данных в функцию base().
В общем хз конечно. Переход на такие шаблоны больше подходит для версии слаеда 7. Уж больно серьезные изменения для шаблонов в целом.

Да, конечно согласен. Но думаю что для опытного пользователя не составит особого труда переписать переменные в своей теме оформления, за основу или в качестве примера можно будет взять стандартную.

Встречный вопрос, есть ли смысл добавления в index.html переменной {%STYLE%}, для вывода стилей темы?

Думаю есть. Возможны проекты, где они и не требуются. А так мы даем возможность самому выбрать - выводить или нет.

Старые шаблоны сделали не актуальными.