[FINAL] Warnings Mod

[dnk]

Посмотрел Ваш мод:
интересно сделали.Причем вроде нет возможности установить нужный период предупреждения (любое число дней), а только из списка.
Если у администраторов установлен мод Глобальных модераторов, то в скрипте warnings.php нужно в массиве

Код: Выделить всё

array(ADMIN,MOD) 

заменить на

Код: Выделить всё

array(ADMIN,GLOBAL_MOD,MOD)

везде, где встретится.
А также найти

Код: Выделить всё

$userdata['user_level'] != ADMIN

и заменить на

Код: Выделить всё

 $userdata['user_level'] != ADMIN || $userdata['user_level'] != GLOBAL_MOD

, если Вы хотите дать супермодераторам админские полномочия по выдаче предупреждений и блокировок и редактрования, снятия и удаления их.
Если установлен мод Супермодераторов, то константа будет не GLOBAL_MOD , а SUPER_MOD - т.е. та, что определена у вас в includes/constants.php

[Xpert]

go
Поправим в следующей версии.

dnk
Да уж, приходится изворачиваться в поисках решений, обеспечиващих максимально возможную производительность.
За рецепты спасибо, только уточню что еще есть проверка $postrow[$i]['user_level'] во viewtopic.php, возможно и там потребуется внести коррективы.

[dnk]

go
только уточню что еще есть проверка $postrow[$i]['user_level'] во viewtopic.php, возможно и там потребуется внести коррективы.

Да-да, это я чуть позже вспомнил - возможно для отображения мини-иконки желтой карты в постах рядом с простыми юзерами.Да, собственно, и ставится в разы быстрее, чем "Демократия" - так как, собственно фич содержит мало (последнюю версию "Демократии" просто "раздули" ими).Отсутсвует система репутации - ну тут, я думаю, мод предполагался только как цель обеспечить администрацию функциями предупреждения, а модераторов еще и функциями блокировки, в т.ч. на время, а не только перманентно.Можно, по идее, если есть модер-панель modcp, взять и скопировать скрипт admin_user_ban.php в нее (подправив, разумеется, футер внизу файла внутри), но бывают такие модеры, которые смогут, используя этот скрипт, типа шутки ради, забанить админа по IP...

[Unix_spv]

Небольшой баг при удалении юзеров из админки. В файле admin/admin_users.php ругается на запрос:

$sql = "SELECT warning_id FROM " . WARNINGS_TABLE . " WHERE warning_userid = $user_id";

тут поле warning_userid неправильно написано, нужно warning_user_id.

Соотвественно правильный запрос такой:

$sql = "SELECT warning_id FROM " . WARNINGS_TABLE . " WHERE warning_user_id = $user_id";

[go]

$sql = "SELECT warning_id FROM " . WARNINGS_TABLE . " WHERE warning_userid = $user_id";

она же(ошибка) не даст назначать модераторов в рубрику....

[Xpert]

она же(ошибка) не даст назначать модераторов в рубрику....

А конкретнее можно, как воспроизвести?

[go]

Панель администратора->Пользователи->Права пользователя->Попытка просмосмотра прав пользователя->Вывод ошибки:
Общая ошибка
Could not select all users private messages

DEBUG MODE

SQL Error : 1054 Unknown column 'warning_userid' in 'where clause'

SELECT warning_id FROM phpbb_warnings WHERE warning_userid = 458

Line : 195
File : admin_users.php
==============
на строке 194:$sql = "SELECT warning_id FROM " . WARNINGS_TABLE . " WHERE warning_userid = $user_id";
на строке 195:$result = $db->sql_query($sql) or message_die(GENERAL_ERROR, 'Could not select all users private messages', '', __LINE__, __FILE__, $sql);
==============
Тоже самое при попытке удалить юзеря.Благополучно исправлено)

[Xpert]

Да это-то понятно, интересно другое - почему он при просмотре пытается этот запрос выполнить...

[Mr. Anderson]

Could not select all users private messages

private messages? Я не ослышался?

[Xpert]

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

[natali]

$lang['Warning_cannot_remove_expired'] = 'Вы не можете снять истекщее предупреждение';

а гед можно псомотреть его в работе?

[Xpert]

Эту языковую переменную? Или мод?

[natali]

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

сам мод посмотреть...

[go]

на счет языковой переменной. в моде присуствует такая языковая переменная:
$lang['Warnings_not_enough_params'] = 'Недостаточно данных';
в каких случаях она появляется?
при использовании мода заметил , что только в одном: когда пытается модаратор дать предупреждение и не вводит тект с объяснением за что дано предупреждение или же бан.
Многие модераторы откровенно тупят видя такое сообщение...Возможно .правильней было бы написать "Вы должны объяснить за что Вы даете предупреждение или бан"

тупят,имеется ввиду,входят в ступор.а так они умные.)

[Lektor]

Идея для аддона:
Выводить причину бана, когда пользователь патается зайти на форум(причина берется та, которую указал модератор при бане), а если он получил бан по совокупности предупреждений, то так и пишется: Бан по совокупности предупреждений и перечисляются.