cotonti.com : Шаблонизатор

Macik

Cum, 10 Tem 2009 02:57:43 -0000

# Sergeich : Macik, по поводу дополнения, собственно вы делаете обратную работу, одним из приоритетов в развитии котонти было вынесенеие всего хтмл из пхп-кода, плюс разбиение крупных тегов (несущих большой кусок хтмл) на мелкие составляющие.

Видимо я не до конца донес мысль. Я не предлягаю переносить ХТМЛ в код, отнюдь. Наоборот из кода он пропадает практически полностью, т.к. все Тэги парсятся отдельной библиотекой, и уже она на основе внутренних шаблонов (тоже настраиваемых) создает готовый к работе ХТМЛ-тэг.
Примерно так:
Дизайнер пишет в шаблоне что-то типа:

 {INPUT_PASSWORD}

Программист пишет примерно следующее:

$ctl['Type'] = 'password';
$ctl['Name'] = 'rpassword';
$ctl['Class'] = '_default';
init_control('INPUT_PASSWORD',$ctl);
parse_control('INPUT_PASSWORD');

На выходе в шаблоне получаем готовый тэг:

В любом случае, это лишь касается моего проекта, так сказать мысли в слух, а не предложение или пожелание к движку.

# Sergeich : По поводу начала поста, а ваша конструкция сможет отрабатывать сложные куски хтмл с включением различных тегов движка? и будет ли это проще для дизайнера?

Да это вопрос. Я просто всегда иду со стороны того, что если не хватает функциональности - постарайся ее сделать доступными (читай предлагаемыми) методами и до последнего не залезать в "Core".

Sergeich

Per, 09 Tem 2009 23:33:35 -0000

Macik, по поводу дополнения, собственно вы делаете обратную работу, одним из приоритетов в развитии котонти было вынесенеие всего хтмл из пхп-кода, плюс разбиение крупных тегов (несущих большой кусок хтмл) на мелкие составляющие. Это всё делалось для получения большего контроля за выводом информации посредством tpl. Разумеется это несколько усложнило шаблоны (хотя, скорее не усложнило, а увеличило кол-во чистого хтмл в шаблонах), но сильно подняло гибкость.

По поводу начала поста, а ваша конструкция сможет отрабатывать сложные куски хтмл с включением различных тегов движка? и будет ли это проще для дизайнера?

Macik

Per, 09 Tem 2009 22:26:29 -0000

В общем согласен.
Хотя логика в блочном виде как сейчас не менее громоздка чем callback-функции.
Отдельного плагина на каждый случай не потребуется - достаточно описать класс с минимальным и достаточным набором callback функций.
Верстальщику вообще практически ничего объяснять не надо - надо показать 1 пример и все будет понятно.

Вот конкретика и примеры:
Вместо вот этого


Описание страницы пустое.

Можно будет писать так:

{PAGE_DESC|IS_NULL('Описание страницы пустое.')}

Все что нужно для этого - создать в системе дочерний класс:

class XT extends XTemplate {
  // функция проверки на предмет присвоено ли тэгу значение
  public function IS_NULL ($tag,$str) {
    // $tag - тело тэга с уже присвоеным значением
    // $str - строка, которую необходимо вывести если тег пустой
    if ($tag=='') {return $str;} else {return $tag;}
  }
}

Аналогичным образом можно реализовать любое другое условие.
Из плюсов такого подхода:
1. Мы не трогаем исходный класс;
2. Никаких отдельных плагинов - один раз описываем в системе класс и прописываем необходимую функциональность;
3. Экономим на размерах шаблона и соответственно меньше шансов допустить в шаблоне ошибку/опечатку.

Как дополнение:

В своем проекте пытаюсь реализовать "прозрачную" (более абстрактную) для верстальщиков систему интерфейса.

Сейчас по сути в шаблонах огромное количество HTML кода, и при том программно зависимого, как например:

...

в таком случае дизайнер/верстальщик должен знать что указать в параметрах action, mothod, name, и т.п.

Идея в том, что бы абстрагироваться от этого. Дизайнер указывает в шаблоне только ключевые Тэги:


Форма регистрации:
{FORM_LOGIN}


Введите свои данные:
Login: {INPUT_USER_LOG}
Password: {INPUT_USER_PASS}
{BUTTON_LOGIN_SEND}
{SOME_TEXT}

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

Trustmaster

Pzt, 06 Tem 2009 14:48:27 -0000

1. а. В целом согласен. А в частности бывают тонкости с отображением отдельных элементов, которые проще реализовать нехитрой логикой в шаблонах, чем делать хак ядра или писать отдельный плагин. Главное меру знать.
1. б. XTemplate развивается очень пассивно. Выше вероятность, что мы перепишем XTemplate, чем то, что выйдет новая версия с важными патчами, которые на придётся включать. Последнее не так уж страшно.
1. в. Опять же, callback-функции надо будет писать в виде специальных плагинов. И верстальщику непросто объяснить, что такое callback-функция.

