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

[Carbofos]

Ой, ой, помедленнее плиз, я не успеваю!
Только-только сделал обновление, а тут порция дальших багов Как теперь с ними быть?..

Обзоры постов и предупреждения/баны теперь можно смотреть из профиля отдельно.
Добавил совместимость с Глобал Мод и Юзер Левел и в остальных файлах.
Из новых фишек все

К размышлению: блоки 'warn' и 'rep' в profile_view_reputation.tpl, наверное, не стоило объединять

поздно! а раскрасить по-разному и так можно.

Да, и плюньте вы на эту необъявленную переменную... я ее переставил куда надо

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

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

Так... считайте, что официально объявляю об апдейте только сейчас (еще немного его переделал - устранил баг с неудалением из бана и с =$post_id)

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

Да, кстати, узнал что за unprepare_message такое. Это 100% не надо, оно & превращает в &, а на это потом матерятся валидаторы.

[Поручик]

Carbofos
а как бы change.log на post_report.php и usercp_reputation.php
У меня они заточены под наш форум. Просто так не зальешь

[ETZel]

Поручик, погодите пока ставить democracy013 В нем не все фиксы и пара новых багов есть. Я сейчас внесу все старые фиксы и будет democracy014

Про changelog:
Универсальный способ (которым я, например, делаю апдейты attach мода, который у меня сильно измнен): берешь любую утилиту для сравнения файлов, скармливаешь ей старый и новый usercp_report.php, а в редакторе открываешь свой usercp_reputation.php и последовательно переносишь все изменения. (Total Commender хорош для этого дела). Если понимаешь patch - можно сделать diff.
"Готовый" changelog, который вы просите, получается одним из вышеозначенных способов.
Для визуального сравнения файлов, кроме TC, есть freeware утилиты ExamDiff (простенькая) и WinMerge (более навороченная). Обе умеют делать diff.

[Поручик]

погодите пока ставить democracy013

Поздно.

пара новых багов есть

Эт точно! (С). Ну мля, я всю ночь не спал.

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

и Siava тоже

[Siava]

и Siava тоже

Что-то у меня пока нет времени баги искать, мне хватило двухкратного сноса Yellow Cards для теста этого мода у себя
Может завтра на чистом форуме погляжу.. но честно говоря, пока я занял позицию хитрого выжидателя готовенького релиза

[ETZel]

пока я занял позицию хитрого выжидателя готовенького релиза

вот! именно об этом я и говорю. Зачем тратить время на багфиксинг и установку заведомо нестабильной версии (0.1.3)? Если я сейчас как раз занимаюсь исправлением всех багов и подготовкой 0.1.4

[Поручик]

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

[lusever]

Ну вы тут как хотите, но это я исправил, что б хотя бы работало!, и делюсь, чтоб в следующей версии багов не было.

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

post_report.php
---[ line 542 ]---
      message_die(GENERAL_MESSAGE, $lang['reputation_report_deleted'] . '<br /><br />' . sprintf($lang['reputation_msg_back_to_reports'], '<a href="' . append_sid($phpbb_root_path . 'post_report.' . $phpEx . '?mode=view') . '">', '</a>'));
---[ REPLACE ]---
      message_die(GENERAL_MESSAGE, $lang['reputation_report_deleted'] . '<br /><br />' . sprintf($lang['reputation_msg_back_to_reports'], '<a href="' . append_sid('post_report.' . $phpEx . '?mode=view') . '">', '</a>'));


viewtopic.php
---[ INSTRUCTION ]---
/////////////democracy////////////////////
      if ( $board_config['reputation_enable_reports'] )
      {
         if( $userdata['user_id'] != ANONYMOUS )
         {
            $temp_url = "post_report.$phpEx?mode=report&amp;" . POST_POST_URL . "=" . $postrow[$i]['post_id'] . "&amp;sid=" . $userdata['session_id'];
            $reportpost_img = '<a href="' . $temp_url . '"><img src="' . $images['report_post'] . '" alt="' . $lang['reputation_report_post'] . '" title="' . $lang['reputation_report_post'] . '" border="0" /></a>';
            $reportpost = '<a href="' . $temp_url . '">' . $lang['reputation_report_post'] . '</a>';
         }
      }
///////////////////////////////////////////

