[BETA] Democracy MOD 0.2.1 (reputation + warnings + reports)

**Поручик** 03.01.2006 11:40

Carbofos писал(а): соответствующее поле из поста,

А такого поля в таблице phpbb_reputation_text и нет. Да и нужно ли оно?

Carbofos писал(а):Только прошу терпения

Да не беспокойтесь. Сами продолжим шлифовку.

Carbofos писал(а):Кстати, на phpbb.com никто на новые баги не жаловался, там все, видимо, старательно всё пропускают через Изю-Мода

Ну это же Европа, Америка. Ну, тупыыые (С)

**Carbofos** 03.01.2006 15:18

Поручик писал(а):А такого поля в таблице phpbb_reputation_text и нет. Да и нужно ли оно?

Есть в phpbb_posts :wink:

А нужно ли... да мало ли, тем же американцам, думаю, нужно

**IncominG** 04.01.2006 15:15

На выходных попробую написать совместимость,оформлю в виде мода, User level не обещаю ибо его я не видел, также уберу лишние строчки.
Спасибо за объявленные глюки.
Обратись к Carbofos.
Надеюсь ты не будешь против.

**Поручик** 04.01.2006 15:34

IncominG писал(а):попробую написать совместимость,оформлю в виде мода, User level не обещаю ибо его я не видел

Это я уже сделал. Ты посмотри на мой код выше. Его достаточно включить в post_report.php

**IncominG** 04.01.2006 15:52

Поручик
Хорошо, к выходным будет готово совместимлсть с user Level и Global Mod не только post_report, но и других файлов.
P.S. подними свой форум

**Поручик** 04.01.2006 16:38

Это хостер дурит.

**ETZel** 04.01.2006 17:37

Этот мод - просто супер! Репутация и предупреждения в одном флаконе. Благодаря ему мне не придется все писать с нуля

Для начала - найденные баги: (если я ошибся - извините)

Код: Выделить всё: #OPEN page_header.php #FIND $sql = 'SELECT id, user_id, modification FROM ' . REPUTATION_TABLE . ' WHERE modification = ' . REPUTATION_WARNING . ' OR modification = ' . REPUTATION_BAN . " AND expire < $current_time"; #REPLACE # добавил скобки вокруг OR - без них все предупреждения сразу становились expired $sql = 'SELECT id, user_id, modification FROM ' . REPUTATION_TABLE . ' WHERE (modification = ' . REPUTATION_WARNING . ' OR modification = ' . REPUTATION_BAN . ") AND expire < $current_time"; #OPEN profile_warning.tpl #FIND <div class="gen">{switch_ban.L_NEVER_EXPIRE}</span> #REPLACE # wrong closing tag <div class="gen">{switch_ban.L_NEVER_EXPIRE}</div> #FIND <div class="gen">{switch_warning.L_EXPIRE} <input type="input" name="expire" size="3" maxlength="3" value="{switch_warning.S_EXPIRE_VALUE}" {switch_warning.S_EXPIRE_DISABLED}/> {switch_warning.L_DAYS}</div><div class="gensmall">{switch_warning.L_EXPIRE_HINT}</div> #REPLACE # wrong closing tag <div class="gen">{switch_warning.L_EXPIRE} <input type="input" name="expire" size="3" maxlength="3" value="{switch_warning.S_EXPIRE_VALUE}" {switch_warning.S_EXPIRE_DISABLED}/> {switch_warning.L_DAYS}</div><div class="gensmall">{switch_warning.L_EXPIRE_HINT}</div> #OPEN user_reputation.php #FIND Already_banned #REPLACE # нет такой переменной $lang['Already_banned'] - надо добавить ??? #FIND if ( $poster_id == ANONYMOUS ) { message_die(GENERAL_MESSAGE, $lang['reputation_anonymous_no_reviews']); } #REPLACE # нет такой переменной $poster_id #FIND во многих местах $expire = ( $warning[$i]['expire'] == -1 ) ? $lang['reputation_never'] : create_date('d.m.Y', $info[$i]['expire'], $board_config['board_timezone']); #REPLACE # нет переменной $warning, есть $info $expire = ( $info[$i]['expire'] == -1 ) ? $lang['reputation_never'] : create_date('d.m.Y', $info[$i]['expire'], $board_config['board_timezone']); #FIND if ( $userdata['session_logged_in'] && $info[$i]['time'] > $userdata['user_lastvisit'] ) #REPLACE # нет $info[$i]['time'], есть $info[$i]['date'] if ( $userdata['session_logged_in'] && $info[$i]['date'] > $userdata['user_lastvisit'] ) #FIND в 2-x местах (второй немного отличается) $pagination = ( $highlight != '' ) ? generate_pagination("profile.$phpEx?mode=reputation&m=" . REPUTATION_VIEW . '&' . POST_POST_URL . "=$post_id&postorder=$post_order&highlight=$highlight", $plus + $minus, $board_config['reputation_reviews_per_page'], $start) : generate_pagination("profile.$phpEx?mode=reputation&m=" . REPUTATION_VIEW . '&' . POST_POST_URL . "=$post_id&postorder=$post_order", $plus + $minus, $board_config['reputation_reviews_per_page'], $start); #REPLACE # $highlight ни при чем, подсвечивать тут нечего $pagination = generate_pagination("profile.$phpEx?mode=reputation&m=" . REPUTATION_VIEW . '&' . POST_POST_URL . "=$post_id&postorder=$post_order", $plus + $minus, $board_config['reputation_reviews_per_page'], $start); #FIND switch ( $info[$i]['modification'] ) #REPLACE # чтобы не было undefined variable добавляем здесь $l_expire $l_expire = ''; switch ( $info[$i]['modification'] ) #FIND только первое входждение из 2-х! if ( !empty($postrow['user_icq']) ) #REPLACE # вместо $postrow надо $info if ( !empty($info[$i]['user_icq']) ) #FIND if ( !isset($rep_mode) ) #REPLACE # это косметика, но все же else #FIND if ( $HTTP_POST_VARS['exp'] == 1 || $HTTP_GET_VARS['exp'] == 1 ) #REPLACE # иначе undefined offset 'exp' if ( !empty($HTTP_POST_VARS['exp']) || !empty($HTTP_GET_VARS['exp']) ) #FIND во многих местах $is_auth = array(); #REPLACE # ??? зачем нужна эта строка перед всеми $is_auth = auth(...); #FIND $lang['reputation_report_post'] = "Report this post to moderator(s)"; #REPLACE # лишняя строка

