[BETA]BannedUser

[deadromeo]

Название расширения: BannedUser
Описание расширения: Расширение добавит на ваш форум Бан-лист, а так же в профилях заблокированных пользователях будут отображаться соответствующие отметки.
Язык: ru

banneduser1.0.5.rar

[xaocZ]

Имхо конечно, разницы в принципе нет, но зачем в стилях:

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

<span class="corners-top"><span></span></span>

и т.п.
Нету ведь в 3.1* закруглений на картинках, можно было и вырезать.

[Romnik]

Косяк "На форуме заблокированных пользователей: 260". Откуда?
На форуме всего 7 пользователей и заблокированных 0.

В мини-профиле информация о блокировке отсутствует.

[Anvar]

В listener.php лишнее лучше удалить.

Скрытый текст

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

global $auth, $db;
	global $template, $user;

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

\phpbb\config\config $config,
\phpbb\auth\auth $auth, 
\phpbb\user $user,

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

$this->config = $config;
$this->auth = $auth;
$this->user = $user;

Событие core.viewtopic_modify_post_row, расположено в цикле. Запросы к базе будут выполнены столько раз, сколько постов в сообщении.

Используем ещё одно событие в listener:

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

'core.viewtopic_modify_post_data'	=> 'viewtopic_modify_post_data',

Заменяем свою функцию banmem2 на:

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

	public function viewtopic_modify_post_data($event)
	{
		$user_ids = array();
		$rowset = $event['rowset'];
		$post_list = $event['post_list'];
		for ($i = 0, $end = sizeof($post_list); $i < $end; ++$i)
		{
			if (!isset($rowset[$post_list[$i]]))
			{
				continue;
			}
			$row = $rowset[$post_list[$i]];
			$poster_id = $row['user_id'];
			if ($poster_id != ANONYMOUS && !$row['foe'] && !$row['hide_post'])
			{
				$user_ids[] = $poster_id;
			}
			unset($rowset[$post_list[$i]]);
		}

		if (sizeof($user_ids))
		{
			$sql = 'SELECT ban_userid
				FROM ' . BANLIST_TABLE . '
				WHERE ' . $this->db->sql_in_set('ban_userid', $user_ids) . '
					AND ban_exclude = 0';
			$result = $this->db->sql_query($sql);
			while ($row = $this->db->sql_fetchrow($result))
			{
				$this->ban_userid[$row['ban_userid']] = $row['ban_userid'];
			}
			$this->db->sql_freeresult($result);
		}
	}
	public function banmem2($event)
	{
		if (!empty($this->ban_userid[$event['poster_id']]))
		{
			$post_row = array(
				'USERB_ID' => $this->ban_userid[$event['poster_id']],
			);
			$event['post_row'] += $post_row;
		}
	}

Теперь будет один запрос к базе вместо 10-15.

[deadromeo]

xaocZ, спасибо, исправим
Romnik, странно, данные берутся то из таблицы ban_list, а точнее считаются все записи по количеству, форум же у вас чистый, из коробки, не конвертированный?
Anvar, спасибо за ценные советы, потиху опыта набираюсь, не все с первого раза выходит гладко

Добавлено спустя 11 минут 37 секунд:
Касательно подключения функции - после предложенных правок идет ошибка

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

SQL ERROR [ mysql4 ]

Unknown column 'user_id' in 'where clause' [1054]

SQL

SELECT ban_userid FROM phpbb_banlist WHERE ban_userid = user_id IN ('2', '48', '2', '50', '2', '50', '2', '50', '2', '53') AND ban_exclude = 0

[Romnik]

форум же у вас чистый, из коробки, не конвертированный?

С нуля.

[deadromeo]

Печально, вроде и запрос прозрачный с выборкой

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

$sql = 'SELECT COUNT(ban_userid) as total_banned_users
			FROM ' . BANLIST_TABLE . '
			WHERE ban_exclude = 0';
		$result = $this->db->sql_query($sql);
		$row = $this->db->sql_fetchrow($result);
		$total_banned_users = (int) $row['total_banned_users'];
		$this->db->sql_freeresult($result);	
		
	$this->template->assign_vars(array(
		
		'TOTAL_BANNED_USERS'	=> $total_banned_users,
	));

Где тут беда не могу найти, обычный подсчет ....

[Romnik]

Если блокирую одного юзера, то отображает: 261.
Откуда берутся 260, тоже пока не найду.

Я думаю красный (броский) блок надо заменить на кнопку под аватарой.

[djkent]

У меня вообще при просмотре сколько людей в бане выходит такой косяк

2014-11-29_210546.png

[Татьяна5]

djkent, скопируйте эти белые строчки, и вставьте их в сообщение текстом в [code][/code]. На скрине не разобрать что написано

[djkent]

Татьяна5

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

[phpBB Debug] PHP Notice: in file [ROOT]/ext/phpbbworld/banneduser/controller/banneduser.php on line 125: Undefined variable: user
[phpBB Debug] PHP Notice: in file [ROOT]/ext/phpbbworld/banneduser/controller/banneduser.php on line 125: Trying to get property of non-object
[phpBB Debug] PHP Notice: in file [ROOT]/ext/phpbbworld/banneduser/controller/banneduser.php on line 130: Undefined variable: user
[phpBB Debug] PHP Notice: in file [ROOT]/ext/phpbbworld/banneduser/controller/banneduser.php on line 130: Trying to get property of non-object

[Romnik]

В моём косяке.
Я думаю не из этого-ли расширения ноги ростут - "Activity Stats 1.2.4-DEV "

[Anvar]

Unknown column 'user_id' in 'where clause' [1054]

Опечатка. Изменил user_id на ban_userid.

Тот код сообразил по быстрому в качестве примера и не проверял.

[djkent]

Anvar А что мне сделать как исправить эти ошибки которые я указал?

[Romnik]

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

Ждать когда автор исправит...