Macik

Paz, 05 Tem 2009 08:46:45 -0000

Наткнулся на обсуждение Xtemplate и решил оживить топик.

1. Не поддерживаю вашу инициативу по внесению логики в шаблоны в таком виде, по следующим причинам:
а. Логика должна быть в программной части, визуальная часть в шаблонах.
б. XTemplate сторонняя разработка и не есть гут ее "строгать" напильником, тем более, что в случае ее обновления будет трудно совмещать одно с другим.
в. Почти все (а может и все) можно реализовать за счет Callback функций - встроенного в XTemplate механизма. Причем без громоздких блочных IF-ENDIF конструкций в шаблоне.

2. В своем проекте наткнулся на баг XTemplate - при попытке переопределния tag_start_delim, tag_end_delim
перестают работать callback функции. Видимо где-то косяк с правилами и сочетанием RegExp. Но разобраться в коде не осилил.

Безотносительно написанного - спасибо вам за вашу работу. И развитие проекта.

medar

Çrş, 25 Mar 2009 17:59:03 -0000

dervan, dayver, супер, молодцы. :)
У меня, к сожалению, сейчас аврал на работе, времени для cotonti сейчас нет. Надеюсь скоро вернуться.

Предложенный вариант xtemplate, мне кажется, можно коммитить.

По поводу вложенности условий - сложно сказать, нужны ли они. Это будет провоцировать людей на программинг в шаблонах, а это все-таки лучше делать в плагинах.

По поводу eval(). В принципе, это дыра в безопасности, так как ушлый сторонний скинмейкер может разместить в условии кусок вредоносного кода. По хорошему надо бы от него совсем отказаться, сделав парсер условных выражений..

dervan

Çrş, 25 Mar 2009 07:24:54 -0000

# Trustmaster : Ещё бы сделали ... разбор условий без eval'а

Хорошая мысль. :)

Пожалуй, совсем от варианта с eval() отказываться не надо - может потребоваться сложное условие. Но для простейших вариантов, когда в IF проверяется значение переменной bool или int, несложно сделать вариант без eval().

Мы с Dayver'ом, когда обсуждали разные варианты условий для IF (с вызовом функцищ, с использованием $GLOBALS для передачи аргументов функцищам), решили по возможности использовать в шаблонах для IF'ов не код PHP, а сформированные специально для шаблонов переменные bool. Именно такой подход использовал Dayver в выложенных им файлах, которые демонстрируют использование IF'ов в шаблонах.

Если придерживаться такого подхода и использовать в условиях IF шаблона только значения переменных bool или int, то эти условия будут проверяться без вызова eval().

Предлагаем улучшенный вариант XTemplate, проверяющий простые условия без eval(): xtemplate.class.php_57.zip
Остальные условия, которые не сводятся к простой проверке значения bool или int, как и прежде будут обрабатываться вызовом eval().

Все условия в выложенных Dayver'ом файлах этим вариантом XTemplate будут проверяться без eval(), т.е. будут обработаны быстрее.

Пример.

# Sergeich : В шаблоне page.tpl мне нужно разделять теги запятой (или любым другим разделителем), сейчас я это реализовал так:
	{PAGE_TAGS_ROW_TAG},
Проблема в том, что после последнего тега также вставляется запятая (разделитель), что есть не красиво. Как бы убрать эту лишнюю запятую?

Если решать эту задачу, исходя из существующего кода плагина tags, получится вот что.

# dervan :
		
		{PAGE_TAGS_ROW_TAG},
		
Только вот есть сомнения, правильно ли это - раз уж убирать HTML из кода, IMHO тогда не следует злоупотреблять кодом в шаблонах. А тут при каждом проходе парсера будет вызываться функцища count(). По-хорошему, в plugins/tags/tags.page.php надо добавить флажок, чтобы IF определял по нему последний проход парсера в блоке PAGE_TAGS_ROW.