Полубаги:
1) Очень много строк типа

Код: Выделить всё: if ($result = $db->sql_query('DELETE ...'))

Запросы INSERT/UPDATE/DELETE не возвращают rowset, зачем тут $result ?

2) $lang['reputation_no_access'] лучше заменить на $lang['Not_Authorised'], зачем дублировать? И вообще новых строк в $lang слишком много, без некоторых можно и обойтись

Технические замечания:
1) (спорный вопрос) слить воедино profile_modify_reputation.tpl и profile_warning.tpl - они достаточно похожи
2) в profile_view_reputation.tpl блоки 'warn' и 'rep' абсолютно идентичны. Нужно оставить только один, второй лишний.
3) аналогично в modcp_post_report_body.tpl блоки 'postrow' и 'reviewrow' почти идентичны, лучше их слить в один

Нужные опции (сделаю сам

):
1) разрешить не указывать причину при изменении репутации (пустой отзыв)
2) разрешить изменять репутацию из профиля (не указывая конкретный пост)
3) ввести временное ограничение на изменение репутации (например, нельзя изменять чаще, чем раз в неделю)
4) систему
5) опция не показывать пост на который ссылаешься при изменении репутации или резать его по длине до 200-500 символов
(посты могут быть длинные => форма отправки отзыва уезжает далеко вниз)
6) снимать предупреждения/баны последовательно, а не независимо. Например, есть 2 предупреждения по 7 дней и сверху еще бан на месяц. Тогда
бан пропадает через 30 дней. Второй предупреждение пропадает через 7 дней после исчезновения бана, третье - через 7 дней после второго и т.д.

Carbofos, тебе нужна помощь в разработке мода? Я бы с удовольствием поучаствовал (хотя у меня тоже сессия

)

**Поручик** 04.01.2006 18:22

ETZel
Это просто супер! Свежий глаз, называется.

**IncominG** 05.01.2006 23:51

Кто-нибудь проверял post_report в действии? У меня он по непонятной причине говорит, что я не могу просматривать эту страницу, как post_report из архива, так и модифицированный мной.

