Сомнительное нововведение

[Sander1 62]

engine/modules/show.short.php

Разве эта строка должна находиться внутри цикла while?

$xfields = xfieldsload();

Я бы так вообще предложил такой вариант, перед while:

	$xfields = xfieldsload();
	foreach ($xfields as $k => $v) {
		if (stripos($tpl->copy_template, '_'.$v[0].']') == false) {
			unset($xfields[$k]);
		}
	}

 

[celsoft 6242]

Надо будет изучить вопрос и посмотреть внимательно. Пока не готов ответить. Взял на заметку.

[celsoft 6242]

Да xfieldsload() в цикле это не хорошо, уберем.

 

Ваш цикл неверен, могут быть проблемы с тегами [ifxfvalue tagname="tagvalue"] Текст [/ifxfvalue] они могут не сработать. Нужно либо убирать '_', но тогда могут быть ложные срабатывания, либо просто не использовать подобный перебор.

[Sander1 62]

Точно, подзабыл я про тег ifxfvalue.

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