Правильнее будет сформировать в plugins/tags/tags.page.php переменную для шаблона:

	$tags_cntdwn = count($tags);
	if($tags_cntdwn > 0)
	{
		foreach($tags as $tag)
		{
			$tags_cntdwn--;

и использовать в шаблоне простое условие:

		
		{PAGE_TAGS_ROW_TAG},

При таком решении условие IF будет проверяться без eval(), и шаблон будет распарсен быстрее.

Trustmaster

Pzt, 23 Mar 2009 12:57:06 -0000

Ух, молодцы! Ещё бы сделали вложенные условия, парсер без многострочных регулярок, разбор условий без eval'а и оптимизировали использование памяти - я бы вам памятник поставил сразу

dervan

Pzt, 23 Mar 2009 05:48:28 -0000

esclkm, больших изменений мы не делали - было улучшено то, что сделал medar. Это он говорил про оптимизацию, но будут ли это частичные изменения, или же предстоит серьёзный рефакторинг - сказать сложно, мы с Dayver'ом этот вопрос не рассматривали.

esclkm

Pzt, 23 Mar 2009 05:00:46 -0000

вау ребята! medar и dervan - вы потрясающие парни! молодцы! а что то я слышал по поводу оптимизации самого шаблонизатора - то как с этим обстоят дела?

Dayver

Pzt, 23 Mar 2009 03:47:28 -0000

А для полноты информации об изменениях и заявленом применении на практике привожу два админовских пхп файлах из которых венесен xhtml

 $pl)
	{
		include_once($cfg['plugins_dir'].'/'.$pl['pl_code'].'/'.$pl['pl_file'].'.php');
	}
}

$sys['inc'] = (empty($m)) ? 'admin.home' : "admin.$m";
$sys['inc'] = (empty($s)) ? $sys['inc'] : $sys['inc'].".$s";
$sys['inc'] = $cfg['system_dir'].'/core/admin/'.$sys['inc'].'.inc.php';

if(!file_exists($sys['inc']))
{
	sed_die();
}

$allow_img['0']['0'] = "";
$allow_img['1']['0'] = "";
$allow_img['0']['1'] = "";
$allow_img['1']['1'] = "";

$lincif_conf = sed_auth('admin', 'a', 'A');
$lincif_page = sed_auth('page', 'any', 'A');
$lincif_user = sed_auth('users', 'a', 'A');

$t = new XTemplate(sed_skinfile('admin.inc', false, true));
$t -> assign(array(
	"ADMINMENU_URL" => sed_url('admin'),
	"ADMINMENU_CONF_URL" => sed_url('admin', "m=config"),
	"ADMINMENU_PAGE_URL" => sed_url('admin', "m=page"),
	"ADMINMENU_FORUMS_URL" => sed_url('admin', "m=forums"),
	"ADMINMENU_USERS_URL" => sed_url('admin', "m=users"),
	"ADMINMENU_PLUG_URL" => sed_url('admin', "m=plug"),
	"ADMINMENU_TOOLS_URL" => sed_url('admin', "m=tools"),
	"ADMINMENU_TRASHCAN_URL" => sed_url('admin', "m=trashcan"),
	"ADMINMENU_OTHER_URL" => sed_url('admin', "m=other")
));
$t -> parse("ADMINMENU");
$adminmenu = $t -> text("ADMINMENU");

require_once($sys['inc']);
$adminhelp = (empty($adminhelp)) ? $L['None'] : $adminhelp;

$title_tags[] = array('{ADMIN}');
$title_tags[] = array('%1$s');
$title_data = array($L['Administration']);
$out['subtitle'] = sed_title('{ADMIN}', $title_tags, $title_data);

require_once($cfg['system_dir'].'/header.php');

$t = new XTemplate(sed_skinfile('admin', false, true));

$t->assign(array(
	"ADMIN_TITLE" => sed_build_adminsection($adminpath),
	"ADMIN_SUBTITLE" => $adminsubtitle,
	"ADMIN_MENU" => $adminmenu,
	"ADMIN_MAIN" => $adminmain,
	"ADMIN_HELP" => $adminhelp,
));

/* === Hook for the plugins === */
$extp = sed_getextplugins('admin.tags');
if(is_array($extp))
{
	foreach($extp as $k => $pl)
	{
		include_once($cfg['plugins_dir'].'/'.$pl['pl_code'].'/'.$pl['pl_file'].'.php');
	}
}
/* ===== */

$t->parse("MAIN");
$t->out("MAIN");

require_once($cfg['system_dir'].'/footer.php');

?>

его тплка




	
		
{PHP.L.Home}
	
	
		
		
{PHP.L.Configuration}
		
		
{PHP.L.Configuration}
		
	
	
		
		
{PHP.L.Pages}
		
		
{PHP.L.Pages}
		
	
	
		
		
{PHP.L.Forums}
		
		
{PHP.L.Forums}
		
	
	
		
		
{PHP.L.Users}
		
		
{PHP.L.Users}
		
	
	
		
		
{PHP.L.Plugins}
		
		
{PHP.L.Plugins}
		
	
	
		
		
{PHP.L.Tools}
		
		
{PHP.L.Tools}
		
	
	
		
		