**Поручик** 05.01.2006 23:56

Да проверял. Просмотр и удаление постов. Нет проблем, коллега IncominG

**IncominG** 06.01.2006 4:01

Посмотри,пожалуйста мой post_report.
В коде ошибок не вижу, но в действии явно что-то не так.

Добавлено спустя 1 час 37 минут 36 секунд:

Ну вот собсно глюк такой...
Когда в default ставлю from и where от админа,всё работает

В противном случае- нет. Видимо не хочет определять мой юзер левел.

Добавлено спустя 16 минут 21 секунду:

Я наконец-то понял суть ошибки!
Он же млин меня разлогиняет.
Вот только почему?...
Объяснить неразумному...

Добавлено спустя 1 час 58 минут 33 секунды:

#FIND
<div class="gen">{switch_warning.L_EXPIRE} <input type="input" name="expire" size="3" maxlength="3" value="{switch_warning.S_EXPIRE_VALUE}" {switch_warning.S_EXPIRE_DISABLED}/> {switch_warning.L_DAYS}</div><div class="gensmall">{switch_warning.L_EXPIRE_HINT}</div>
#REPLACE
# wrong closing tag
<div class="gen">{switch_warning.L_EXPIRE} <input type="input" name="expire" size="3" maxlength="3" value="{switch_warning.S_EXPIRE_VALUE}" {switch_warning.S_EXPIRE_DISABLED}/> {switch_warning.L_DAYS}</div><div class="gensmall">{switch_warning.L_EXPIRE_HINT}</div>

не так... вот:

Код: Выделить всё: #FIND <div class="gen">{switch_warning.L_EXPIRE} <input type="input" name="expire" size="3" maxlength="3" value="{switch_warning.S_EXPIRE_VALUE}" {switch_warning.S_EXPIRE_DISABLED}/> {switch_warning.L_DAYS}</div><div class="gensmall">{switch_warning.L_EXPIRE_HINT}</span> #REPLACE WITH # wrong closing tag <div class="gen">{switch_warning.L_EXPIRE} <input type="input" name="expire" size="3" maxlength="3" value="{switch_warning.S_EXPIRE_VALUE}" {switch_warning.S_EXPIRE_DISABLED}/> {switch_warning.L_DAYS}</div><div class="gensmall">{switch_warning.L_EXPIRE_HINT}</div>

#FIND
Already_banned
#REPLACE
# нет такой переменной $lang['Already_banned'] - надо добавить
???

Ну во первых это не переменная, а текст, но если вам от этого легче, вводим:

Код: Выделить всё: #FIND 'Already_banned' #REPLACE WITH # $lang['reputation_already_banned'] #OPEN lang_main.php #FIND $lang['reputation_already_warned'] #AFTER ADD $lang['reputation_already_banned'] = 'Пользователь уже забанен.';

#FIND
if ( $poster_id == ANONYMOUS )
{
message_die(GENERAL_MESSAGE, $lang['reputation_anonymous_no_reviews']);
}
#REPLACE
# нет такой переменной $poster_id

А это что по вашему?

Код: Выделить всё: #FIND $poster_id = $postrow['user_id'];

И последний мой штрих...
При переходе на post_report.php не сохраняется сессия... Вот вам мой фикс:

Код: Выделить всё: #FIND $u_reported_posts = "<a href=\"post_report.$phpEx?mode=view\" class=\"gensmall\"$link_color>$l_reported_posts</a><br />"; #REPLACE WITH $repost_sid = append_sid("post_report.$phpEx?mode=view"); $u_reported_posts = "<a href=\"$repost_sid\" class=\"gensmall\"$link_color>$l_reported_posts</a><br />";

Воот...
А так свежий глаз в целом граммотный оказался

Вот только по поводу if ( !empty($info[$i]['user_icq']) ) я бы с вами поспорил

Спасибо за внимание, Вот готовые post_report.php & usercp_reputation.php
Спакойной ночи

**Carbofos** 06.01.2006 10:27

ETZel
IncominG
Спасибо большое за помощь!
Т.к. я успешно сдал матанализ, на выходных выдастся немного времени, сделаю очередной апдейт, учитывая ваши замечания.
Интересно, будет ли Изя-Мод ругаться если ему сказать перезаписать файлы, которые там уже есть?..

