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

[dds 47]

2 часа назад, Travor сказал:

при просмотре категорий сейчас так

https://post-images.org/photo-page.php?photo=wre9qTW3

а при просмотре тега так

https://post-images.org/photo-page.php?photo=VK0sijpA

как вот вывести также количество публикаций? 

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

В файле engine/modules/main.php найти:

$tpl->load_template ( 'main.tpl' );

и добавить выше:

$tag_name = $nam_e;
$cached_result = false;
$cached_time = 3600;      // Время жизни кеша в секундах (1 час)

$cached_data = dle_cache('tags_count_' . $tag_name, $cached_time);
    if ($cached_data !== false) {
        $tag_count = $cached_data;
    } else {
        $tag_result = $db->super_query("SELECT COUNT(*) as count FROM " . PREFIX . "_post WHERE FIND_IN_SET('{$tag_name}', REPLACE(REPLACE(tags, ', ', ','), ' ', '')) > 0");
        $tag_count = $tag_result['count'];
        create_cache('tags_count_' . $tag_name, $tag_count, $cached_time);
    }

$tpl->set('{tags-title}', $tag_name);
$tpl->set('{tags-count}', $tag_count);

Хоть каждый тег и кэшируется ( Файлов кэша будет столько , сколько у тебя тегов ), но для каждого тега будет свой запрос.

[Travor 0]

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

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

В файле engine/modules/main.php найти:

$tpl->load_template ( 'main.tpl' );

и добавить выше:

$tag_name = $nam_e;
$cached_result = false;
$cached_time = 3600;      // Время жизни кеша в секундах (1 час)

$cached_data = dle_cache('tags_count_' . $tag_name, $cached_time);
    if ($cached_data !== false) {
        $tag_count = $cached_data;
    } else {
        $tag_result = $db->super_query("SELECT COUNT(*) as count FROM " . PREFIX . "_post WHERE FIND_IN_SET('{$tag_name}', REPLACE(REPLACE(tags, ', ', ','), ' ', '')) > 0");
        $tag_count = $tag_result['count'];
        create_cache('tags_count_' . $tag_name, $tag_count, $cached_time);
    }

$tpl->set('{tags-title}', $tag_name);
$tpl->set('{tags-count}', $tag_count);

Хоть каждый тег и кэшируется ( Файлов кэша будет столько , сколько у тебя тегов ), но для каждого тега будет свой запрос.

блин, дружище, спасибо большое)

[dds 47]

18 минут назад, Travor сказал:

блин, дружище, спасибо большое)

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

$tag_name = $nam_e;
$cached_time = 3600; // Время жизни кеша в секундах (1 час)

$cached_data = dle_cache('tags_data_cache', $cached_time); 
$cached_array = ($cached_data !== false) ? unserialize($cached_data) : []; 

if (!isset($cached_array[$tag_name])) {
    $tag_result = $db->super_query("SELECT COUNT(*) as count FROM " . PREFIX . "_post WHERE FIND_IN_SET('{$tag_name}', REPLACE(REPLACE(tags, ', ', ','), ' ', '')) > 0");
    $tag_count = $tag_result['count'];
    
    $cached_array[$tag_name] = $tag_count; 
    
    create_cache('tags_data_cache', serialize($cached_array), $cached_time);
} else {
    $tag_count = $cached_array[$tag_name]; 
}

$tpl->set('{tags-title}', $tag_name);
$tpl->set('{tags-count}', $tag_count);

 

[Gameer 310]

2 минуты назад, dds сказал:

$cached_time = 3600; // Время жизни кеша в секундах (1 час)

Кэш жизни тут не работает, скажите это своему chat-gpt

[dds 47]

20 минут назад, Gameer сказал:

Кэш жизни тут не работает, скажите это своему chat-gpt

При чем тут chat-gpt?  Простите вы всех так равняете? Типа вы один всезнающий, а остальные не способны ни на что?  Пусть мои знания недостаточны, но меня этому никто не учил. Все постигается долгими вечерами. Это вам так на заметку

[fanera 22]

https://developers.google.com/search/docs/appearance/structured-data/search-gallery?hl=ru

