From 7abeb203d8b6d537a55b0dd9a1ccac32e6f948ec Mon Sep 17 00:00:00 2001 From: Matthijs Kuiper Date: Sat, 4 Feb 2017 23:57:49 +0100 Subject: Fix MySQL ONLY_FULL_GROUP_BY errors --- html/pages/match_player.php | 125 +++++++++++++++++++++++--------------------- 1 file changed, 66 insertions(+), 59 deletions(-) (limited to 'html/pages/match_player.php') diff --git a/html/pages/match_player.php b/html/pages/match_player.php index 4cc73a0..63a3787 100755 --- a/html/pages/match_player.php +++ b/html/pages/match_player.php @@ -2,23 +2,28 @@ $mid = preg_replace('/\D/', '', $_GET[mid]); $pid = preg_replace('/\D/', '', $_GET[pid]); -$r_infos = small_query("SELECT p.playerid, p.country, pi.name, pi.banned, p.gid, g.name AS gamename FROM uts_player p, uts_pinfo pi, uts_games g WHERE p.gid = g.id AND p.pid = pi.id AND p.pid = '$pid' AND matchid = '$mid' LIMIT 0,1;"); +$r_infos = small_query("SELECT p.playerid, p.country, pi.name, pi.banned, p.gid, g.name AS gamename + FROM uts_player p, uts_pinfo pi, uts_games g + WHERE p.gid = g.id AND p.pid = pi.id AND p.pid = '$pid' + AND matchid = '$mid' + LIMIT 0,1;"); if (!$r_infos) { - echo "Unable to retrieve data!"; - include("includes/footer.php"); - exit; + echo "Unable to retrieve data!"; + include("includes/footer.php"); + exit; } + if ($r_infos['banned'] == 'Y') { - if (isset($is_admin) and $is_admin) { - echo "Warning: Banned player - Admin override
"; - } else { - echo "Sorry, this player has been banned!"; - include("includes/footer.php"); - exit; - } + if (isset($is_admin) and $is_admin) { + echo "Warning: Banned player - Admin override
"; + } else { + echo "Sorry, this player has been banned!"; + include("includes/footer.php"); + exit; + } } - + $playerid = $r_infos['playerid']; $playername = $r_infos['name']; $country = $r_infos['country']; @@ -28,9 +33,10 @@ $gid = $r_infos['gid']; echo' - +
Individual Match Stats for '.FlagImage($country) .' '. htmlentities($playername) .' - '. RankImageOrText($pid, $playername, NULL, $gid, $gamename, true, '(%IT% in %GN% with %RP% ranking points)') .' - Individual Match Stats for + '.FlagImage($country) .' '. htmlentities($playername) .' + '. RankImageOrText($pid, $playername, NULL, $gid, $gamename, true, '(%IT% in %GN% with %RP% ranking points)') .' +

'; @@ -56,19 +62,20 @@ echo ' '; $r_gsumm = zero_out(small_query("SELECT gamescore, frags, SUM(frags+suicides) AS kills, deaths, suicides, teamkills, eff, accuracy, ttl, gametime, spree_kill, spree_rampage, spree_dom, spree_uns, spree_god -FROM uts_player WHERE matchid = $mid AND pid = '$pid' -GROUP BY pid")); + FROM uts_player + WHERE matchid = $mid AND pid = '$pid' + GROUP BY pid, gamescore, frags, deaths, suicides, teamkills, eff, accuracy, ttl, gametime, spree_kill, spree_rampage, spree_dom, spree_uns, spree_god")); echo' - '.$r_gsumm[frags].' - '.$r_gsumm[kills].' - '.$r_gsumm[deaths].' - '.$r_gsumm[suicides].' - '.$r_gsumm[eff].' - '.$r_gsumm[accuracy].' - '.$r_gsumm[ttl].' - '.GetMinutes($r_gsumm[gametime]).' + '.$r_gsumm[frags].' + '.$r_gsumm[kills].' + '.$r_gsumm[deaths].' + '.$r_gsumm[suicides].' + '.$r_gsumm[eff].' + '.$r_gsumm[accuracy].' + '.$r_gsumm[ttl].' + '.GetMinutes($r_gsumm[gametime]).' '; echo' @@ -96,30 +103,30 @@ echo' '; $r_gsumm = zero_out(small_query("SELECT spree_double, spree_multi, spree_ultra, spree_monster, spree_kill, spree_rampage, spree_dom, spree_uns, spree_god -FROM uts_player WHERE matchid = $mid AND pid = '$pid' -GROUP BY pid")); + FROM uts_player + WHERE matchid = $mid AND pid = '$pid' + GROUP BY pid, spree_double, spree_multi, spree_ultra, spree_monster, spree_kill, spree_rampage, spree_dom, spree_uns, spree_god")); $sql_firstblood = small_query("SELECT firstblood FROM uts_match WHERE id = $mid"); -IF ($sql_firstblood[firstblood] == $pid) { - $firstblood = "Yes"; +if ($sql_firstblood[firstblood] == $pid) { + $firstblood = "Yes"; } else { - $firstblood = "No"; + $firstblood = "No"; } - - echo' +echo' - '.$firstblood.' - '.$r_gsumm[spree_double].' - '.$r_gsumm[spree_multi].' - '.$r_gsumm[spree_ultra].' - '.$r_gsumm[spree_monster].' - '.$r_gsumm[spree_kill].' - '.$r_gsumm[spree_rampage].' - '.$r_gsumm[spree_dom].' - '.$r_gsumm[spree_uns].' - '.$r_gsumm[spree_god].' + '.$firstblood.' + '.$r_gsumm[spree_double].' + '.$r_gsumm[spree_multi].' + '.$r_gsumm[spree_ultra].' + '.$r_gsumm[spree_monster].' + '.$r_gsumm[spree_kill].' + '.$r_gsumm[spree_rampage].' + '.$r_gsumm[spree_dom].' + '.$r_gsumm[spree_uns].' + '.$r_gsumm[spree_god].'
'; @@ -129,23 +136,23 @@ weaponstats($mid, $pid); $r_pings = small_query("SELECT lowping, avgping, highping FROM uts_player WHERE pid = $pid and matchid = $mid and lowping > 0"); if ($r_pings and $r_pings['lowping']) { -echo ' -
- - - - - - - - - - - - - - -
Pings
MinAvgMax
'.ceil($r_pings['lowping']).''.ceil($r_pings['avgping']).''.ceil($r_pings['highping']).'
'; + echo ' +
+ + + + + + + + + + + + + + +
Pings
MinAvgMax
'.ceil($r_pings['lowping']).''.ceil($r_pings['avgping']).''.ceil($r_pings['highping']).'
'; } ?> -- cgit