Sander1 61 Опубликовано: 10 октября 2023 Рассказать Опубликовано: 10 октября 2023 При массовом подтверждении комментариев если случайно обновить страницу и подтвердить повторную отправку формы, то в новостях будет повторно увеличен счётчик comm_num+1 Не выполняется проверка на существование комментария и находится ли он на модерации. Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 6 035 Опубликовано: 11 октября 2023 Рассказать Опубликовано: 11 октября 2023 Спасибо. Принято. Будет испрравлено в следующей версии DLE. Цитата Ссылка на сообщение Поделиться на других сайтах
Sander1 61 Опубликовано: 14 октября 2023 Рассказать Опубликовано: 14 октября 2023 Автор Не охота создавать отдельный топик, файл engine/ajax/adminfunctions.php – небольшая опечатка: index.php??subaction PS. К слову в этом файле для метода commentspublic тоже можно добавить проверку существования коммента на всякий случай. Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 6 035 Опубликовано: 14 октября 2023 Рассказать Опубликовано: 14 октября 2023 2 часа назад, Sander1 сказал: PS. К слову в этом файле для метода commentspublic тоже можно добавить проверку существования коммента на всякий случай. Там браузер нельзя перегрузить, это AJAX запрос а комментарий удалиться после первого же обращения. Но на всякий случай в будущем предусмотрим. Спасибо. Цитата Ссылка на сообщение Поделиться на других сайтах
Sander1 61 Опубликовано: 31 октября 2023 Рассказать Опубликовано: 31 октября 2023 Автор Проблемы с очисткой кеша. 1. При добавлении коммента, который уходит на модерацию – всё равно чистится кеш. Самое плохое что чистится кеш всего контента – `news_` 2. При ajax подтверждении комментария – излишне чистится почти весь кеш всего сайта, кроме того который реально необходимо чистить – `'comm_' . $post_id` Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 6 035 Опубликовано: 31 октября 2023 Рассказать Опубликовано: 31 октября 2023 6 минут назад, Sander1 сказал: Самое плохое что чистится кеш всего контента – `news_` 2. При ajax подтверждении комментария – излишне чистится почти весь кеш всего сайта, кроме того который реально необходимо чистить – `'comm_' . $post_id` Вы забываете про количество комментариев, которое существует еще и у кратких новостей, поэтому DLE делает все правильно Цитата Ссылка на сообщение Поделиться на других сайтах
Sander1 61 Опубликовано: 31 октября 2023 Рассказать Опубликовано: 31 октября 2023 (изменено) Автор 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 6 035 Опубликовано: 1 ноября 2023 Рассказать Опубликовано: 1 ноября 2023 10 часов назад, Sander1 сказал: Комментарий отправляется на модерцию. Счётчик новости не меняется. Зачем в таком случае чистить кеш? Когда я писам то имел ввиду что: Цитата Самое плохое что чистится кеш всего контента – `news_` Это нужно делать. И это делает DLE правильно. 10 часов назад, Sander1 сказал: И обратите внимание на п.2. При подтверждении комментария через AJAX обработчик не очищается кеш комментариев новости. Это мы посмотрим и исправим. 10 часов назад, Sander1 сказал: PS. И ещё, чтобы отдельную тему не создавать. Пользователь с именем `noname` проклят невозможностью выставлять оценки в рейтинге. Да будет он проклят, потому что nonane и сам решил таким стать ))). Это шутка если что )) Цитата Ссылка на сообщение Поделиться на других сайтах
Рекомендованные сообщения
Присоединяйтесь к обсуждению
Вы можете опубликовать сообщение сейчас, а зарегистрироваться позже. Если у вас есть аккаунт, войдите в него для написания от своего имени.