IncominG
Не против, если я использую твой фикс?

**ETZel** 06.01.2006 16:23

IncominG

А это что по вашему?
Код:
#FIND
$poster_id = $postrow['user_id'];

Не надо спорить с error_reporting(E_ALL); :mrgreen:

(в первой строке - использование, в последней - объявление):

Код: Выделить всё: if ( $poster_id == ANONYMOUS ) { message_die(GENERAL_MESSAGE, $lang['reputation_anonymous_no_reviews']); } $is_auth = array(); $is_auth = auth(AUTH_ALL, $postrow['forum_id'], $userdata); /* HH HH UU UU GGGGF EEEEEE * HH HH UU UU GG EE * HHHHHH UU UU GG GGG EEEE * HH HH UU UU GG GG EE ( to display related post ) */ // A HH HH UUUU GGGGG EEEEEE ( modified ) piece of code from viewtopic.php $poster_id = $postrow['user_id'];

IncominG писал(а):Ну во первых это не переменная, а текст, но если вам от этого легче, вводим:

Нет, это не просто текст, это индекс в массиве $lang (см. message_die). В "просто тексте" не употребляют подчеркивания.

IncominG писал(а):А так свежий глаз в целом граммотный оказался

фу как невежливо :twisted:

А местами я что, безграмотный?

IncominG, почему в fix.rar/usercp_reputation.php исчезли куски кода, создающие кнопки EMAIL/PM/ICQ ?

Баг в fix.rar/usercp_reputation.php

Код: Выделить всё: #FIND только второе вхождение!!! if ( $voter_id == $userdata['user_id'] ) #REPLACE # $voter_id был в блоке REPUTATION_EDIT, в блоке REPUTATION_DELETE объявлен $user_id if ( $user_id == $userdata['user_id'] )

Еще баги в usercp_reputation.php

Код: Выделить всё: #FIND for ($k = 0; $k < l; $k++ ) #REPLACE for ($k = 0; $k < $warnings; $k++ ) #FIND message_die(GENERAL_MESSAGE, '<a href="profile.php?mode=reputation&m=' . REPUTATION_VIEW . } } } #AFTER ADD else { $s_display_expired = ''; }

И поправка к старому:

Код: Выделить всё: #FIND $l_expire = ''; switch ( $info[$i]['modification'] ) #REPLACE $expire = $review = $l_expire = ''; switch ( $info[$i]['modification'] )

Добавлено спустя 3 часа 33 минуты 20 секунд:

post_report.php от IncominG отличается только 1) добавленной поддержкой мода user level, 2) исправленным if ($enable_html) и 3) неправильной последней строкой (там редирект, как в democracy011, а в democracy012 написано message_die). Странный какой-то фикс

Еще баги в usercp_reputation.php

