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

Carbofos · Сообщение **Carbofos** » 09.01.2006 17:32

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

Как теперь с ними быть?..

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

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

поздно!

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

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

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

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

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

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

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

Поручик · Сообщение **Поручик** » 10.01.2006 2:32

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

ETZel · Сообщение **ETZel** » 10.01.2006 12:23

Поручик, погодите пока ставить democracy013

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

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

Поручик · Сообщение **Поручик** » 10.01.2006 17:41

ETZel писал(а):погодите пока ставить democracy013

Поздно.

ETZel писал(а):пара новых багов есть

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

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

и Siava тоже

Сообщение **Siava** » 11.01.2006 0:56

Поручик писал(а):и Siava тоже

Что-то у меня пока нет времени баги искать, мне хватило двухкратного сноса Yellow Cards для теста этого мода у себя

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

ETZel · Сообщение **ETZel** » 11.01.2006 1:41

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

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

Поручик · Сообщение **Поручик** » 11.01.2006 1:53

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

lusever · Сообщение **lusever** » 14.01.2006 8:51

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

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

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&" . POST_POST_URL . "=" . $postrow[$i]['post_id'] . "&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&" . POST_POST_URL . '=' . $postrow[$i]['post_id'] . '&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&m=" . REPUTATION_WARNING . "&" . POST_POST_URL . "=" . $postrow[$i]['post_id'] . "&sid=" . $userdata['session_id'];
---[ REPLACE ]---
				$temp_url = append_sid("profile.$phpEx?mode=reputation&m=" . REPUTATION_WARNING . '&' . POST_POST_URL . '=' . $postrow[$i]['post_id'] . '&sid=' . $userdata['session_id']);

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

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

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

Сообщение **Xpert** » 14.01.2006 9:30

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

Никто · Сообщение **Никто** » 14.01.2006 15:50

lusever писал(а):В кратце, у многих ссылок не хватает функции 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&m=" . REPUTATION_VIEW . '&exp=1&' . POST_POST_URL . "=$post_id") . "#$post_id">" . $lang['reputation_display_expired'] . '</a>')

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

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

ETZel · Сообщение **ETZel** » 14.01.2006 15:52

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

Никто · Сообщение **Никто** » 14.01.2006 15:59

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

Никто писал(а):да и...
Я так и не заметил разделения между репутацией и варнами/банами...

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

ETZel · Сообщение **ETZel** » 14.01.2006 17:22

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

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

Никто писал(а):Я так и не заметил разделения между репутацией и варнами/банами...

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

Никто · Сообщение **Никто** » 15.01.2006 4:30

Попробуйте поставить 2 варна подряд и попробуйте верхний исправить... Вот это начинаются глюки всем глюкам

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

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

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

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

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

Князь Тьмы · 15.01.2006 4:47

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