[dev] Advanced warnings for phpBB3

[VladPowers]

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

update xxx_users set user_warnings = 0 where user_warnings < 0

*где xxx имя Вашего префикса базы данных.

[JuanLUX]

Hi all,

First of all I would like to apologize for writing this in english instead of the Russian official language of this board but as you can imagine my knowledge of your language is null.

Please forgive me and think than It could be also worse as, my mother tongue is Spanish .

My question to rxu (the author of this awesome MOD) is the following:

In the mod instructions for the installation of this MOD you stated that, in the includes/mcp/mcp_warn.php file we have to look for

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

            'WARNING_TIME'   => $user->format_date($row['user_last_warning']),
            'WARNINGS'      => $row['user_warnings'],

and replace it by:

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

            'WARNING_TIME'      => ($row['warning_end']) ? $user->format_date($row['warning_end']) : $user->lang['PERMANENT'],
            'WARNINGS'         => $user->format_date($row['warning_time']),
            'WARNING_STATUS'   => ($row['warning_status']) ? true : false,
            'U_WARNING_POST_URL'=> ($row['post_id']) ? append_sid("{$phpbb_root_path}viewtopic.$phpEx", 'p=' . $row['post_id'] . '#p' . $row['post_id']) : '',
            'U_EDIT'         => ($auth->acl_get('m_warn')) ? append_sid("{$phpbb_root_path}mcp.$phpEx", 'i=warn&mode=' . (($row['post_id']) ? 'warn_post&p=' . $row['post_id'] : 'warn_user') . '&u=' . $row['user_id'] . '&warn_id=' . $row['warning_id']) : '',

However, at least in version 3.0.7-PL1 of phpBB, this search pattern appears twice in the mcp_warn.php file eve including the same number of leading TABS. I guess that the correct replacement should take place inside the body of the mcp_warn_list_view function, but I'm not sure.

May you possibly confirm this?

Thanks in advance and again sorry for not being able to write this in Russian.

[rxu]

JuanLUX
Installing MODs, you have to perform search sequentially.
For example: you find

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

      view_warned_users($users, $user_count, $config['topics_per_page'], $start, $sql_where, $sql_sort);

and replace it with

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

      view_warnings_list($users, $user_count, $config['topics_per_page'], $start, $sql_where, $sql_sort);

so this code

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

            'WARNING_TIME'   => $user->format_date($row['user_last_warning']),
            'WARNINGS'      => $row['user_warnings'],


follows the previous search entry (and yes, this is the end of function mcp_warn_list_view ).

[JuanLUX]

I think I have found a bug in the MOD. If you try to ban a user through the MCP and write as a ban reason a long text (>255 chars) you will obtain the following SQL ERROR:

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

Data too long for column 'ban_reason' at row 1 [1406]

INSERT INTO phpbb_banlist (ban_userid, ban_start, ban_end, ban_exclude, ban_reason, ban_give_reason) VALUES ......

It seems that the lenght of this "ban_reason" field needs to be checked.

How can we do that?

[rxu]

As a quick fix, try the following:

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

OPEN
styles/prosilver/template/mcp_warn_post.html

FIND
<textarea name="warning" id="warning" class="inputbox" cols="40" rows="3"><!-- IF WARNING -->{WARNING}<!-- ELSE -->{L_WARNING_POST_DEFAULT}<!-- ENDIF --></textarea>

REPLACE WITH
<input name="warning" id="warning" type="text" class="inputbox" maxlength="255" value="<!-- IF WARNING -->{WARNING}<!-- ELSE -->{L_WARNING_POST_DEFAULT}<!-- ENDIF -->" />

OPEN
styles/prosilver/template/mcp_warn_user.html

FIND
<textarea name="warning" id="warning" class="inputbox" cols="40" rows="3"><!-- IF WARNING -->{WARNING}<!-- ENDIF --></textarea>

REPLACE WITH
<input name="warning" id="warning" type="text" class="inputbox" maxlength="255" value="<!-- IF WARNING -->{WARNING}<!-- ENDIF -->" />

[JuanLUX]

Thanks rxu,

Of course this works fine although now the form doesn't look as "attractive" as you initially designed. It's a pity that the <textarea> HTML tag hasn't got a maxlenght attribute.

Again many thanks for your help and support.

[Альтаир Бергадлер]

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

Добавлено спустя 1 час 11 минут 25 секунд:
Поставил на 307

при выдаче преды
mcp/mcp_warn.php on line 778

вот собственно код

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

$cache->destroy('sql', WARNINGS_TABLE);
   // We add this to the mod log too for moderators to see that a specific user got warned.

проверял, все сделано по инструкции

[Альтаир Бергадлер]

Вопрос уже всплывал, но отыскать его так и не смог.
Где ковырять цвет желтой полоски предупреждений?

Благодарю

[hdv]

в шаблоне вывода тем viewtopic_body.html
найти:

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

dd style="background-color: rgb(255, 255, 153);

и играться параметрами функции rgb

rxu, подскажите, можно ставить мод на боевой форум или нет? он все еще в стадии "бета"? спасибо

[rxu]

hdv
Он всё ещё в стадии development, ставить можно только в случае, если сможете сами разбираться в глюках.

[MAzZY]

Он всё ещё в стадии development, ставить можно только в случае, если сможете сами разбираться в глюках

Какое слово объяснить?

[kak2z]

Да все понятно. Просто хотелось бы услышать отзывы тех кто уже пользуется продолжительное время. Спасибо.

[FEAR93]

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

Drakoon 109.205.252.119 Чт июн 10, 2010 5:13 pm Разблокирован пользователь» Хардкорный

. Но есть одно но, пользователь Drakoon не является ни администратором ни модератором, данный лог привёл как пример. На его месте часто бывает имя другого пользователя. Версия форума 3.0.7-PL1.

[Nekstati]

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

[FEAR93]

Nekstati, спасибо за объяснение.
Но, можете, пожалуйста, немного вдаться в подробности и рассказать из-за чего это происходит? Чисто для себя интересно.