Пожелания для новых версий DataLife Engine

[celsoft 6 179]

3 часа назад, Gameer сказал:

• Включёна поддержка related news.
• Теги вывода в полной не используем
• Вместо этого используем custom с тегом {related-ids}
• В итоге у нас дополнительный ненужный запрос на получение новостей в related-news и обработка tpl файла который также не нужен в этом кейсе.

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

[Gameer 317]

8 часов назад, celsoft сказал:

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

То есть вы предлагаете каждый раз вручную нажимать кнопку? Отличное решение!

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

[celsoft 6 179]

11 часов назад, Gameer сказал:

То есть вы предлагаете каждый раз вручную нажимать кнопку? Отличное решение!

Я не предлагаю. Я лишь говорю, что это ваш выбор, либо экономить один запрос к БД и делать это вручную, либо не экономить запрос и все будет на автомате.

11 часов назад, Gameer сказал:

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

Вот именно. Я сторонник оптимизации, а вы предлагаете свой частный случай навязать абсолютно всем пользователям DLE, и делать запросы к БД тем кому не нужно, потому как если DLE будет искать похожие когда это отключено, он будет это делать всем, даже тем кто не использует вашу частную конфигурацию для вывода.. Похожие новости из ниоткуда ведь невозмуться, их все равно искать нужно, а значит делать запрос. И этому я противник, потому как это нагрузка тем кому это не нужна. Частные случаи делаются либо вручную как я сказал выше, либо плагином в три строки, а не навязываются всем по умолчанию в коробочной CMS

[Gameer 317]

1 час назад, celsoft сказал:

Я не предлагаю. Я лишь говорю, что это ваш выбор, либо экономить один запрос к БД и делать это вручную, либо не экономить запрос и все будет на автомате.

Вот именно. Я сторонник оптимизации, а вы предлагаете свой частный случай навязать абсолютно всем пользователям DLE, и делать запросы к БД тем кому не нужно, потому как если DLE будет искать похожие когда это отключено, он будет это делать всем, даже тем кто не использует вашу частную конфигурацию для вывода.. Похожие новости из ниоткуда ведь невозмуться, их все равно искать нужно, а значит делать запрос. И этому я противник, потому как это нагрузка тем кому это не нужна. Частные случаи делаются либо вручную как я сказал выше, либо плагином в три строки, а не навязываются всем по умолчанию в коробочной CMS

Что? Вы явно не понимаете о чём идёт речь. Строка 1214

if ( $row['related_ids'] ) {

Заменить на 

if ($row['related_ids']) {
    $first_show = false;
}
if ($row['related_ids'] && stripos($tpl->copy_template, "{related-news") !== false) {

Потом строка 1235

} else {

Заменить на

} elseif (!$row['related_ids']) {

И код шаблона весь обернуть в проверку используется ли тег {related-news}

$tpl2 = new dle_template();
$tpl2->dir = TEMPLATE_DIR;
$tpl2->load_template( 'relatednews.tpl' );

А вы совсем не то пишите, какой-то запрос постоянно использовать. Никто не говорит включать запрос на постоянку, я пишу о том чтобы убрать запрос этот если вывод {related-news} идёт через {custom с помощью тега {related-ids} ибо в таком случае у вас висит не использованный запрос и скомпилированный шаблон relatednews.tpl

$db->query( "SELECT id, date, short_story, xfields, title, category, alt_name FROM " . PREFIX . "_post WHERE id IN({$row['related_ids']}) AND approve=1 ORDER BY FIND_IN_SET(id, '{$row['related_ids']}') LIMIT " . $config['related_number'] );

 

Изменено 8 мая пользователем Gameer

[ntrtv 59]

Для тегов [next-url] и [prev-url] добавьте, пожалуйста, возможность вывода изображений из предыдущей и следующий новости, короткого описания для более гибкого вывода данного блока.

[fanera 28]

Есть опция "Уведомлять при редактировании, что новость уже редактируется", но никак не указано, что она работает только для админки. Она не работает при редактировании новости через сайт, а также не работает, если пользователь редактирует новость через сайт, а другой открывает через админку, и наоборот, когда кто-то редактирует через админку, а другой открывает через сайт. Было бы неплохо уведомлять во всех случаях

Изменено В понедельник в 18:35 пользователем fanera

[MGHaze 3]

1. Недавно был добавлен полезный параметр для custom - name="X", а также тег [custom=X] Текст [/custom]. Теперь, если юзаешь кастом в каком-то блоке в категориях (например, слайдере новостей данной категории), то заголовок (или весь блок) блока можно закинуть в [custom=X] Текст [/custom] и он не будет отображаться там, где недо (например, на пустых или несуществующих категориях). Хотелось бы такой же тег проверки наличия контента в {content}, аля [сcontent] Текст [/content].

2. Добавить возможность прописать в вышеупомянутом [custom=X] сразу несколько блоков через запятую. Кейс - один и тот же блок выводиться с заголовком и на главной и в категориях, но на главной custom выводит новости из всех категорий, а в категория только из текущей {custom category="{category-id}"... Весь блок имеет один и тот же код, кроме параметров кастома. Но чтобы скрывать этот блок методом из п.1, мне надо дублировать весь код блока, за исключением параметров custom. Но если бы работал тег для нескольких блоков сразу, например [custom="main-slider, cat-slider"], то можно было бы просто во внутрянке прописать 2 кастома, а не дублировать блок целиком.

3. Добавить для custom параметры new и updated (аналогичны одноименным тегам в полной/краткой новости), но чтобы они работали независимо от чекбоксов в новостях "Выводить информацию о том что публикация обновлена". Либо добавить параметру updated дополнительное значение, которое будет учитывать и эту галку. Например updated=yes/no/only

[MGHaze 3]

Не знаю, насколько будет правильным, но хотелось бы избавиться в коде от тегов таблиц. В файлах userinfo.tpl, registration.tpl, addnews.tpl тег {xfields} выводится таблицей. Адаптировать это то еще удовольствие. Учитывая, что это допполя с 2мя элементами (заголовок и значение), то можно было бы обойти чем то вроде:

<label></label>
<input/>

...как в некоторых других местах. Плюс, раздать классы. Или просто дивами, или как-то еще.

Также мб стоило бы избавиться от таблиц в ЛС, но тут я не уверен.