{PHP.L.Trashcan}
		
		
{PHP.L.Trashcan}
		
	
	
		
{PHP.L.Other}

 0) ? true : false;
    $lincif_rightsmode = ($authentries[$row['ct_code']] > 0) ? true : false;
	$cfgcode = "disable_".$row['ct_code'];//Непнятна а зачем?

	$t -> assign(array(
		"ADMIN_OTHER_CT_CODE" => $row['ct_code'],
		"ADMIN_OTHER_CT_TITLE_LOC" => (empty($L["core_".$row['ct_code']])) ? $row['ct_title'] : $L["core_".$row['ct_code']],
		"ADMIN_OTHER_CT_CODE_URL" => sed_url('admin', "m=".$row['ct_code']),
		"ADMIN_OTHER_RIGHTS" => ($authentries[$row['ct_code']] > 0) ? sed_url('admin', "m=rightsbyitem&ic=".$row['ct_code']."&io=a") : '#',
		"ADMIN_OTHER_CONFIG" => ($cfgentries[$row['ct_code']] > 0) ? sed_url('admin', "m=config&n=edit&o=core&p=".$row['ct_code']) : '#'
	));
	$t -> parse("OTHER.OTHER_ROW");
}

$lincif_conf = sed_auth('admin', 'a', 'A');
$lincif_user = sed_auth('users', 'a', 'A');

$t -> assign(array(
	"ADMIN_OTHER_URL_CACHE" => sed_url('admin', "m=cache"),
	"ADMIN_OTHER_URL_BBCODE" => sed_url('admin', "m=bbcode"),
	"ADMIN_OTHER_URL_URLS" => sed_url('admin', "m=urls"),
	"ADMIN_OTHER_URL_BANLIST" => sed_url('admin', "m=banlist"),
	"ADMIN_OTHER_URL_HITS" => sed_url('admin', "m=hits"),
	"ADMIN_OTHER_URL_REFERS" => sed_url('admin', "m=referers"),
	"ADMIN_OTHER_URL_LOG" => sed_url('admin', "m=log"),
	"ADMIN_OTHER_URL_INFOS" => sed_url('admin', "m=infos")
));
$t -> parse("OTHER");
$adminmain = $t -> text("OTHER");

?>

его тпл




	{PHP.L.Modules} {PHP.L.adm_clicktoedit}
	{PHP.L.Rights}
	{PHP.L.Configuration}



	
	
		{ADMIN_OTHER_CT_TITLE_LOC}
	
		{ADMIN_OTHER_CT_TITLE_LOC}
	
	
	
	
		
	
		 
	
	
	
		
	
		 
	



	
		
		{PHP.L.adm_internalcache}
		
		{PHP.L.adm_internalcache}
		
	


	
		
		{PHP.L.adm_bbcodes}
		
		{PHP.L.adm_bbcodes}
		
	


	
		
		{PHP.L.adm_urls}
		
		{PHP.L.adm_urls}
		
	


	
		
		{PHP.L.Banlist}
		
		{PHP.L.Banlist}
		
	


	
		{PHP.L.Hits}
	


	
		
		{PHP.L.Referers}
		
		{PHP.L.Referers}
		
	


	
		
		{PHP.L.adm_log}
		
		{PHP.L.adm_log}
		
	


	
		
		{PHP.L.adm_infos}
		
		{PHP.L.adm_infos}

Эти два файла так сказать наиболее показательны в плане применения логики в шаблонизаторе потому код остальных переделаных файлов приводить не буду. Посему просьба к медару потестить скорость так сказать переделаного шаблонизатора с реальным примером применения. Если не возникнет замечаний то и шаблонизатор и части переделаной админки пойдут в транк.[/][/][/]

Sergeich

Pzt, 23 Mar 2009 03:37:40 -0000

Ну вы звери :). Вариант с мне понравился. Это значительно проще и понятнее для неподготовленного юзера (типа меня). Будем пробовать. :)

dervan

Pzt, 23 Mar 2009 02:50:33 -0000

Мы с Dayver'ом представляем результаты практического применения XTemplate с IF'ами - из опыта работы Dayver'а по выносу HTML из админки.

Предлагаем для тестирования доработанный XTemplate: xtemplate.class.php_57.zip

Просьба к medar'у - посмотреть внесённые изменения, может будут какие замечания. Существенных изменений не вносилось, все исправления - простые.

1.
Использование переменных bool приводило к ошибке.

Пример.
Переменная в коде PHP:

$is_test = false;

Условие в шаблоне:

not is_test

В результате - ошибка:

Parse error: syntax error, unexpected ')' in mysite.com/system/xtemplate.class.php(781) : eval()'d code on line 1

Причина - в XTemplate не было поддержки bool, они не нужны в шаблонах. Это мы поправили, тут вроде обсуждать нечего - bool в условиях использоваться будут.

