[dev] Advanced warnings for phpBB3

[rxu]

https://github.com/rxu/advanced_warnings

[valdos]

Простите, а с Auto Groups у вас не воспроизводится? Будете что-то делаться или можно не ждать?

[rxu]

Пока не смотрел.

[Evangelion]

И ещё хотелось бы опционально вкл/выкл отображение причины предупреждения/бана в сообщении:

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

[southklad]

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

[Evangelion]

А так же при нажатии на пред в профиле выкидывало на то сообщение за которое оно было выдано

Достаточно зайти в профиль к пользователю и можно переходить к каждому сообщению за которое были предупреждения

[LavIgor]

1. Нужно переделать механизм работы с блокировками. Т.е. разблокировку через Cron заменить на блокировку на правильный срок.
2. Как насчёт интегрирования в это расширение функциональности мода Warning Reasons? Очень хороший мод, давно думал над портированием в 3.1 (как раз-таки Advanced будет данное расширение).

Могу заняться этим позже.

[Gubkin]

если у юзера уже были предупреждения, то последующая выдача бана не видна под аватаркой...
помогает только полная очистка предупреждений, а затем выдача бана... тогда всё как положено, как тут Re: Advanced warnings for phpBB3 (Пост Evangelion #467277)

[LavIgor]

Gubkin, последняя версия (именно актуальная на сегодня) расширения установлена?
В ней любые блокировки аккаунтов должны быть видны в теме.
Если не видны, то позже поправим.

[rxu]

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

Не совсем понял, можно подробнее?

[LavIgor]

можно подробнее?

Сейчас пользователи блокируются бессрочно, если они набрали нужное для блокировки число предупреждений. Из-за этого возникает ряд проблем.
Лучше вместе этого при выдаче такого предупреждения высчитывать правильную дату окончания блокировки (наименьшую из возможных) и банить уже на эту дату. Тогда разблокировку в Cron можно будет просто убрать.

[rxu]

при выдаче такого предупреждения высчитывать правильную дату окончания блокировки (наименьшую из возможных) и банить уже на эту дату

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

[LavIgor]

Да, кстати, rxu, в той строчке $user_ban_id = (int) user_ban('user', $ban, 0, 0, 0, $warning, $warning);.
Почему так? Ведь user_ban возвращает только булевские true/false, а не id заблокированных пользователей.

[rxu]

Почему так?

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

[LavIgor]

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

Почему не получится? Берём дату блокировки, сравниваем с датой окончания текущего предупреждения (больше/меньше).
Если предупреждений на данный момент нужное количество (аккурат 3, например, но не больше и не меньше), а пользователь заблокирован на меньший срок (или не заблокирован), то блокируем пользователя на наименьшую дату окончания одного из его предупреждений (включая текущее).
Если же нужное количество предупреждений было накоплено ранее, то вот здесь уже придётся немного подумать. Нужно будет брать не самую маленькую дату, как в случае выше, а вторую (третью, четвёртую, пятую... в зависимости от того, сколько пользователь умудрился нахватать лишних предупреждений) по величине. Реализовать описанное очень просто, если правильно отсортировать массив предупреждений и выбрать правильную дату.

Я могу сделать это, тогда баги с блокировкой будут исправлены. Только не сегодня.

[Alecto]

1 - установил расширение (через аплоадер), включил, после чего зашел в тему - получил ошибку кэша (помнится, где-то rxu писал, что при установке кэш чистится автоматически; ошибку не сохранил, но чего-то там ...twig...)
пролечилось чисткой кэша.
2 - когда выключил расширение (через аплоадер), обновил тему F5 и попробовал выдать предупреждение за сообщение - получил ошибку отсутствующего модуля ( ошибку не сохранил).
опять же, пролечилось чисткой кэша.
3 - зачем выдавать бессрочные предупреждения по-умолчанию? почему нельзя поставить дефолтный срок = 1 месяц?
сразу этого не заметил, выдал нескольким персонам предупреждения, как теперь изменить длительность?