Skip to content

Commit d39386c

Browse files
committed
Fix bans page getPlayerNameByAccount + getPlayerNameById
1 parent b5bbae6 commit d39386c

File tree

1 file changed

+22
-28
lines changed

1 file changed

+22
-28
lines changed

system/pages/bans.php

+22-28
Original file line numberDiff line numberDiff line change
@@ -167,37 +167,31 @@ function getBanType($typeId)
167167

168168
function getPlayerNameByAccount($id)
169169
{
170-
global $vowels, $ots, $db;
171-
if(is_numeric($id))
172-
{
173-
$player = new OTS_Player();
174-
$player->load($id);
175-
if($player->isLoaded())
176-
return $player->getName();
177-
else
178-
{
179-
$playerQuery = $db->query('SELECT `id` FROM `players` WHERE `account_id` = ' . $id . ' ORDER BY `lastlogin` DESC LIMIT 1;')->fetch();
170+
global $db;
171+
if(!is_numeric($id)) {
172+
return '';
173+
}
180174

181-
$tmp = "*Error*";
182-
/*
183-
$acco = new OTS_Account();
184-
$acco->load($id);
185-
if(!$acco->isLoaded())
186-
return "Unknown name";
175+
$playerQuery = $db->query('SELECT `name` FROM `players` WHERE `account_id` = ' . $id . ' ORDER BY `lastlogin` DESC LIMIT 1;');
176+
if ($playerQuery->rowCount() == 0) {
177+
return "*Error*";
178+
}
187179

188-
foreach($acco->getPlayersList() as $p)
189-
{
190-
$player= new OTS_Player();
191-
$player->find($p);*/
192-
$player->load($playerQuery['id']);
193-
//echo 'id gracza = ' . $p . '<br/>';
194-
if($player->isLoaded())
195-
$tmp = $player->getName();
196-
// break;
197-
//}
180+
$playerQuery = $playerQuery->fetch();
181+
return $playerQuery['name'];
182+
}
183+
184+
function getPlayerNameById($id)
185+
{
186+
if (!is_numeric($id)) {
187+
return '';
188+
}
198189

199-
return $tmp;
200-
}
190+
global $db;
191+
$playerQuery = $db->query('SELECT `name` FROM `players` WHERE `id` = ' . $id);
192+
if ($playerQuery->rowCount()) {
193+
$player = $playerQuery->fetch(PDO::FETCH_ASSOC);
194+
return $player['name'];
201195
}
202196

203197
return '';

0 commit comments

Comments
 (0)