2.
Исходя из практики, кроме короткого варианта с IF нужен ещё и вариант с ELSE. Такой вариант добавлен, сейчас в шаблоне возможны такие условия:

['then' content]
['then' content]['else' content]

Реализация получилась простой - в коде medar'а исправлено 2 строки, и добавлена одна необязательная (в стиле уже сделанного им кода).

Немного усложнился regex, но в целом применение такой конструкции должно давать выигрыш во времени обработки.

Примеры из этого топика.

# medar :


{PAGE_TITLE|str_replace('жуй', '', %s)} ц-ц-ц как нехорошо


{PAGE_TITLE}

# dervan :


вы - гость


вы - зарегистрированный посетитель

В каждом из этих примеров парсится 2 условия IF, т.е. regex обрабатывается 2 раза. Кроме того, в 1-м примере в условиях IF дважды вызывается функцища strpos(), а во 2-м примере - дважды парсится тег {PHP.usr.id}. Если переписать эти примеры с ELSE, они будут работать быстрее, хотя regex для реализации ELSE чуть сложнее.


{PAGE_TITLE|str_replace('жуй', '', %s)} ц-ц-ц как нехорошо

{PAGE_TITLE}


вы - гость

вы - зарегистрированный посетитель

В шаблоне для админки, который переделал Dayver, подобных конструкций - подавляющее большинство, поэтому исходя из практики выгоднее использовать вариант с ELSE.

Предлагаем оставить в XTemplate вариант с поддержкой ELSE.

3.
Проблема из-за копирования глобальных переменных методом scan_globals(), в результате чего в шаблоне не видны глобальные переменные, присвоенные после создания объекта XTemplate. На практике это создаёт большие сложности.

Пример.
Эту переменную $lincif_conf можно использовать в шаблоне:

$lincif_conf = sed_auth('admin', 'a', 'A');
$t = new XTemplate(sed_skinfile('admin.inc', false, true));

А эту переменную $lincif_conf использовать в шаблоне невозможно:

$t = new XTemplate(sed_skinfile('admin.inc', false, true));
$lincif_conf = sed_auth('admin', 'a', 'A');

Для решения проблемы удалось легко реализовать то, о чём говорил medar:

# medar : Надо будет решить эту проблему по-другому - вообще отказаться от scan_globals(), а при парсинге тэгов {PHP.xxx} обращаться непосредственно к $GLOBALS.

Из метода scan_globals() убрано копирование $GLOBALS:

	public function scan_globals () {
		if ($this->force_globals && ini_get('auto_globals_jit') == true) {
			$tmp = $_SERVER;
			$tmp = $_ENV;
			$tmp = $_REQUEST;
			unset($tmp);
		}
	}

А в методе parse() присваивается указатель:

$this->vars['PHP'] =& $GLOBALS;

и уничтожается перед выходом из parse(). Т.е. $GLOBALS теперь не копируется, и в то же время нет опасности уничтожения $GLOBALS через указатель $this->vars['PHP'] - этот указатель теперь не существует вне метода parse().

Возникавшая из-за работы через указатель напрямую с $GLOBALS накладка, которая проявлялась в копировании в пустые строки умолчательных значений, определённых define(), в методе parse() была исправлена. Сообщение об этой накладке:

# dervan : К тому же есть нюансы при работе метода parse(). Например, такой код в конце system/header.php, плюс тег {PHP.TEST_DEFINE} в header.tpl:
define('TEST_DEFINE', 'test_define');
$TEST_DEFINE = '';

$t->parse("HEADER");
$t->out("HEADER");

echo $TEST_DEFINE, '
';
В случае оригинального scan_globals() глобальная переменная $TEST_DEFINE останется пустой строкой (значение будет присвоено её копии), а с переделанным scan_globals() после работы парсера $TEST_DEFINE получит значение 'test_define'.

Из метода get_vars_value() аналогичный код был удалён - пустую строку, которая пойдёт на проверку в IF, не надо подменять умолчательным значением.

Остальной код в parse() был внимательно изучен, других накладок из-за работы через указатель напрямую с $GLOBALS не обнаружено.

Предлагаем оставить в XTemplate этот вариант без копирования $GLOBALS.

Dayver

Pzt, 16 Mar 2009 23:44:30 -0000

medar а как с логикой типа else? Не получилось реализовать? Или уже есть такое(если есть то моно примерчик?)?

medar

Pzt, 16 Mar 2009 04:31:09 -0000

Да, я на самом деле зря тут перестраховался, на скорости это серьезно не скажется, сделаю попозже удаление мусора.

dervan

Paz, 15 Mar 2009 18:43:27 -0000

