Перейти к публикации

Доработать класс комментариев


Рекомендованные сообщения

Просьба разработчикам доработать класс комментариев, ибо вынесен как класс он был не зря, а наверное для того чтобы сторонние модули могли его использовать?

Сам вывод и постройка навигации работает, но вот работа с редактированием комментарием (формирование ссылки [com-edit]), удаление ([com-del]) - становится невозможным. Из-за этого большая просьба в след. версиях сделать хотя бы чтобы ссылка формировалась не по умолчанию:

index.php?do=comments
А с помощью доп. переменной, к примеру: index.php?do=" . $this->module . " Ну и сам класс формировался:
function DLE_Comments( $db, $total_comments, $comments_per_pages, $module = "comments" ) {

$this->module = $module;

Уверен с этим согласятся и другие разработчики модулей

Ссылка на сообщение
Поделиться на других сайтах

celsoft

Ну так что? Изменения в класс коммента будут внесены для разработчиков? Или всё таки писать класс под себя надо будет?

Ссылка на сообщение
Поделиться на других сайтах
  • 4 недели спустя...

Lion__,

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

Ссылка на сообщение
Поделиться на других сайтах

celsoft

И снова делать проверку прав юзера, является ли админом, проверять второй раз то, что уже проверил класс?

Этот же пример, допустим надо сделать удаление комментария, но для для другого модуля который использует этот класс:

Так как я выше предложил:

Создание класса как обычно, но указать 4-рты параметр, например demo

Так как вы предложили:

1) Выполнить запрос к бд для получения ника юзера которы написал коммент

2) Выполнить еще раз проверку: является ли активный юзер зарегистрированным, есть ли у активного юзера права на удаление своих комментов, является ли активный юзер админом, является ли активный юзер автором данного коммента

3) Создать правило

Разница на глазах, неправда ли?

Изменено пользователем Lion__
Ссылка на сообщение
Поделиться на других сайтах

Так как вы предложили:

1) Выполнить запрос к бд для получения ника юзера которы написал коммент

2) Выполнить еще раз проверку: является ли активный юзер зарегистрированным, есть ли у активного юзера права на удаление своих комментов, является ли активный юзер админом, является ли активный юзер автором данного коммента

3) Создать правило

Разница на глазах, неправда ли?

Если честно, то неправда. Никаких запросов к БД делать не нужно, получать ники или что то еще. Реализация проверки и нового тега, это две строчки кода, как собственно и в стандартном классе, поэтому эти пункты, мягко говоря преувеличение. А вот передача каких то параметров в класс ничего не дает, удаление от этого не заработает.

Ссылка на сообщение
Поделиться на других сайтах

Если честно, то неправда. Никаких запросов к БД делать не нужно, получать ники или что то еще.

Просветите меня...

А вот передача каких то параметров в класс ничего не дает, удаление от этого не заработает. 

от класса и не требуется удаление, только формирование ссылки

Ссылка на сообщение
Поделиться на других сайтах

Просветите меня...

Читайте документацию по доступным в скрипте массивам данных http://dle-news.ru/extras/online/index.html?modules_include.html

от класса и не требуется удаление, только формирование ссылки

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

Ссылка на сообщение
Поделиться на других сайтах

Читайте документацию по доступным в скрипте массивам данных

Я про: текущий юзер = автор коммента, надо делать запрос, хоть он и так выполнялся в классе

Ссылка на сообщение
Поделиться на других сайтах

Присоединяйтесь к обсуждению

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

Гость
Ответить в тему...

×   Вставлено в виде отформатированного текста.   Вставить в виде обычного текста

  Разрешено не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отобразить как ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставить изображения напрямую. Загрузите или вставьте изображения по ссылке.

×
×
  • Создать...