MySql Query, Top Fragger... Web Page.

[Morrison]

Hello, I have a problem.
What is wrong in this Query?

$fragger = $SQL->query('SELECT COUNT(1) frags, `players`.`name` FROM `players`, `player_deaths` WHERE `is_player` = 1 AND `player_deaths`.`player_id` = `players`.`id` GROUP BY `killed_by` ORDER BY ' . $SQL->fieldName('frags') . ' DESC, ' . $SQL->fieldName('name') . ' ASC LIMIT 30;')->fetch();

This Query show me the most Noob player, just i need this show me the most Top Fragger/Killer.
Example player Stin Ninja he die 4 time for another players.

Most information: http://tibierots.sytes.net/

Spoiler: Image:

Another Optional Muve "MOST POWERFULL GUILDS" to middle. and showoff moft beautifull. THX

All code from latesnews.php

    $query = $SQL->query("SELECT * FROM `players` WHERE `group_id` = 1 ORDER BY `experience` DESC;")->fetch();
    $query2 = $SQL->query('SELECT `id`, `name` FROM `players` ORDER BY `id` DESC LIMIT 1;')->fetch();
    $housesfree = $SQL->query('SELECT COUNT(*) FROM `houses` WHERE `owner`=0;')->fetch();
    $housesrented = $SQL->query('SELECT COUNT(*) FROM `houses` WHERE `owner`=1;')->fetch();
    $players = $SQL->query('SELECT COUNT(*) FROM `players` WHERE `id`>0;')->fetch();
    $accounts = $SQL->query('SELECT COUNT(*) FROM `accounts` WHERE `id`>0;')->fetch();
    $fragger = $SQL->query('SELECT COUNT(1) frags, `players`.`name` FROM `players`, `player_deaths` WHERE `is_player` = 1 AND `player_deaths`.`player_id` = `players`.`id` GROUP BY `killed_by` ORDER BY ' . $SQL->fieldName('frags') . ' DESC, ' . $SQL->fieldName('name') . ' ASC LIMIT 30;')->fetch();
    // $fragger = $SQL->query('SELECT COUNT(1) frags, `players`.`name` FROM `players`, `player_deaths` WHERE `is_player` = 1 AND `player_deaths`.`player_id` = `players`.`id` GROUP BY `killed_by` ORDER BY `frags` DESC;')->fetch();
    // $guilds = $SQL->query('SELECT COUNT(*) FROM `guilds` WHERE `id`>0;')->fetch();
    ///End Queries ///
  
  
        // top kills - guilds
$main_content .= '<div class="NewsHeadline">
    <div class="NewsHeadlineBackground" style="background-image:url(' . $layout_name . '/images/news/newsheadline_background.gif)">
        <table border="0">
            <tr>
                <td style="text-align: center; font-weight: bold;">
                    <font color="white">Most powerfull guilds</font>
                </td>
            </tr>
        </table>
    </div>
</div>
<table border="0" cellspacing="3" cellpadding="4" width="100%">
    <tr>';

  
  
  
$guilds = $SQL->query('SELECT `g`.`id` AS `id`, `g`.`name` AS `name`, COUNT(`g`.`name`) as `frags` FROM `players` p LEFT JOIN `player_deaths` pd ON `pd`.`killed_by` = `p`.`name` LEFT JOIN `guild_membership` gm ON `p`.`id` = `gm`.`player_id` LEFT JOIN `guilds` g ON `gm`.`guild_id` = `g`.`id` WHERE `pd`.`unjustified` = 1 GROUP BY `name` ORDER BY `frags` DESC, `name` ASC LIMIT 0, 5;')->fetchAll();
  
foreach($guilds as $guild)
    $main_content .= '        <td style="width: 25%; text-align: center;">
            <a href="?subtopic=guilds&action=show&guild=' . $guild['id'] . '">' . $guild['name'] . '</a><br />' . $guild['frags'] . ' kills
        </td>';

$main_content .= '    </tr>
</table>';
  
  
    $main_content .= '<table bgcolor='.$config['site']['darkborder'].' border=0 cellpadding=4 cellspacing=1 width=100%>
    <tr bgcolor='. $config['site']['vdarkborder'] .'><td align="center" class=white colspan=1><b>Welcome to '.$config['server']['serverName'].'</b></td></tr>
    <tr><td><table border=0 cellpadding=1 cellspacing=1 width=100%>

    <tr bgcolor='. $config['site']['lightborder'] .'><td><center><b>Last joined us: </b><a href="?subtopic=characters&name='.urlencode($query2['name']).'">'.$query2['name'].'</a>, <b>player number </b>'.$query2['id'].'. <b>Welcome and wish you a nice game!</b></center></td></tr>
  
    <tr bgcolor='. $config['site']['lightborder'] .'><td><center><b>Currently, the best player on the server is:</b> <a href="index.php?subtopic=characters&name='.urlencode($query['name']).'"> '.$query['name'].'</a> ('.urlencode($query['level']).').<b> Congratulations!</b></center></td></tr>
  
    <tr bgcolor='. $config['site']['lightborder'] .'><td><center><b>Top Killer:</b> <a href="index.php?subtopic=characters&name='.urlencode($fragger['name']).'"> '.$fragger['name'].'</a> ('.urlencode($fragger['frags']).').<b> Unstoppable!</b></center></td></tr>
  
    <table border=0 cellpadding=0 cellspacing=1 width=100%>
  
      <tr bgcolor='. $config['site']['lightborder'] .'><td><center><b>Free Houses:</b> '.$housesfree[0].'</center></td>
    
    <td><center><b>Rented Houses:</b> '.$housesrented[0].'</center></td></tr>
  
    <tr bgcolor='. $config['site']['lightborder'] .'><td><center><b>Accounts in database: </b>'.$accounts[0].'</center></td>
  
    <td><center><b>Players in database: </b>'.$players[0].'</center></td></tr>

    </table></td></tr></table><br />';