Модерация комментариев

[Sander1 62]

При массовом подтверждении комментариев если случайно обновить страницу и подтвердить повторную отправку формы, то в новостях будет повторно увеличен счётчик comm_num+1
Не выполняется проверка на существование комментария и находится ли он на модерации.

[celsoft 6242]

Спасибо. Принято. Будет испрравлено в следующей версии DLE.

[Sander1 62]

Не охота создавать отдельный топик, файл engine/ajax/adminfunctions.php – небольшая опечатка:

index.php??subaction

PS. К слову в этом файле для метода commentspublic тоже можно добавить проверку существования коммента на всякий случай.

[celsoft 6242]

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

PS. К слову в этом файле для метода commentspublic тоже можно добавить проверку существования коммента на всякий случай.

Там браузер нельзя перегрузить, это AJAX запрос а комментарий удалиться после первого же обращения. Но на всякий случай в будущем предусмотрим. Спасибо.

[Sander1 62]

Проблемы с очисткой кеша.

1. При добавлении коммента, который уходит на модерацию – всё равно чистится кеш. Самое плохое что чистится кеш всего контента – `news_`

2. При ajax подтверждении комментария – излишне чистится почти весь кеш всего сайта, кроме того который реально необходимо чистить – `'comm_' . $post_id`

[celsoft 6242]

6 минут назад, Sander1 сказал:

Самое плохое что чистится кеш всего контента – `news_`

2. При ajax подтверждении комментария – излишне чистится почти весь кеш всего сайта, кроме того который реально необходимо чистить – `'comm_' . $post_id`

Вы забываете про количество комментариев, которое существует еще и у кратких новостей, поэтому DLE делает все правильно

[Sander1 62]

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

DLE делает все правильно

Комментарий отправляется на модерцию. Счётчик новости не меняется. Зачем в таком случае чистить кеш?
engine/modules/addcomments.php
Перед строкой:

clear_cache( array( 'news_', 'comm_'.$post_id, $cprefix, 'stats' ) );

Достаточно добавить код:

if ($where_approve)

И обратите внимание на п.2. При подтверждении комментария через AJAX обработчик не очищается кеш комментариев новости.

 

PS. И ещё, чтобы отдельную тему не создавать. Пользователь с именем `noname` проклят невозможностью выставлять оценки в рейтинге.
Все гости записываются в таблицу `dle_logs` и `dle_comment_rating_log` под именем `noname`.
Может будет правильнее для гостей записывать NULL или хотя бы пустую строку в качестве имени?

Изменено 31 октября 2023 пользователем Sander1

[celsoft 6242]

10 часов назад, Sander1 сказал:

Комментарий отправляется на модерцию. Счётчик новости не меняется. Зачем в таком случае чистить кеш?

Когда я писам то имел ввиду что:

Цитата

Самое плохое что чистится кеш всего контента – `news_`

Это нужно делать. И это делает DLE правильно.

10 часов назад, Sander1 сказал:

И обратите внимание на п.2. При подтверждении комментария через AJAX обработчик не очищается кеш комментариев новости.

Это мы посмотрим и исправим.

10 часов назад, Sander1 сказал:

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

Да будет он проклят, потому что nonane и сам решил таким стать ))). Это шутка если что ))