# Ratibor : Осталось только доработать, чтоб все лишнее вырезалось если условие совпадает.

А у medar'а для этого всё готово.

				$short_tpl_block = $ifs[2][$i];
...
				if(!$if_result)
				{
					// delete this IF block with all content
					$copy = str_replace($full_tpl_block, "", $copy);
				}
				else
				{
					// delete only IF construction
					// ? delete or not - speed ?
				}

Осталось выяснить, как сильно упадёт скорость работы парсера, если коментарий

					// ? delete or not - speed ?

заменить на код

					$copy = str_replace($full_tpl_block, $short_tpl_block, $copy);

Delete or not delete? :)

Ratibor

Paz, 15 Mar 2009 14:36:47 -0000

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

dervan

Paz, 15 Mar 2009 06:17:47 -0000

# Sergeich : ... если на сайт зашёл гость - показывается один блок, а если юзер - другой.


вы - гость


вы - зарегистрированный посетитель

Sergeich

Paz, 15 Mar 2009 05:25:16 -0000

Приведите пример пожалуйста для случая с зарегеным юзером и гостем. Т.е. если на сайт зашёл гость - показывается один блок, а если юзер - другой.

Ratibor

Paz, 15 Mar 2009 04:06:11 -0000

# medar : Использование.

Итак, у нас в шаблонах стала доступна вот такая вот конструкция:
этот текст будет показан если  является true
- это логическое php-выражение, только вместо переменных используются tpl-тэги (в том числе и глобальные!)

Примеры:

привет, админ !

Попробовал сейчас добавить логику в шаблон.
Работает вроде все нормально, но есть небольшое замечание.

Чтоб было более понятно, объясню на приведенном выше примере:
если имя не admin, т.е. условие не срабатывает, то все нормально,
всмысле в выходном коде это все вырезается,
но если это admin, т.е. условие срабатывает, то в выходном коде мы видим следующее:


привет, админ !

Не порядок, надо бы вырезать вот это:

medar

Sal, 10 Mar 2009 15:28:08 -0000

Я просто жду реакции англоязычных товарищей, но её, походу, не будет.
Приличия соблюдены, сейчас закоммичу.

Dayver

Sal, 10 Mar 2009 04:51:26 -0000

# medar : Логику написали, теперь регулярки поправить внутри и все ок будет.

Тогда может уже закинешь в транк хотя бы с прикрученой логикой(в англ ветке отписался = офф. часть выполнил .... там молчат - знач моно закидывать!)? .... а с регулярками уже как нить опосля!?

dervan

Sal, 10 Mar 2009 04:05:57 -0000

# medar : dervan
Надо будет решить эту проблему по-другому - вообще отказаться от scan_globals(), а при парсинге тэгов {PHP.xxx} обращаться непосредственно к $GLOBALS.

Кстати, medar, если так переделывать, то вот от таких накладок при работе parse() всё равно не избавишься:

# dervan : К тому же есть нюансы при работе метода parse(). Например, такой код в конце system/header.php, плюс тег {PHP.TEST_DEFINE) в header.tpl:
define('TEST_DEFINE', 'test_define');
$TEST_DEFINE = '';

$t->parse("HEADER");
$t->out("HEADER");

echo $TEST_DEFINE, '
';
В случае оригинального scan_globals() глобальная переменная $TEST_DEFINE останется пустой строкой (значение будет присвоено её копии), а с переделанным scan_globals() после работы парсера $TEST_DEFINE получит значение 'test_define'.

А действия, подобные

$t->assign('PHP', 'test_assign');

это совершенно идиотские действия, что со старым scan_globals(), что с переделанным - из того же разряда, что и доступ к глобальным переменным через $t->vars. Так что если отказываться от локальной копии $GLOBALS, пожалуй нет смысла совсем убирать scan_globals(), проще его подправить.

medar

Sal, 10 Mar 2009 03:30:59 -0000

ну так может посмотрим пристально на другие шаблонизаторы

не, проще обустроить xtemplate.
Логику написали, теперь регулярки поправить внутри и все ок будет.

esclkm

Sal, 10 Mar 2009 02:55:59 -0000

раз уже на икстеплейте то на икстемплейте.
тем более что у нас появился специалист по xtemplate - поэтому возможно скоро мы увидим medarTemplate

Sergeich

Sal, 10 Mar 2009 01:47:38 -0000

ну так может посмотрим пристально на другие шаблонизаторы. Если всё так не здорово в нашем, то пока не поздно внедрить новый, правильный шаблонизатор :). Пока плагинов мало под кот - это можно будет сделать относительно безболеззненно. написать хороший скрипт миграции и вперёд :).

dervan

Sal, 10 Mar 2009 00:44:57 -0000