В следующей версии стоит перепроверить разметку и добавить новую - ProfilePage https://developers.google.com/search/docs/appearance/structured-data/profile-page?hl=ru, это будет очень полезно 

Например, для SoftwareApplication https://developers.google.com/search/docs/appearance/structured-data/software-app?hl=ru Google рекомендуют указать BusinessApplication или GameApplication (там список намного больше), а сейчас DLE выводит название категории. Это совсем неверно и не соблюдает гайдлайны гугла.

Кроме того, свойство description не поддерживается, его можно вообще убрать

Изменено 7 июня пользователем fanera

[celsoft 6 076]

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

Например, для SoftwareApplication https://developers.google.com/search/docs/appearance/structured-data/software-app?hl=ru Google рекомендуют указать BusinessApplication или GameApplication (там список намного больше), а сейчас DLE выводит название категории. Это совсем неверно и не соблюдает гайдлайны гугла. Кроме того, свойство description не поддерживается, его можно вообще убрать

Микроразметка определяется не поисковиком Google, а стандартом https://schema.org Категории могут быть согласно стандартам не то что там пишет гугл и использоваться не только для Google но и для множества других сервисов.  И никаких противоречий стандартам в данном случае нет. Можете посмотреть внимательно стандарт данного типа https://schema.org/SoftwareApplication А если вам нужно задать именно под гугл, то это не может быть универсально для всех сайтов, это персонально под конретный сайт. Можете это прописать под значения названия категорий, можете их отдельно плагином прописать, как вам угодно. DLE предоставляет универсальное решение, именно под стандарт, для любого сервиса, а не только для гугла.

[NlCKRUS 48]

можете добавить тэги по типу

[xfgiven_x][xfvalue_x][/xfgiven_x]

для шаблона комментариев, только чтобы это было не имя дополнительного поля профиля пользователя, а имя доп.поля новости?

[Crashlabs 56]

Который раз замечаю, что в текущем виде категориям не хватает гибкости - бывает возникает необходимость к каждой категории, например, привязать собственное изображение (помимо иконки), либо указать двойное название и тд. Думаю данную проблему решила бы возможность создавать дополнительные поля для категорий по аналогии с новостями.

[celsoft 6 076]

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

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

И что мешает это сделать сейчас? Теги которые выводят контент только в опеределенной категории в шаблонах никто не отменял.

[Crashlabs 56]

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

И что мешает это сделать сейчас? Теги которые выводят контент только в опеределенной категории в шаблонах никто не отменял.

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

Изменено 22 июня пользователем Crashlabs

[celsoft 6 076]

5 часов назад, Crashlabs сказал:

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

подключаете тегами {include ...} отдельный файл шаблона. И все редактируется не сложнее чем написать в админпанели. На каждое предложение полей не надобавляешся в админпанели.. Кому то нужно одну картинку, кому то две, кому то заголовок, кому то подзаголовок и т.д. Все подобное уже нужно делать в шаблонах, а не полями админпанель завешивать. Ваш случай очень и очень частный чтобы быть полем в админпанели.

[fanera 22]

Для custom хотелось бы иметь возможность учитывать не только дату публикации, но и дату редактирования. Пример:

{custom days="2" limit="10" order="reads"}

Как при таком подходе вывести новости с учетом тех, что некоторые более старые могли быть отредактированы? days="2" берет новости за последние 2 дня, а если в этот промежуток ничего нового не публиковалось, но редактировались другие новости?

Т.е. к примеру мне нужно вывести новости по кол-ву чтению за последние 2 дня публикации или редактированию, но для последнего просто нет возможности 

{topnews} с той же проблемой, нет возможности добавить учет того, чтобы в выборку попадали отредактированные новости

Изменено 29 июня пользователем fanera

[fanera 22]

Ещё не хватает в разделе управления комментариями помечать гостевые комментарии, иначе непонятно - это авторизованный пользователь или просто гость написал. При этом ссылка ведёт на ....mod=editusers&action=edituser&user=name, но при открытии открывает основную страницу редактирования пользователей, что порой мешает и вводит в заблуждение

[NlCKRUS 48]

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

вот тут ответ на самый первый комментарий, пока вручную имя не написал, не понятно кому ответили.

https://ibb.co/CtJ3BT5

