From 362272e122519f112d72b72fd112df4342b43fdb Mon Sep 17 00:00:00 2001
From: Ant Zucaro <azucaro@gmail.com>
Date: Sun, 22 May 2011 10:20:04 -0400
Subject: [PATCH 1/1] Add html_colors support to player model. Use it in player
 templates!

---
 xonstat/models.py                        | 4 ++++
 xonstat/templates/player_game_index.mako | 4 ++--
 xonstat/templates/player_info.mako       | 6 +++---
 3 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/xonstat/models.py b/xonstat/models.py
index 317e004..e75ccde 100644
--- a/xonstat/models.py
+++ b/xonstat/models.py
@@ -10,6 +10,10 @@ Base = declarative_base()
 
 # define objects for all tables
 class Player(object):
+
+    def nick_html_colors(self):
+        return html_colors(self.nick)
+
     def __repr__(self):
         return "<Player(%s, %s, %s, %s)>" % (self.player_id, self.nick, 
                 self.create_dt, self.location)
diff --git a/xonstat/templates/player_game_index.mako b/xonstat/templates/player_game_index.mako
index 7b23bf2..308fe43 100755
--- a/xonstat/templates/player_game_index.mako
+++ b/xonstat/templates/player_game_index.mako
@@ -1,14 +1,14 @@
 <%inherit file="base.mako"/>
 
 <%block name="title">
-Player Game Index for ${player.nick} - ${parent.title()}
+Player Game Index for ${player.nick_html_colors()} - ${parent.title()}
 </%block>
 
 % if not games:
 <h2>Sorry, no games yet. Get playing, scrub!</h2>
 
 % else:
-<h2>Recent Games by ${player.nick}</h2>
+<h2>Recent Games by ${player.nick_html_colors()}</h2>
 <ul>
 % for (playergamestat, game, server, map) in games:
    <li><a href="${request.route_url("game_info", id=game.game_id)}" name="Game info page for game #${game.game_id}">#${game.game_id}:</a> <a href="${request.route_url("map_info", id=map.map_id)}" name="Map info page for ${map.name}">${map.name}</a></li>
diff --git a/xonstat/templates/player_info.mako b/xonstat/templates/player_info.mako
index 7d046f4..561da17 100755
--- a/xonstat/templates/player_info.mako
+++ b/xonstat/templates/player_info.mako
@@ -2,7 +2,7 @@
 
 <%block name="title">
 % if player:
-Player Information for ${player.nick} - 
+Player Information for ${player.nick_html_colors()} - 
 % endif
 
 ${parent.title()}
@@ -14,7 +14,7 @@ ${parent.title()}
 <p>Seriously though, he probably doesn't exist...just a figment of your imagination. Carry on then!</p>
 
 % else:
-<h2>${player.nick}</h2>
+<h2>${player.nick_html_colors()}</h2>
 <p>
    Joined: ${player.create_dt.strftime('%m/%d/%Y at %I:%M %p')} <br />
 </p>
@@ -27,5 +27,5 @@ ${parent.title()}
    <li><a href="${request.route_url("game_info", id=game.game_id)}" name="Game info page for game #${game.game_id}">#${game.game_id}:</a> <a href="${request.route_url("map_info", id=map.map_id)}" name="Map info page for ${map.name}">${map.name}</a> on <a href="${request.route_url("server_info", id=server.server_id)}" name="Server info page for ${server.name}">${server.name}</a></li>
 % endfor
 </ul>
-<a href="${request.route_url("player_game_index", player_id=player.player_id, page=1)}" title="Game index for ${player.nick}">More games</a> played by ${player.nick}...
+<a href="${request.route_url("player_game_index", player_id=player.player_id, page=1)}" title="Game index for ${player.nick}">More games</a> played by ${player.nick_html_colors()}...
 % endif
-- 
2.39.2