Код: Выделить всё: #FIND for ($k = 0; $k < l; $k++ ) #REPLACE for ($k = 0; $k < $postrow['warnings']; $k++ ) #FIND $sql = 'SELECT r.post_id, r.voter_id, r.user_id, r.modification FROM ' . REPUTATION_TABLE . " r #REPLACE $sql = 'SELECT r.post_id, r.voter_id, r.user_id, r.modification, r.forum_id FROM ' . REPUTATION_TABLE . " r #FIND в 2-х местах if ( !$post_deleted ) #BEFORE ADD # чтобы небыло undefined variable, но можно и забить. этих undefined в phpBB все равно до*** (т.е. много) $approve = false; #FIND $forum_id = $row['forum_id']; #AFTER ADD # нужен дальше для $emailer->use_template($e_template, $user_lang); $user_lang = $row['user_lang']; #FIND $lang['reputation_warning_successfull'] . $post_action . '<br /><br />' . #REPLACE # нет такой переменной $post_action $lang['reputation_warning_successfull'] . '<br /><br />' . #FIND $row_color = ( !($i % 2) ) ? : $theme['td_color2']; $row_class = ( !($i % 2) ) ? : $theme['td_class2']; $template->assign_block_vars('postrow', array( 'ROW_COLOR' => '#' . $row_color, 'ROW_CLASS' => $row_class, #REPLACE # нет переменной $i, пост всего один $template->assign_block_vars('postrow', array( 'ROW_COLOR' => '#' . $theme['td_color1'], 'ROW_CLASS' => $theme['td_class1'], #FIND везде CRITICAL_ERROR #REPLACE # см. коммент ниже GENERAL_ERROR #FIND prepage_display(&$message, &$post_subject, $bbcode_uid, $postrow['enable_html'], $postrow['enable_smilies']); #BEFORE ADD // // Editing information // if ( $postrow[$i]['post_edit_count'] ) { $l_edit_time_total = ( $postrow[$i]['post_edit_count'] == 1 ) ? $lang['Edited_time_total'] : $lang['Edited_times_total']; $l_edited_by = '<br /><br />' . sprintf($l_edit_time_total, $poster, create_date($board_config['default_dateformat'], $postrow[$i]['post_edit_time'], $board_config['board_timezone']), $postrow[$i]['post_edit_count']); } else { $l_edited_by = ''; } #FIND 'QUOTE_IMG' => $quote_img, 'QUOTE' => $quote, #REPLACE # эти строки не нужны # далее два фикса для занесения IP в БД (добавил в INSERT поле poster_ip) #FIND $sql = 'INSERT INTO ' . REPUTATION_TABLE . " (modification, user_id, voter_id, post_id, forum_id, date) VALUES ('" . str_replace("\'", "''", $rep_mode) #REPLACE $sql = 'INSERT INTO ' . REPUTATION_TABLE . " (modification, user_id, voter_id, post_id, forum_id, poster_ip, date) VALUES ('" . str_replace("\'", "''", $rep_mode) . "', '{$row['user_id']}', '{$userdata['user_id']}', '$post_id', '$forum_id', '$user_ip', " . time() . ")"; #FIND $sql = 'INSERT INTO ' . REPUTATION_TABLE . " (modification, user_id, voter_id, post_id, forum_id, date, expire) VALUES ('$modification', '" . $row['user_id'] #REPLACE $sql = 'INSERT INTO ' . REPUTATION_TABLE . " (modification, user_id, voter_id, post_id, forum_id, poster_ip, date, expire) VALUES ('$modification', '" . $row['user_id'] . "', '" . $userdata['user_id'] . "', '$sql_post_id', '$forum_id', '$user_ip', '$current_time', '$expire')";

Еще технические замечания:
1) Много messages_die(CRITIACL_ERROR... - лучше исп. GENERAL_ERROR, поскольку CRITICAL_ERROR - это для редких фатальных случаев, облом sql запроса таковым не является (по крайней мере по мнению Paul S. Owen)
2) в usercp_reputation.php

Код: Выделить всё: $user_warnings = '<div style="margin:4px 1px 4px 1px"><a href="' . $temp_url . POST_USERS_URL . '=' . $postrow['user_id'] . '&m=0">';

<div style="margin:4px 1px 4px 1px"> лучше убрать из кода и перенести в шаблон, поскольку он вписывается не во все стили (например, в мой не вписался

).

**IncominG** 07.01.2006 2:11

Carbofos писал(а):Не против, если я использую твой фикс?

Сочту за честь

ETZel писал(а):Не надо спорить с error_reporting(E_ALL); (в первой строке - использование, в последней - объявление):

У меня тож стоит и все замечателдьно работает. Переменная объявлена - не стоит спорить...

ETZel писал(а):Нет, это не просто текст, это индекс в массиве $lang (см. message_die). В "просто тексте" не употребляют подчеркивания.

Даже в стандартном phpbb употреблено, так что давайте не будем...

ETZel писал(а):фу как невежливо А местами я что, безграмотный?

Есть чуток

ETZel писал(а):post_report.php от IncominG отличается только 1) добавленной поддержкой мода user level, 2) исправленным if ($enable_html) и 3) неправильной последней строкой (там редирект, как в democracy011, а в democracy012 написано message_die). Странный какой-то фикс

Редирект оставлен намеренно!
Остальное - будем фиксить... Выложу готовый архив так-же...

ETZel писал(а): EMAIL/PM/ICQ ?

Ой это моя редакция извиняюсь

но другого фикшеного юзерцп у меня нет

так что... могу приложить мною отредактированный profile_reputation

**lusever** 07.01.2006 3:28

Быстрее следующую версию выкладывайте. Все ждут:!:
Спасибо, кстати, за мод.

[BETA] Democracy MOD 0.2.1 (reputation + warnings + reports)

Кто сейчас на конференции