---[ REPLACE ]---
/////////////democracy////////////////////
      if ( $board_config['reputation_enable_reports'] )
      {
         if( $userdata['user_id'] != ANONYMOUS )
         {
            $temp_url = append_sid("post_report.$phpEx?mode=report&amp;" . POST_POST_URL . '=' . $postrow[$i]['post_id'] . '&amp;sid=' . $userdata['session_id']);
            $reportpost_img = '<a href="' . $temp_url . '"><img src="' . $images['report_post'] . '" alt="' . $lang['reputation_report_post'] . '" title="' . $lang['reputation_report_post'] . '" border="0" /></a>';
            $reportpost = '<a href="' . $temp_url . '">' . $lang['reputation_report_post'] . '</a>';
         }
      }
///////////////////////////////////////////

---[ FIND ]---
            $temp_url = "profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_WARNING . "&amp;" . POST_POST_URL . "=" . $postrow[$i]['post_id'] . "&amp;sid=" . $userdata['session_id'];
---[ REPLACE ]---
            $temp_url = append_sid("profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_WARNING . '&amp;' . POST_POST_URL . '=' . $postrow[$i]['post_id'] . '&amp;sid=' . $userdata['session_id']);

---[ FIND ]---
               $temp_url = "profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_BAN . "&amp;" . POST_POST_URL . "=" . $postrow[$i]['post_id'] . "&amp;sid=" . $userdata['session_id'];
---[ REPLACE ]---
               $temp_url = append_sid("profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_BAN . '&amp;' . POST_POST_URL . '=' . $postrow[$i]['post_id'] . '&amp;sid=' . $userdata['session_id']);

usercp_perutation.php
---[ FIND ]---
   $sql = 'SELECT r.post_id, r.voter_id, r.user_id, r.modification FROM ' . REPUTATION_TABLE . ' r
      LEFT JOIN ( SELECT post_id, forum_id FROM ' . POSTS_TABLE . ' WHERE 1 ) p ON r.post_id = p.post_id
         WHERE r.id = ' . $review_id;
---[ REPLACE ]---
   $sql = 'SELECT r.post_id, r.voter_id, r.user_id, r.modification FROM ' . REPUTATION_TABLE . ' r,
      ' . POSTS_TABLE . ' p
         WHERE r.post_id = p.post_id AND r.id = ' . $review_id;

---[ FIND ]---
         $temp_url = "profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_DELETE . '&' . POST_REVIEWS_URL . '=' . $info[$i]['id'] . "&amp;sid=" . $userdata['session_id'] . $u_ret;
---[ REPLACE ]---
         $temp_url = append_sid("profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_DELETE . '&amp;' . POST_REVIEWS_URL . '=' . $info[$i]['id'] . '&amp;sid=' . $userdata['session_id'] . $u_ret);
---[ FIND ]---
            $temp_url = "post_report.$phpEx?mode=report&amp;" . POST_REVIEWS_URL . "=" . $info[$i]['id'] . "&amp;sid=" . $userdata['session_id'];
---[ REPLACE ]---
            $temp_url = append_sid("post_report.$phpEx?mode=report&amp;" . POST_REVIEWS_URL . '=' . $info[$i]['id'] . '&amp;sid=' . $userdata['session_id']);
---[ FIND ]---
            $temp_url = "profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_WARNING . "&amp;" . POST_POST_URL . "=" . $postrow['post_id'] . "&amp;sid=" . $userdata['session_id'];
---[ REPLACE ]---
            $temp_url = append_sid("profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_WARNING . '&amp;' . POST_POST_URL . '=' . $postrow['post_id'] . '&amp;sid=' . $userdata['session_id']);
---[ FIND ]---
               $temp_url = "profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_BAN . "&amp;" . POST_POST_URL . "=" . $postrow['post_id'] . "&amp;sid=" . $userdata['session_id'];
---[ REPLACE ]---
               $temp_url = append_sid("profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_BAN . '&amp;' . POST_POST_URL . '=' . $postrow['post_id'] . '&amp;sid=' . $userdata['session_id']);
---[ FIND ]---
         $temp_url = "modcp.$phpEx?mode=ip&amp;" . POST_POST_URL . "=" . $postrow['post_id'] . "&amp;" . POST_TOPIC_URL . "=" . $postrow['topic_id'] . "&amp;sid=" . $userdata['session_id'];
---[ REPLACE ]---
         $temp_url = append_sid("modcp.$phpEx?mode=ip&amp;" . POST_POST_URL . '=' . $postrow['post_id'] . '&amp;' . POST_TOPIC_URL . '=' . $postrow['topic_id'] . '&amp;sid=' . $userdata['session_id']);
---[ FIND ]---
         $temp_url = "posting.$phpEx?mode=delete&amp;" . POST_POST_URL . "=" . $postrow['post_id'] . "&amp;sid=" . $userdata['session_id'];
---[ REPLACE ]---
         $temp_url = append_sid("posting.$phpEx?mode=delete&amp;" . POST_POST_URL . '=' . $postrow['post_id'] . '&amp;sid=' . $userdata['session_id']);
---[ FIND ]---
            $temp_url = "posting.$phpEx?mode=delete&amp;" . POST_POST_URL . "=" . $postrow['post_id'] . "&amp;sid=" . $userdata['session_id'];
---[ REPLACE ]---
            $temp_url = append_sid("posting.$phpEx?mode=delete&amp;" . POST_POST_URL . '=' . $postrow['post_id'] . '&amp;sid=' . $userdata['session_id']);
---[ FIND ]---
               $temp_url = "warning.$phpEx?m=4&amp;" . POST_POST_URL . "=" . $postrow['post_id'] . "&amp;sid=" . $userdata['session_id'];
---[ REPLACE ]---
               $temp_url = append_sid("warning.$phpEx?m=4&amp;" . POST_POST_URL . '=' . $postrow['post_id'] . '&amp;sid=' . $userdata['session_id']);



Добавлено спустя 2 минуты 11 секунд:

В кратце, у многих ссылок не хватает функции append_sid.
И одна бага с SQL запросом которая на MySQL 4.0... не захотела работать - по быстрому переписал.

[Xpert]

lusever
Вот как раз что касается ссылок, то код автора верен. В тех разделах, которые требуют 100%-го получения sid, его надо добавлять вручную... Это напрямую влияет на безопасность.

[Никто]

В кратце, у многих ссылок не хватает функции append_sid.

Чтобы у вас не было проблем с потерей сессий настройте куки правильно, а не переписывайте под мода

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

ETZel
Будут фиксы для 0,1,3?
А то я жду и не ставлю

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

Скопировал новую демократию заменив файлы и...

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

Parse error: syntax error, unexpected T_STRING, expecting ')' in /home/users/o/oowil/docs/forum/includes/usercp_reputation.php on line 1035

закомментил строчку чтоб работало пока что...
Вот строка:

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

'L_NO_REVIEWS' => '<a href="' append_sid("profile.$phpEx?mode=reputation&amp;m=" . REPUTATION_VIEW . '&amp;exp=1&amp;' . POST_POST_URL . "=$post_id") . "#$post_id">" . $lang['reputation_display_expired'] . '</a>')

Добавлено спустя 17 минут 4 секунды:

да и...
Я так и не заметил разделения между репутацией и варнами/банами...

[ETZel]

Никто
Фиксов не будет, будет Democracy 0.1.4
Но чтобы его опубликовать нужен patch2mod (которым я сейчас и занимаюсь).
Про строку с ошибкой: том точка нужна перед append_sid.

[Никто]

ETZel
нехорошо выпускать без автора мода... не красиво это... =( лучше выложи фиксы и автор напишет как надо...
2) что с:

да и...
Я так и не заметил разделения между репутацией и варнами/банами...

?
3) зайди в асю

[ETZel]

нехорошо выпускать без автора мода... не красиво это... =( лучше выложи фиксы и автор напишет как надо...

Бууу... я с автором уже договорился Что я ему сошлю все фиксы, а он их выложит.

Я так и не заметил разделения между репутацией и варнами/банами...

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

[Никто]

Попробуйте поставить 2 варна подряд и попробуйте верхний исправить... Вот это начинаются глюки всем глюкам
Бан на время пропал
И ваще чем 0,1,3 лучше 0,1,2? по мне так старый был лучше! Пошел ставить назад!

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

Да и еще! Переходишь по ссылке на варн и...

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

Общая ошибка   
   
Не указан пользователь!

[Князь Тьмы]

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