[dev] Advanced warnings for phpBB3

[rxu]

https://github.com/rxu/advanced_warnings

[rxu]

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

[Alecto]

потом пред могут отредактировать по сроку в любую сторону

как?

[rxu]

В модераторском разделе в списке предупреждений напротив каждого (если не истекло) есть ссылка.

[Alecto]

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

[Gubkin]

последняя версия (именно актуальная на сегодня) расширения установлена?

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

[rxu]

имеем то, что на скринах.

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

[Alecto]

Если пред бессрочный

да, именно так.
имеет ли какое-то значение то, что предупреждения выдавались до установки расширения?

Видимо, баг последних правок

как отлавливать/бороться?

здесь, на гуру, используется это же расширение?

Отправлено спустя 4 минуты 46 секунд:
на тестовом сайте, где предупреждения выдавались после установки расширения, все именно так, как вы писали, возможность редактирования есть.

Отправлено спустя 1 минуту 14 секунд:
теперь только удалить все предупреждения и блокировки, и выдать их еще раз после установки расширения? или есть другой способ?

[rxu]

имеет ли какое-то значение то, что предупреждения выдавались до установки расширения?

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

[LavIgor]

Здесь, прежде, чем править, надо над алгоритмом подумать

Алгоритм я предложил.
Попробую реализовать (может, и сегодня).
Первым делом протестирую, конечно.

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

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

выданы бессрочные предупреждения, их время не может выйти

По-моему, на тестовом у меня такая ситуация была. Очень давно, ещё когда phpBB 3.1 официально не вышел.
Посмотрю, в общем.

[Alecto]

LavIgor, спасибо.
я пока основу не трогаю. на тесте можно будет экспериментировать и тестировать сколько угодно.

[LavIgor]

Нашёл. warning_status - столбец, добавляемый расширением.
Но именно от него зависит, истекло ли предупреждение или нет.
Зачем он нужен, совсем не понятно: ведь и без того можно высчитать по дате, истекло ли конкретное предупреждение.
Т.е. столбец можно по сути просто удалить.

Теперь что касается старых предупреждений. Как по мне, так это дыра в совместимости.
В самой первой миграции, по-хорошему, нужно каждому имеющемуся предупреждению присвоить его дату на основании конфигурационной настройки warnings_expire_days.
И только потом заменять её нулём.

Требуется доработка. Ничего страшного, статус расширения до сих пор dev.

[Alecto]

мне оно нужно было, чтобы показать "забанен". свою задачу пока выполняет. а там, глядишь - и допилят :)

[Gubkin]

а можно сделать так, чтобы красное поле при бане было кликабельно ?
т.е. если человек в бане, то можно было перейти на его сообщение где была выдача
например чтоб при нажатии на "Забанен: Бессрочно" у Пчелкина выходить на Re: Предложения и замечания (Пост Пчелкин #468532)

[LavIgor]

Gubkin, можно, но большого смысла в этом нет.
Обычно человека банят не за одно сообщение, а за его общее поведение.
К тому же, ссылка на сообщение, за которое был получен бан, имеется в профиле пользователя.

[Stan_S]

LavIgor, Надо бы сделать думаю)