CMS DataLife Engine - Система управления сайтами

Sign in to follow this  
Sander1

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

Recommended Posts

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]);
		}
	}

 

  • Like 1

Share this post


Link to post
Share on other sites

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

Share this post


Link to post
Share on other sites

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

 

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

Share this post


Link to post
Share on other sites

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

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

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this