Эта гипотеза. Всё то, что я перечислил как доводы против переделки scan_globals(), сам бы ни за что не стал делать - изврат на изврате. :)

medar

Sal, 10 Mar 2009 00:08:49 -0000

Вопрос, есть ли такие. Я верю в здравый смысл плагинописателей.
Если человек копался в коде и знает, что существует $t->vars, то он должен понимать, что брать оттуда данные, а не из например $cfg[] напрямую - это оперировать гланды через задницу. :)[/]

Trustmaster

Sal, 10 Mar 2009 00:08:27 -0000

С чего бы? Через $t->vars обычно вытаскивают локальные переменные, к глобальным обращаются напрямую, иначе очень большая глупость получается.

ЗЫ: я тут глянул еще раз на код XTemplate. Возникает законный вопрос "неужели это всё хоть сколько-нибудь работоспособно"

dervan

Pzt, 09 Mar 2009 23:42:51 -0000

# medar : dervan
Надо будет решить эту проблему по-другому - вообще отказаться от scan_globals(), а при парсинге тэгов {PHP.xxx} обращаться непосредственно к $GLOBALS.

Тогда посыпятся плагины, которые используют $t->vars.

medar

Pzt, 09 Mar 2009 22:38:55 -0000

dervan
Да, наверное, лучше убрать.
Надо будет решить эту проблему по-другому - вообще отказаться от scan_globals(), а при парсинге тэгов {PHP.xxx} обращаться непосредственно к $GLOBALS.

Sergeich

Pzt, 09 Mar 2009 22:18:30 -0000

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

medar

Pzt, 09 Mar 2009 22:13:47 -0000

UPD
Провел тестирование на линуксовом серваке. Цифры стали логичными и предсказуемыми.
Все-таки php под виндой - это странная вещь. :)