[celsoft 6 076]

4 часа назад, NlCKRUS сказал:

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

Оно так и есть, а не как то иначе. И на вашем скриншоте точно также все. Вы просто не один ответили, а до вас ему же отвечали другие. Если бы вы ответили другому, то оно было бы в дереве уже другого.

 

4 часа назад, NlCKRUS сказал:

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

Написать логин кому вы пишите также можно, достаточно начать писать символ "@" и начао логина

[NlCKRUS 48]

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

Написать логин кому вы пишите также можно, достаточно начать писать символ "@" и начао логина

было бы удобно, чтобы логин подставлялся в текст комментария автоматически при нажатии кнопки "ответить".

[Gameer 310]

Пожелания по улучшению работы с категориями

1. Полностью ajax добавление, ибо при добавлении категории, честно, очень сильно устаёшь кликать "Добавить" -> "Вернуться назад" -> "Добавить категорию" -> "Добавить" и так по кругу. Особенно если их много.
2. Не понятно, почему включены галочки "Включить Дзен по умолчанию" и "Включить Яндекс Турбо по умолчанию", каждый раз приходиться их снимать, потому что эти настройки мне вообще не нужны. По какой причине разработчики посчитали что они должны быть включены по умолчанию - не понятно. Я думал если в RSS отключу настройки связанные с Дзен и Яндекс Турбо то они не будут иметь статус включенных по умолчанию - но не тут то было.
3. Увеличить окно добавления категории.
4. Убрать возможность если клик приходится мимо окна добавления категории - чтобы окно не закрывалось по умолчанию. Только на "крестик" в окне или кнопке "Отмена". Или же добавить проверку, если есть измененные данные в окне добавления - то вместо закрытия будет уведомление "Вы точно хотите закрыть окно добавления категории?"
5. Возможность просматривать данные о категории без необходимости переходить на страницу её редактирования.

Это пока навскидку.

[celsoft 6 076]

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

Не понятно, почему включены галочки "Включить Дзен по умолчанию" и "Включить Яндекс Турбо по умолчанию", каждый раз приходиться их снимать, потому что эти настройки мне вообще не нужны. По какой причине разработчики посчитали что они должны быть включены по умолчанию - не понятно. Я думал если в RSS отключу настройки связанные с Дзен и Яндекс Турбо то они не будут иметь статус включенных по умолчанию - но не тут то было.

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

[Gameer 310]

14 минут назад, celsoft сказал:

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

Я это понял, я лишь о том, что настройки "остаются включенными по умолчанию" (они не будут работать, я знаю) при создании категории. Просто чисто с точки зрения UX было б лучше если глобальная настройка отключена то и галочки там больше по умолчанию не включать.

[celsoft 6 076]

34 минуты назад, Gameer сказал:

Я это понял, я лишь о том, что настройки "остаются включенными по умолчанию" (они не будут работать, я знаю) при создании категории. Просто чисто с точки зрения UX было б лучше если глобальная настройка отключена то и галочки там больше по умолчанию не включать.

Плохая логика. Когда вы по тем или иным причинам решите вернуть потоки и включить их. Что вы должны потом должны будете делать по вашей логике?  Идти и редактировать все категории и включать их? Это уж точно не логичное действие.

[Gameer 310]

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

Плохая логика. Когда вы по тем или иным причинам решите вернуть потоки и включить их. Что вы должны потом должны будете делать по вашей логике?  Идти и редактировать все категории и включать их? Это уж точно не логичное действие.

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

Я лишь предложил, а не указываю или требую. Решать вам.

[celsoft 6 076]

47 минут назад, Gameer сказал:

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

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

[ntrtv 55]

Сделайте, пожалуйста, чтобы тег {vote-num} выводил только число. без <span>. Для микроразметки через JSON иначе он не подходит. Спасибо!

[celsoft 6 076]

12 часов назад, ntrtv сказал:

Сделайте, пожалуйста, чтобы тег {vote-num} выводил только число. без <span>. Для микроразметки через JSON иначе он не подходит. Спасибо!

Невозможно. <span> не просто так и не для красоты. Он нужен чтобы работало автоматическое обновление контента при AJAX действиях без перезагрузки страницы.