From 990496da7cede2fdfcbb12edfd4909216e7d3d0b Mon Sep 17 00:00:00 2001
From: Ant Zucaro <azucaro@gmail.com>
Date: Wed, 23 Jan 2013 07:13:27 -0500
Subject: [PATCH] Make latency displays parameter-driven.

You can now use the show_latency query parameter to show ping
information in the scoreboards on the game_info page.
---
 xonstat/templates/game_info.mako  |  2 +-
 xonstat/templates/scoreboard.mako | 15 ++++++++-------
 xonstat/views/game.py             |  7 +++++++
 3 files changed, 16 insertions(+), 8 deletions(-)

diff --git a/xonstat/templates/game_info.mako b/xonstat/templates/game_info.mako
index 678d612..12f8c23 100644
--- a/xonstat/templates/game_info.mako
+++ b/xonstat/templates/game_info.mako
@@ -49,7 +49,7 @@ Game Information
 <div class="row">
   <div class="span12 game">
     <h3>Scoreboard</h3>
-    ${scoreboard(game.game_type_cd, pgstats, show_elo)}
+    ${scoreboard(game.game_type_cd, pgstats, show_elo, show_latency)}
   </div>
 </div>
 
diff --git a/xonstat/templates/scoreboard.mako b/xonstat/templates/scoreboard.mako
index 3304a95..ae58fbd 100644
--- a/xonstat/templates/scoreboard.mako
+++ b/xonstat/templates/scoreboard.mako
@@ -1,6 +1,7 @@
-<%def name="scoreboard(game_type_cd, pgstats, show_elo=False)">
+<%def name="scoreboard(game_type_cd, pgstats, show_elo=False, show_latency=False)">
 <table  class="table table-bordered table-condensed">
 ${scoreboard_header(game_type_cd, pgstats[0])}
+${show_latency}
   <tbody>
   % for pgstat in pgstats:
     <tr class="${pgstat.team_html_color()}">
@@ -14,11 +15,11 @@ ${scoreboard_header(game_type_cd, pgstats[0])}
         <span class="nick">${pgstat.nick_html_colors()|n}</span>
       % endif
       </td>
-      % if pgstat.avg_latency is not None:
+      % if show_latency and pgstat.avg_latency is not None:
       <td>
         ${int(round(pgstat.avg_latency))}
       </td>
-      % else:
+      % elif show_latency:
       <td></td>
       % endif
       ${scoreboard_row(game_type_cd, pgstat)}
@@ -42,7 +43,7 @@ ${scoreboard_header(game_type_cd, pgstats[0])}
     <thead>
     <tr>
       <th class="nick">Nick</th>
-      % if pgstat.avg_latency is not None:
+      % if show_latency:
       <th class="ping">Ping</th>
       % endif
       <th class="kills">Kills</th>
@@ -60,7 +61,7 @@ ${scoreboard_header(game_type_cd, pgstats[0])}
     <thead class="ctf ${pgstat.team_html_color()}">
     <tr>
       <th class="nick">Nick</th>
-      % if pgstat.avg_latency is not None:
+      % if show_latency:
       <th class="ping">Ping</th>
       % endif
       <th class="kills">Kills</th>
@@ -80,7 +81,7 @@ ${scoreboard_header(game_type_cd, pgstats[0])}
     <thead class="ca ${pgstat.team_html_color()}">
     <tr>
       <th class="nick">Nick</th>
-      % if pgstat.avg_latency is not None:
+      % if show_latency:
       <th class="ping">Ping</th>
       % endif
       <th class="kills">Kills</th>
@@ -96,7 +97,7 @@ ${scoreboard_header(game_type_cd, pgstats[0])}
     <thead class="freezetag ${pgstat.team_html_color()}">
     <tr>
       <th class="nick">Nick</th>
-      % if pgstat.avg_latency is not None:
+      % if show_latency:
       <th class="ping">Ping</th>
       % endif
       <th class="kills">Kills</th>
diff --git a/xonstat/views/game.py b/xonstat/views/game.py
index 2a7fbb5..11be028 100644
--- a/xonstat/views/game.py
+++ b/xonstat/views/game.py
@@ -62,6 +62,11 @@ def _game_info_data(request):
     else:
         show_elo = False
 
+    if request.params.has_key('show_latency'):
+        show_latency = True
+    else:
+        show_latency = False
+
     try:
         notfound = False
 
@@ -112,6 +117,7 @@ def _game_info_data(request):
         pwstats = None
         captimes = None
         show_elo = False
+        show_latency = False
         raise inst
 
     return {'game':game,
@@ -121,6 +127,7 @@ def _game_info_data(request):
             'pwstats':pwstats,
             'captimes':captimes,
             'show_elo':show_elo,
+            'show_latency':show_latency,
             }
 
 
-- 
2.39.2