Старый xtemplate
Requests per second: 16.30 [#/sec] (mean)
Time per request: 245.377 [ms] (mean)
Time per request: 61.344 [ms] (mean, across all concurrent requests)
Transfer rate: 123.87 [Kbytes/sec] received

Новый xtemplate + хак dervan (здесь и далее) шаблон без логики
Requests per second: 17.47 [#/sec] (mean)
Time per request: 229.007 [ms] (mean)
Time per request: 57.252 [ms] (mean, across all concurrent requests)
Transfer rate: 132.60 [Kbytes/sec] received

Новый xtemplate, шаблон с логикой, 1 условие
Requests per second: 16.78 [#/sec] (mean)
Time per request: 238.446 [ms] (mean)
Time per request: 59.611 [ms] (mean, across all concurrent requests)
Transfer rate: 128.34 [Kbytes/sec] received

Новый xtemplate, шаблон с логикой, 25 условий
Requests per second: 15.60 [#/sec] (mean)
Time per request: 256.485 [ms] (mean)
Time per request: 64.121 [ms] (mean, across all concurrent requests)
Transfer rate: 119.25 [Kbytes/sec] received

dervan

Pzt, 09 Mar 2009 22:08:14 -0000

# medar : Кто-нибудь может прокомментировать, что происходит ? :) Как-то это всё нелогично..

Получается, что по ссылке на $GLOBALS парсинг работает дольше, чем при использовании локальной копии.

================

medar, я извиняюсь - не пойдёт этот хак scan_globals().

Во-первых, я не учёл, что после такой переделки весь $GLOBALS можно убить методом assign(), например так:

$t->assign('PHP', 'test_assign');
$t->assign(array('PHP' => 'test_assign_array'));

Добавлял в конец system/header.php такой код, всё так и есть - $GLOBALS уничтожается:

$t->assign('PHP', 'test_assign');

$t->parse("HEADER");
$t->out("HEADER");

echo '', strip_tags(print_r($GLOBALS, true)), '';

В принципе, можно в методе assign() проверять ключ на значение 'PHP' и не присваивать в этом случае значение. Но $GLOBALS всё равно можно убить напрямую через vars:

$t->vars['PHP'] = 'test_assign';

А vars - public, и менять это нельзя, могут отказаться работать какие-нибудь плагины.

К тому же есть нюансы при работе метода parse(). Например, такой код в конце system/header.php, плюс тег {PHP.TEST_DEFINE) в header.tpl:

define('TEST_DEFINE', 'test_define');
$TEST_DEFINE = '';

$t->parse("HEADER");
$t->out("HEADER");

echo $TEST_DEFINE, '
';

В случае оригинального scan_globals() глобальная переменная $TEST_DEFINE останется пустой строкой (значение будет присвоено её копии), а с переделанным scan_globals() после работы парсера $TEST_DEFINE получит значение 'test_define'.

И вдобавок по твоим тестам с переделанным scan_globals() парсер работает медленнее. Так что в сад. :)

Ещё раз извиняюсь.

medar

Pzt, 09 Mar 2009 15:35:27 -0000

Проапдейтил файл с учетом изменений от dervan.

Бенчмарк на локалхосте, php под виндой.

ab -c 4 -n1000 "http://localhost/svn.cotonti.com/trunk/page.php?id=2"

Новый xtemplate, два логических блока в tpl.
Requests per second: 8.48 [#/sec] (mean)
Time per request: 471.563 [ms] (mean)
Time per request: 117.891 [ms] (mean, across all concurrent requests)
Transfer rate: 64.49 [Kbytes/sec] received

Новый xtemplate, без логических блоков.
Requests per second: 8.81 [#/sec] (mean)
Time per request: 454.188 [ms] (mean)
Time per request: 113.547 [ms] (mean, across all concurrent requests)
Transfer rate: 66.25 [Kbytes/sec] received

Разницы практически нет, ок.

Теперь старый xtemplate
Requests per second: 13.06 [#/sec] (mean)
Time per request: 306.188 [ms] (mean)
Time per request: 76.547 [ms] (mean, across all concurrent requests)
Transfer rate: 99.96 [Kbytes/sec] received

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

Новый xtemplate, но со старой функцией scan_globals() (без хака dervan)
Requests per second: 9.41 [#/sec] (mean)
Time per request: 424.875 [ms] (mean)
Time per request: 106.219 [ms] (mean, across all concurrent requests)
Transfer rate: 70.83 [Kbytes/sec] received

Кто-нибудь может прокомментировать, что происходит ? :) Как-то это всё нелогично..

Надо проштудировать хабр, вроде бы кто-то уже сталкивался с подобной ситуацией.

Dayver

Pzt, 09 Mar 2009 08:42:48 -0000

Афигенно.... если все работает, и без глюков да еще и без тормозов то на днях таки активизирую свою работу по устранению xhtml кода из админских php файлов(если данная фича будет закомичена в транк).....а как покончу с админкой(почти половину уже осилил) и дефолтными плугами(а там остался всего то один плуг поиска) возьмусь за остальные части папочки systems

dervan

Pzt, 09 Mar 2009 08:30:45 -0000

# medar : Да, я тоже хотел сделать ссылкой, но меня насторожил код ниже:
case 'GLOBALS':
// Stop Recursion
break;
Наверное он там не просто так.
...

Это ссылка в $GLOBALS на самого себя - она там ещё с PHP3, когда ещё в языке ссылки не поддерживались. Проверка - чтобы не зацикливаться при копировании.

Про этот хак в XTemplate подробно рассказано здесь.

Sergeich

Pzt, 09 Mar 2009 08:16:52 -0000

Хорошо, бум пробовать, но слегонца пугают меня непрограмёра такие конструкции :)

medar

Pzt, 09 Mar 2009 08:08:42 -0000

Да, я тоже хотел сделать ссылкой, но меня насторожил код ниже:

case 'GLOBALS':
// Stop Recursion
break;

Наверное он там не просто так.

Вроде со ссылкой на глаз все работает.
Но print_r($this->vars); показывает вот что:

Array
(
    [PHP] => Array
        (
            [GLOBALS] => Array
 *RECURSION*

Я такое вижу в первый раз, не знаю, чем это может нам грозить :)

dervan

Pzt, 09 Mar 2009 07:33:20 -0000

# medar : Вообще, когда я увидел ужас, летящий на крыльях ночи в xtemplate в виде массы регулярок и дублирования всех переменных движка, ...

От дублирования $GLOBALS можно избавиться, поправив в XTemplate метод scan_globals():

	public function scan_globals ()
	{
		if ($this->force_globals && ini_get('auto_globals_jit') == true)
		{
			$tmp = $_SERVER;
			$tmp = $_ENV;
			$tmp = $_REQUEST;
			unset($tmp);
		}
		$this->vars['PHP'] =& $GLOBALS;
	}

{PHP.L.Modules} {PHP.L.adm_clicktoedit}	{PHP.L.Rights}	{PHP.L.Configuration}
{ADMIN_OTHER_CT_TITLE_LOC} {ADMIN_OTHER_CT_TITLE_LOC}
{PHP.L.adm_internalcache} {PHP.L.adm_internalcache}
{PHP.L.adm_bbcodes} {PHP.L.adm_bbcodes}
{PHP.L.adm_urls} {PHP.L.adm_urls}
{PHP.L.Banlist} {PHP.L.Banlist}
{PHP.L.Hits}
{PHP.L.Referers} {PHP.L.Referers}
{PHP.L.adm_log} {PHP.L.adm_log}
{PHP.L.adm_infos} {PHP.L.adm_infos}