]> de.git.xonotic.org Git - xonotic/xonstat.git/commitdiff
Merge pull request #25 from antzucaro/models-refactor
authorAnt Zucaro <azucaro@gmail.com>
Sun, 6 Nov 2016 00:28:18 +0000 (20:28 -0400)
committerGitHub <noreply@github.com>
Sun, 6 Nov 2016 00:28:18 +0000 (20:28 -0400)
Refactor all of the models into separate modules.

23 files changed:
xonstat/batch/badges/gen_badges.py
xonstat/batch/badges/playerdata.py
xonstat/elo.py
xonstat/models.py [deleted file]
xonstat/models/__init__.py [new file with mode: 0644]
xonstat/models/game.py [new file with mode: 0644]
xonstat/models/main.py [new file with mode: 0644]
xonstat/models/map.py [new file with mode: 0644]
xonstat/models/mixins.py [new file with mode: 0644]
xonstat/models/player.py [new file with mode: 0644]
xonstat/models/server.py [new file with mode: 0644]
xonstat/templates/player_index.mako
xonstat/templates/player_info.mako
xonstat/templates/search.mako
xonstat/views/admin.py
xonstat/views/game.py
xonstat/views/helpers.py
xonstat/views/main.py
xonstat/views/map.py
xonstat/views/player.py
xonstat/views/search.py
xonstat/views/server.py
xonstat/views/submission.py

index d0d6fb226e5c0b46bb629a2eaf84ac8fc401c9c1..908aba06623e730578a33df4017ddac4f647c450 100644 (file)
@@ -1,17 +1,14 @@
 #-*- coding: utf-8 -*-
 
 import sys
-from datetime import datetime
-import sqlalchemy as sa
-import sqlalchemy.sql.functions as func
-from sqlalchemy import distinct
-from pyramid.paster import bootstrap
-from xonstat.models import *
-from xonstat.util import datetime_seconds
+from datetime import datetime, timedelta
 
-from skin import Skin
 from playerdata import PlayerData
-
+from pyramid.paster import bootstrap
+from skin import Skin
+from sqlalchemy import distinct
+from xonstat.models import DBSession, PlayerGameStat, Player, PlayerElo
+from xonstat.util import datetime_seconds
 
 # maximal number of query results (for testing, set to None to get all)
 NUM_PLAYERS = None
index ed4fca4081fafae898e4fae8eca095625c987438..b353e26925dc763ecd34d2ce715db9dfe545050c 100644 (file)
@@ -1,6 +1,4 @@
-import sqlalchemy as sa
-import sqlalchemy.sql.functions as func
-from xonstat.models import *
+from xonstat.models import DBSession, Player
 from xonstat.views.player import get_games_played, get_overall_stats, get_ranks, get_elos
 
 
index f9b289df4d1848b9af1bd2a9b1a894081bc19292..01a1e8a93728fdbd8ce81d70efc65ed42a611cc8 100644 (file)
@@ -1,10 +1,8 @@
 import datetime
 import logging
 import math
-import random
-import sys
-from xonstat.models import *
 
+from xonstat.models import PlayerElo
 
 log = logging.getLogger(__name__)
 
diff --git a/xonstat/models.py b/xonstat/models.py
deleted file mode 100644 (file)
index 56f4260..0000000
+++ /dev/null
@@ -1,493 +0,0 @@
-import json
-import logging
-import math
-import sqlalchemy
-import sqlalchemy.sql.functions as sfunc
-from calendar import timegm
-from datetime import datetime as dt
-from datetime import timedelta
-from sqlalchemy.orm import mapper
-from sqlalchemy.orm import scoped_session
-from sqlalchemy.orm import sessionmaker
-from sqlalchemy.ext.declarative import declarative_base
-from xonstat.util import qfont_decode, strip_colors, html_colors, pretty_date
-
-log = logging.getLogger(__name__)
-
-DBSession = scoped_session(sessionmaker())
-Base = declarative_base()
-
-# define objects for all tables
-class Player(object):
-    def nick_html_colors(self, limit=None):
-        if self.nick is None:
-            return "Anonymous Player"
-        else:
-            return html_colors(self.nick, limit)
-
-    def nick_strip_colors(self):
-        if self.nick is None:
-            return "Anonymous Player"
-        else:
-            return strip_colors(self.nick)
-
-    def joined_pretty_date(self):
-        return pretty_date(self.create_dt)
-
-    def __repr__(self):
-        return "<Player(%s, %s)>" % (self.player_id, self.nick.encode('utf-8'))
-
-    def to_dict(self):
-        return {'player_id':self.player_id, 'nick':self.nick,
-            'joined':self.create_dt.strftime('%Y-%m-%dT%H:%M:%SZ'),
-            'active_ind':self.active_ind, 'location':self.location,
-            'stripped_nick':qfont_decode(self.stripped_nick)}
-
-    def epoch(self):
-        return timegm(self.create_dt.timetuple())
-
-
-class GameType(object):
-    def __repr__(self):
-        return "<GameType(%s, %s, %s)>" % (self.game_type_cd, self.descr, self.active_ind)
-
-    def to_dict(self):
-        return {'game_type_cd':self.game_type_cd, 'name':self.descr, 'active':self.active_ind}
-
-
-class Weapon(object):
-    def __repr__(self):
-        return "<Weapon(%s, %s, %s)>" % (self.weapon_cd, self.descr, self.active_ind)
-
-    def to_dict(self):
-        return {'weapon_cd':self.weapon_cd, 'name':self.descr, 'active':self.active_ind}
-
-
-class Server(object):
-    def __init__(self, name=None, hashkey=None, ip_addr=None):
-        self.name = name
-        self.hashkey = hashkey
-        self.ip_addr = ip_addr
-        self.create_dt = dt.utcnow()
-
-    def __repr__(self):
-        return "<Server(%s, %s)>" % (self.server_id, self.name.encode('utf-8'))
-
-    def to_dict(self):
-        return {'server_id':self.server_id, 'name':self.name,
-            'ip_addr':self.ip_addr, 'location':self.location}
-
-    def fuzzy_date(self):
-        return pretty_date(self.create_dt)
-
-    def epoch(self):
-        return timegm(self.create_dt.timetuple())
-
-
-class Map(object):
-    def __init__(self, name=None):
-        self.name = name
-
-    def __repr__(self):
-        return "<Map(%s, %s, %s)>" % (self.map_id, self.name, self.version)
-
-    def to_dict(self):
-        return {'map_id':self.map_id, 'name':self.name, 'version':self.version,}
-
-    def fuzzy_date(self):
-        return pretty_date(self.create_dt)
-
-    def epoch(self):
-        return timegm(self.create_dt.timetuple())
-
-
-class Game(object):
-    def __init__(self, game_id=None, start_dt=None, game_type_cd=None,
-            server_id=None, map_id=None, winner=None):
-        self.game_id = game_id
-        self.start_dt = start_dt
-        self.game_type_cd = game_type_cd
-        self.server_id = server_id
-        self.map_id = map_id
-        self.winner = winner
-
-    def __repr__(self):
-        return "<Game(%s, %s, %s, %s)>" % (self.game_id, self.start_dt, self.game_type_cd, self.server_id)
-
-    def to_dict(self):
-        return {'game_id':self.game_id, 'start':self.start_dt.strftime('%Y-%m-%dT%H:%M:%SZ'),
-                'game_type_cd':self.game_type_cd, 'server_id':self.server_id}
-
-    def fuzzy_date(self):
-        return pretty_date(self.start_dt)
-
-    def epoch(self):
-        return timegm(self.start_dt.timetuple())
-
-
-class PlayerGameStat(object):
-    def __init__(self, player_game_stat_id=None, create_dt=None):
-        self.player_game_stat_id = player_game_stat_id
-        self.create_dt = create_dt
-
-    def __repr__(self):
-        return "<PlayerGameStat(%s, %s, %s)>" % (self.player_id, self.game_id, self.create_dt)
-
-    def to_dict(self):
-        return {'player_id':self.player_id, 'game_id':self.game_id,
-            'create_dt':self.create_dt.strftime('%Y-%m-%dT%H:%M:%SZ'),
-            'alivetime':self.alivetime, 'rank':self.rank, 'score':self.score, 'team':self.team}
-
-    def nick_stripped(self):
-        if self.nick is None:
-            return "Anonymous Player"
-        else:
-            return strip_colors(self.nick)
-
-    def nick_html_colors(self, limit=None):
-        if self.nick is None:
-            return "Anonymous Player"
-        else:
-            return html_colors(self.nick, limit)
-
-    def team_html_color(self):
-        if self.team == 5:
-            return "red"
-        if self.team == 14:
-            return "blue"
-        if self.team == 13:
-            return "yellow"
-        if self.team == 10:
-            return "pink"
-
-
-class Achievement(object):
-    def __repr__(self):
-        return "<Achievement(%s, %s, %s)>" % (self.achievement_cd, self.descr, self.limit)
-
-    def to_dict(self):
-        return {'achievement_cd':self.achievement_cd, 'name':self.descr, 'limit':self.limit}
-
-
-class PlayerAchievement(object):
-    def __repr__(self):
-        return "<PlayerAchievement(%s, %s)>" % (self.player_id, self.achievement_cd)
-
-    def to_dict(self):
-        return {'player_id':self.player_id, 'achievement_cd':self.achievement_cd}
-
-
-class PlayerWeaponStat(object):
-    def __init__(self, player_id=None, game_id=None, weapon_cd=None):
-        self.player_id = player_id
-        self.game_id = game_id
-        self.weapon_cd = weapon_cd
-        self.fired = 0
-        self.max = 0
-        self.hit = 0
-        self.actual = 0
-        self.frags = 0
-
-    def __repr__(self):
-        return "<PlayerWeaponStat(%s, %s, %s)>" % (self.player_weapon_stats_id, self.player_id, self.game_id)
-
-    def to_dict(self):
-        return {
-            'weapon_cd':self.weapon_cd,
-            'player_weapon_stats_id':self.player_weapon_stats_id,
-            'player_id':self.player_id,
-            'game_id':self.game_id,
-            'fired':self.fired,
-            'max':self.max,
-            'hit':self.hit,
-            'actual':self.actual,
-            'frags':self.frags,
-        }
-
-
-class Hashkey(object):
-    def __init__(self, player_id=None, hashkey=None):
-        self.player_id = player_id
-        self.hashkey = hashkey
-
-    def __repr__(self):
-        return "<Hashkey(%s, %s)>" % (self.player_id, self.hashkey)
-
-    def to_dict(self):
-        return {'player_id':self.player_id, 'hashkey':self.hashkey}
-
-
-class PlayerNick(object):
-    def __repr__(self):
-        return "<PlayerNick(%s, %s)>" % (self.player_id, qfont_decode(self.stripped_nick))
-
-    def to_dict(self):
-        return {'player_id':self.player_id, 'name':qfont_decode(self.stripped_nick)}
-
-
-class PlayerElo(object):
-    def __init__(self, player_id=None, game_type_cd=None, elo=None):
-
-        self.player_id = player_id
-        self.game_type_cd = game_type_cd
-        self.elo = elo
-        self.score = 0
-        self.games = 0
-
-    def __repr__(self):
-        return "<PlayerElo(pid=%s, gametype=%s, elo=%s, games=%s)>" % (self.player_id, self.game_type_cd, self.elo, self.games)
-
-    def to_dict(self):
-        return {'player_id':self.player_id, 'game_type_cd':self.game_type_cd, 'elo':self.elo, 'games':self.games}
-
-
-class PlayerRank(object):
-
-    def nick_html_colors(self, limit=None):
-        if self.nick is None:
-            return "Anonymous Player"
-        else:
-            return html_colors(self.nick, limit)
-
-    def __repr__(self):
-        return "<PlayerRank(pid=%s, gametype=%s, rank=%s)>" % (self.player_id, self.game_type_cd, self.rank)
-
-    def to_dict(self):
-        return {'player_id':self.player_id, 'game_type_cd':self.game_type_cd, 'rank':self.rank}
-
-
-class PlayerCaptime(object):
-    def __init__(self, player_id=None, game_id=None, map_id=None,
-            fastest_cap=None, mod=None):
-        self.player_id = player_id
-        self.game_id = game_id
-        self.map_id = map_id
-        self.fastest_cap = fastest_cap
-        self.mod = mod
-
-    def __repr__(self):
-        return "<PlayerCaptime(pid=%s, map_id=%s, mod=%s)>" % (self.player_id,
-                self.map_id, self.mod)
-
-    def fuzzy_date(self):
-        return pretty_date(self.create_dt)
-
-    def epoch(self):
-        return timegm(self.create_dt.timetuple())
-
-
-class SummaryStat(object):
-    def __repr__(self):
-        return "<SummaryStat(total_players=%s, total_games=%s, total_servers=%s)>" % (self.total_players, self.total_games, self.total_servers)
-
-
-class TeamGameStat(object):
-    def __init__(self, team_game_stat_id=None, create_dt=None):
-        self.team_game_stat_id = team_game_stat_id
-        self.create_dt = create_dt
-
-    def __repr__(self):
-        return "<TeamGameStat(%s, %s, %s)>" % (self.team_game_stat_id, self.game_id, self.team)
-
-    def to_dict(self):
-        return {
-            'team_game_stat_id':self.team_game_stat_id,
-            'game_id':self.game_id,
-            'team':self.team,
-            'score':self.score,
-            'rounds':self.rounds,
-            'caps':self.caps,
-            'create_dt':self.create_dt.strftime('%Y-%m-%dT%H:%M:%SZ'),
-        }
-
-    def team_html_color(self):
-        if self.team == 5:
-            return "red"
-        if self.team == 14:
-            return "blue"
-        if self.team == 13:
-            return "yellow"
-        if self.team == 10:
-            return "pink"
-
-
-class PlayerGameAnticheat(object):
-    def __init__(self, player_id=None, game_id=None, key=None,
-            value=None, create_dt=None):
-        self.player_id                = player_id
-        self.game_id                  = game_id
-        self.key                      = key
-        self.value                    = value
-        self.create_dt                = create_dt
-
-    def __repr__(self):
-        return "<PlayerGameAnticheat(%s, %d)>" % (self.key, self.value)
-
-
-class PlayerGroups(object):
-    def __init__(self, player_id=None, group_name=None):
-        self.player_id  = player_id
-        self.group_name = group_name
-
-    def __repr__(self):
-        return "<PlayerGroups(%s, %s)>" % (self.player_id, self.group_name)
-
-
-class MapCapTime(object):
-    """Fastest flag capture times per map, assembled from a SQLAlchemy query"""
-    def __init__(self, row):
-        self.fastest_cap          = row.fastest_cap
-        self.create_dt            = row.create_dt
-        self.create_dt_epoch      = timegm(row.create_dt.timetuple())
-        self.create_dt_fuzzy      = pretty_date(row.create_dt)
-        self.player_id            = row.player_id
-        self.player_nick          = row.player_nick
-        self.player_nick_stripped = strip_colors(row.player_nick)
-        self.player_nick_html     = html_colors(row.player_nick)
-        self.game_id              = row.game_id
-        self.server_id            = row.server_id
-        self.server_name          = row.server_name
-
-    def to_dict(self):
-        return {
-            "fastest_cap"          : self.fastest_cap.total_seconds(),
-            "create_dt_epoch"      : self.create_dt_epoch,
-            "create_dt_fuzzy"      : self.create_dt_fuzzy,
-            "player_id"            : self.player_id,
-            "player_nick"          : self.player_nick,
-            "player_nick_stripped" : self.player_nick_stripped,
-            "game_id"              : self.game_id,
-            "server_id"            : self.server_id,
-            "server_name"          : self.server_name,
-            }
-
-
-class PlayerCapTime(object):
-    """Fastest flag capture times per player, assembled from a SQLAlchemy query"""
-    def __init__(self, row):
-        self.fastest_cap = row.fastest_cap
-        self.create_dt = row.create_dt
-        self.create_dt_epoch = timegm(row.create_dt.timetuple())
-        self.create_dt_fuzzy = pretty_date(row.create_dt)
-        self.player_id = row.player_id
-        self.game_id = row.game_id
-        self.map_id = row.map_id
-        self.map_name = row.map_name
-        self.server_id = row.server_id
-        self.server_name = row.server_name
-
-    def to_dict(self):
-        return {
-            "fastest_cap" : self.fastest_cap.total_seconds(),
-            "create_dt_epoch": self.create_dt_epoch,
-            "create_dt_fuzzy": self.create_dt_fuzzy,
-            "game_id":self.game_id,
-            "map_id": self.map_id,
-            "map_name": self.map_name,
-            "server_id": self.server_id,
-            "server_name": self.server_name,
-            }
-
-
-class ActivePlayer(object):
-    def __init__(self, sort_order=None, player_id=None, nick=None,
-            alivetime=None):
-        self.sort_order = sort_order
-        self.player_id = player_id
-        self.nick = nick
-        self.alivetime = alivetime
-
-    def nick_html_colors(self):
-        return html_colors(self.nick)
-
-    def __repr__(self):
-        return "<ActivePlayer(%s, %s)>" % (self.sort_order, self.player_id)
-
-
-class ActiveServer(object):
-    def __init__(self, sort_order=None, server_id=None, server_name=None,
-            games=None):
-        self.sort_order = sort_order
-        self.server_id = server_id
-        self.server_name = server_name
-        self.games = games
-
-    def __repr__(self):
-        return "<ActiveServer(%s, %s)>" % (self.sort_order, self.server_id)
-
-
-class ActiveMap(object):
-    def __init__(self, sort_order=None, map_id=None, map_name=None,
-            games=None):
-        self.sort_order = sort_order
-        self.map_id = map_id
-        self.map_name = map_name
-        self.games = games
-
-    def __repr__(self):
-        return "<ActiveMap(%s, %s)>" % (self.sort_order, self.map_id)
-
-
-class PlayerMedal(object):
-    def __repr__(self):
-        return "<PlayerRank(pid=%s, place=%s, alt=%s)>" % (self.player_id,
-                self.place, self.alt)
-
-
-def initialize_db(engine=None):
-    DBSession.configure(bind=engine)
-    Base.metadata.bind = engine
-    Base.metadata.create_all(engine)
-    MetaData = sqlalchemy.MetaData(bind=engine)
-    MetaData.reflect()
-
-    # assign all those tables to an object
-    achievements_table = MetaData.tables['achievements']
-    cd_achievement_table = MetaData.tables['cd_achievement']
-    cd_game_type_table = MetaData.tables['cd_game_type']
-    cd_weapon_table = MetaData.tables['cd_weapon']
-    db_version_table = MetaData.tables['db_version']
-    games_table = MetaData.tables['games']
-    hashkeys_table = MetaData.tables['hashkeys']
-    maps_table = MetaData.tables['maps']
-    player_game_stats_table = MetaData.tables['player_game_stats']
-    players_table = MetaData.tables['players']
-    player_weapon_stats_table = MetaData.tables['player_weapon_stats']
-    servers_table = MetaData.tables['servers']
-    player_nicks_table = MetaData.tables['player_nicks']
-    player_elos_table = MetaData.tables['player_elos']
-    player_ranks_table = MetaData.tables['player_ranks']
-    player_captimes_table = MetaData.tables['player_map_captimes']
-    summary_stats_table = MetaData.tables['summary_stats']
-    team_game_stats_table = MetaData.tables['team_game_stats']
-    player_game_anticheats_table = MetaData.tables['player_game_anticheats']
-    player_groups_table = MetaData.tables['player_groups']
-    active_players_table = MetaData.tables['active_players_mv']
-    active_servers_table = MetaData.tables['active_servers_mv']
-    active_maps_table = MetaData.tables['active_maps_mv']
-    player_medals_table = MetaData.tables['player_medals']
-
-    # now map the tables and the objects together
-    mapper(PlayerAchievement, achievements_table)
-    mapper(Achievement, cd_achievement_table)
-    mapper(GameType, cd_game_type_table)
-    mapper(Weapon, cd_weapon_table)
-    mapper(Game, games_table)
-    mapper(Hashkey, hashkeys_table)
-    mapper(Map, maps_table)
-    mapper(PlayerGameStat, player_game_stats_table)
-    mapper(Player, players_table)
-    mapper(PlayerWeaponStat, player_weapon_stats_table)
-    mapper(Server, servers_table)
-    mapper(PlayerNick, player_nicks_table)
-    mapper(PlayerElo, player_elos_table)
-    mapper(PlayerRank, player_ranks_table)
-    mapper(PlayerCaptime, player_captimes_table)
-    mapper(SummaryStat, summary_stats_table)
-    mapper(TeamGameStat, team_game_stats_table)
-    mapper(PlayerGameAnticheat, player_game_anticheats_table)
-    mapper(PlayerGroups, player_groups_table)
-    mapper(ActivePlayer, active_players_table)
-    mapper(ActiveServer, active_servers_table)
-    mapper(ActiveMap, active_maps_table)
-    mapper(PlayerMedal, player_medals_table)
diff --git a/xonstat/models/__init__.py b/xonstat/models/__init__.py
new file mode 100644 (file)
index 0000000..884ba05
--- /dev/null
@@ -0,0 +1,85 @@
+"""
+Model initialization and mapping.
+"""
+
+from sqlalchemy import MetaData
+from sqlalchemy.ext.declarative import declarative_base
+from sqlalchemy.orm import scoped_session, sessionmaker, mapper
+
+from xonstat.models.game import *
+from xonstat.models.main import *
+from xonstat.models.map import *
+from xonstat.models.player import *
+from xonstat.models.server import *
+
+DBSession = scoped_session(sessionmaker())
+Base = declarative_base()
+
+
+def initialize_db(engine=None):
+    """
+    Initialize the database using reflection.
+
+    :param engine: The SQLAlchemy engine instance to bind.
+    :return: None
+    """
+    DBSession.configure(bind=engine)
+
+    Base.metadata.bind = engine
+    Base.metadata.create_all(engine)
+
+    # Since the schema is actually defined elsewhere, we use reflection to determine the
+    # structure of the models instead of specifying them over again in Python.
+    metadata = MetaData(bind=engine)
+    metadata.reflect()
+
+    # Assign all the tables to objects
+    achievements_table = metadata.tables['achievements']
+    cd_achievement_table = metadata.tables['cd_achievement']
+    cd_game_type_table = metadata.tables['cd_game_type']
+    cd_weapon_table = metadata.tables['cd_weapon']
+    db_version_table = metadata.tables['db_version']
+    games_table = metadata.tables['games']
+    hashkeys_table = metadata.tables['hashkeys']
+    maps_table = metadata.tables['maps']
+    player_game_stats_table = metadata.tables['player_game_stats']
+    players_table = metadata.tables['players']
+    player_weapon_stats_table = metadata.tables['player_weapon_stats']
+    servers_table = metadata.tables['servers']
+    player_nicks_table = metadata.tables['player_nicks']
+    player_elos_table = metadata.tables['player_elos']
+    player_ranks_table = metadata.tables['player_ranks']
+    player_captimes_table = metadata.tables['player_map_captimes']
+    summary_stats_table = metadata.tables['summary_stats']
+    team_game_stats_table = metadata.tables['team_game_stats']
+    player_game_anticheats_table = metadata.tables['player_game_anticheats']
+    player_groups_table = metadata.tables['player_groups']
+    active_players_table = metadata.tables['active_players_mv']
+    active_servers_table = metadata.tables['active_servers_mv']
+    active_maps_table = metadata.tables['active_maps_mv']
+    player_medals_table = metadata.tables['player_medals']
+
+    # Map the tables and the objects together
+    mapper(PlayerAchievement, achievements_table)
+    mapper(Achievement, cd_achievement_table)
+    mapper(GameType, cd_game_type_table)
+    mapper(Weapon, cd_weapon_table)
+    mapper(Game, games_table)
+    mapper(Hashkey, hashkeys_table)
+    mapper(Map, maps_table)
+    mapper(PlayerGameStat, player_game_stats_table)
+    mapper(Player, players_table)
+    mapper(PlayerWeaponStat, player_weapon_stats_table)
+    mapper(Server, servers_table)
+    mapper(PlayerNick, player_nicks_table)
+    mapper(PlayerElo, player_elos_table)
+    mapper(PlayerRank, player_ranks_table)
+    mapper(PlayerCaptime, player_captimes_table)
+    mapper(SummaryStat, summary_stats_table)
+    mapper(TeamGameStat, team_game_stats_table)
+    mapper(PlayerGameAnticheat, player_game_anticheats_table)
+    mapper(PlayerGroups, player_groups_table)
+    mapper(ActivePlayer, active_players_table)
+    mapper(ActiveServer, active_servers_table)
+    mapper(ActiveMap, active_maps_table)
+    mapper(PlayerMedal, player_medals_table)
diff --git a/xonstat/models/game.py b/xonstat/models/game.py
new file mode 100644 (file)
index 0000000..18cf9e8
--- /dev/null
@@ -0,0 +1,194 @@
+"""
+Models related to games.
+"""
+
+from xonstat.models.mixins import FuzzyDateMixin, EpochMixin
+from xonstat.util import strip_colors, html_colors
+
+
+class Game(FuzzyDateMixin, EpochMixin):
+    """
+    An individual game.
+    """
+
+    def __init__(self, game_id=None, start_dt=None, game_type_cd=None, server_id=None, map_id=None,
+                 winner=None):
+        self.game_id = game_id
+        self.start_dt = start_dt
+        self.game_type_cd = game_type_cd
+        self.server_id = server_id
+        self.map_id = map_id
+        self.winner = winner
+
+    def __repr__(self):
+        return "<Game({0.game_id}, {0.start_dt}, {0.game_type_cd}, {0.server_id})>".format(self)
+
+    def to_dict(self):
+        return {
+            'game_id': self.game_id,
+            'start': self.start_dt.strftime('%Y-%m-%dT%H:%M:%SZ'),
+            'game_type_cd': self.game_type_cd,
+            'server_id': self.server_id
+        }
+
+
+class PlayerGameStat(object):
+    """
+    The individual statistics a player has gained/lost during a game.
+    """
+
+    def __init__(self, player_game_stat_id=None, create_dt=None):
+        self.player_game_stat_id = player_game_stat_id
+        self.create_dt = create_dt
+
+    def __repr__(self):
+        return "<PlayerGameStat({0.player_id}, {0.game_id}, {0.create_dt})>".format(self)
+
+    def to_dict(self):
+        return {
+            'player_id': self.player_id,
+            'game_id': self.game_id,
+            'create_dt': self.create_dt.strftime('%Y-%m-%dT%H:%M:%SZ'),
+            'alivetime': self.alivetime,
+            'rank': self.rank,
+            'score': self.score,
+            'team': self.team
+        }
+
+    def nick_stripped(self):
+        if self.nick is None:
+            return "Anonymous Player"
+        else:
+            return strip_colors(self.nick)
+
+    def nick_html_colors(self, limit=None):
+        if self.nick is None:
+            return "Anonymous Player"
+        else:
+            return html_colors(self.nick, limit)
+
+    def team_html_color(self):
+        if self.team == 5:
+            return "red"
+        if self.team == 14:
+            return "blue"
+        if self.team == 13:
+            return "yellow"
+        if self.team == 10:
+            return "pink"
+
+
+class PlayerWeaponStat(object):
+    """
+    The metrics for a single weapon in a game for a player.
+    """
+
+    def __init__(self, player_id=None, game_id=None, weapon_cd=None):
+        self.player_id = player_id
+        self.game_id = game_id
+        self.weapon_cd = weapon_cd
+        self.fired = 0
+        self.max = 0
+        self.hit = 0
+        self.actual = 0
+        self.frags = 0
+
+    def __repr__(self):
+        return ("<PlayerWeaponStat({0.player_weapon_stats_id}, {0.player_id}, {0.game_id})>"
+                .format(self))
+
+    def to_dict(self):
+        return {
+            'weapon_cd': self.weapon_cd,
+            'player_weapon_stats_id': self.player_weapon_stats_id,
+            'player_id': self.player_id,
+            'game_id': self.game_id,
+            'fired': self.fired,
+            'max': self.max,
+            'hit': self.hit,
+            'actual': self.actual,
+            'frags': self.frags,
+        }
+
+
+class TeamGameStat(object):
+    """
+    Team level metrics.
+    """
+
+    def __init__(self, team_game_stat_id=None, create_dt=None):
+        self.team_game_stat_id = team_game_stat_id
+        self.create_dt = create_dt
+
+    def __repr__(self):
+        return "<TeamGameStat({0.team_game_stat_id}, {0.game_id}, {0.team})>".format(self)
+
+    def to_dict(self):
+        return {
+            'team_game_stat_id': self.team_game_stat_id,
+            'game_id': self.game_id,
+            'team': self.team,
+            'score': self.score,
+            'rounds': self.rounds,
+            'caps': self.caps,
+            'create_dt': self.create_dt.strftime('%Y-%m-%dT%H:%M:%SZ'),
+        }
+
+    # TODO: move this function to util
+    def team_html_color(self):
+        if self.team == 5:
+            return "red"
+        if self.team == 14:
+            return "blue"
+        if self.team == 13:
+            return "yellow"
+        if self.team == 10:
+            return "pink"
+
+
+class PlayerGameAnticheat(object):
+    """
+    Anticheat metrics sent by the server to identify odd patterns.
+    """
+
+    def __init__(self, player_id=None, game_id=None, key=None, value=None, create_dt=None):
+        self.player_id = player_id
+        self.game_id = game_id
+        self.key = key
+        self.value = value
+        self.create_dt = create_dt
+
+    def __repr__(self):
+        return "<PlayerGameAnticheat({0.key}, {0.value})>".format(self)
+
+
+class GameType(object):
+    """
+    A particular type of game.
+    """
+
+    def __repr__(self):
+        return "<GameType({0.game_type_cd}, {0.descr}, {0.active_ind})>".format(self)
+
+    def to_dict(self):
+        return {
+            'game_type_cd': self.game_type_cd,
+            'name': self.descr,
+            'active': self.active_ind,
+        }
+
+
+class Weapon(object):
+    """
+    A particular type of weapon.
+    """
+
+    def __repr__(self):
+        return "<Weapon({0.weapon_cd}, {0.descr}, {0.active_ind})>".format(self)
+
+    def to_dict(self):
+        return {
+            'weapon_cd': self.weapon_cd,
+            'name': self.descr,
+            'active': self.active_ind,
+        }
\ No newline at end of file
diff --git a/xonstat/models/main.py b/xonstat/models/main.py
new file mode 100644 (file)
index 0000000..6eec9ac
--- /dev/null
@@ -0,0 +1,63 @@
+"""
+Models related to the main index page.
+"""
+
+from xonstat.util import html_colors
+
+
+class SummaryStat(object):
+    """
+    The high level summary line that is shown on the main page.
+    """
+
+    def __repr__(self):
+        return ("<SummaryStat(total_players={0.total_players}, total_games={0.total_games}, "
+                "total_servers={0.total_servers})>".format(self))
+
+
+class ActivePlayer(object):
+    """
+    A record in the "Most Active Players" list.
+    """
+
+    def __init__(self, sort_order=None, player_id=None, nick=None, alivetime=None):
+        self.sort_order = sort_order
+        self.player_id = player_id
+        self.nick = nick
+        self.alivetime = alivetime
+
+    def nick_html_colors(self):
+        return html_colors(self.nick)
+
+    def __repr__(self):
+        return "<ActivePlayer({0.sort_order}, {0.player_id})>".format(self)
+
+
+class ActiveServer(object):
+    """
+    A record in the "Most Active Servers" list.
+    """
+
+    def __init__(self, sort_order=None, server_id=None, server_name=None, games=None):
+        self.sort_order = sort_order
+        self.server_id = server_id
+        self.server_name = server_name
+        self.games = games
+
+    def __repr__(self):
+        return "<ActiveServer({0.sort_order}, {0.server_id})>".format(self)
+
+
+class ActiveMap(object):
+    """
+    A record in the "Most Active Maps" list.
+    """
+
+    def __init__(self, sort_order=None, map_id=None, map_name=None, games=None):
+        self.sort_order = sort_order
+        self.map_id = map_id
+        self.map_name = map_name
+        self.games = games
+
+    def __repr__(self):
+        return "<ActiveMap({0.sort_order}, {0.map_id})>".format(self)
diff --git a/xonstat/models/map.py b/xonstat/models/map.py
new file mode 100644 (file)
index 0000000..5549559
--- /dev/null
@@ -0,0 +1,60 @@
+"""
+Models related to maps.
+"""
+
+from calendar import timegm
+
+from xonstat.models.mixins import FuzzyDateMixin, EpochMixin
+from xonstat.util import pretty_date, strip_colors, html_colors
+
+
+class Map(FuzzyDateMixin, EpochMixin):
+    """
+    A playable map. Roughly equivalent to a pk3 file, but distinguished by name instead.
+    """
+
+    def __init__(self, name=None):
+        self.name = name
+
+    def __repr__(self):
+        return "<Map({0.map_id}, {0.name}, {0.version})>".format(self)
+
+    def to_dict(self):
+        return {
+            'map_id': self.map_id,
+            'name': self.name,
+            'version': self.version,
+        }
+
+
+# TODO: investigate if this model is truly a model, or really just a query (i.e. namedtuple)
+class MapCapTime(object):
+    """
+    Fastest flag capture times per map.
+    """
+
+    def __init__(self, row):
+        self.fastest_cap = row.fastest_cap
+        self.create_dt = row.create_dt
+        self.create_dt_epoch = timegm(row.create_dt.timetuple())
+        self.create_dt_fuzzy = pretty_date(row.create_dt)
+        self.player_id = row.player_id
+        self.player_nick = row.player_nick
+        self.player_nick_stripped = strip_colors(row.player_nick)
+        self.player_nick_html = html_colors(row.player_nick)
+        self.game_id = row.game_id
+        self.server_id = row.server_id
+        self.server_name = row.server_name
+
+    def to_dict(self):
+        return {
+            "fastest_cap": self.fastest_cap.total_seconds(),
+            "create_dt_epoch": self.create_dt_epoch,
+            "create_dt_fuzzy": self.create_dt_fuzzy,
+            "player_id": self.player_id,
+            "player_nick": self.player_nick,
+            "player_nick_stripped": self.player_nick_stripped,
+            "game_id": self.game_id,
+            "server_id": self.server_id,
+            "server_name": self.server_name,
+        }
diff --git a/xonstat/models/mixins.py b/xonstat/models/mixins.py
new file mode 100644 (file)
index 0000000..7274421
--- /dev/null
@@ -0,0 +1,29 @@
+"""
+Mixins for methods used by several model classes.
+"""
+from calendar import timegm
+from xonstat.util import pretty_date, html_colors
+
+
+class FuzzyDateMixin(object):
+    """Provides a class with a "create_dt" attribute the ability to return a fuzzy date."""
+
+    def fuzzy_date(self):
+        return pretty_date(self.create_dt)
+
+
+class EpochMixin(object):
+    """Provides a class with a "create_dt" attribute the ability to return the epoch time."""
+
+    def epoch(self):
+        return timegm(self.create_dt.timetuple())
+
+
+class NickColorsMixin(object):
+    """Provides a class with a "nick" attribute the ability to return the nick's HTML colors."""
+
+    def nick_html_colors(self, limit=None):
+        if self.nick is None:
+            return "Anonymous Player"
+        else:
+            return html_colors(self.nick, limit)
diff --git a/xonstat/models/player.py b/xonstat/models/player.py
new file mode 100644 (file)
index 0000000..d9df075
--- /dev/null
@@ -0,0 +1,209 @@
+"""
+Models related to players.
+"""
+
+from calendar import timegm
+
+from xonstat.models.mixins import FuzzyDateMixin, EpochMixin, NickColorsMixin
+from xonstat.util import strip_colors, pretty_date, qfont_decode
+
+
+class Player(EpochMixin, NickColorsMixin, FuzzyDateMixin):
+    """
+    A player, which can represent either a human or a bot.
+    """
+
+    def nick_strip_colors(self):
+        if self.nick is None:
+            return "Anonymous Player"
+        else:
+            return strip_colors(self.nick)
+
+    def __repr__(self):
+        return "<Player({}, {})>".format(self.player_id, self.nick.encode('utf-8'))
+
+    def to_dict(self):
+        return {
+            'player_id': self.player_id,
+            'nick': self.nick,
+            'joined': self.create_dt.strftime('%Y-%m-%dT%H:%M:%SZ'),
+            'active_ind': self.active_ind,
+            'location': self.location,
+            'stripped_nick': qfont_decode(self.stripped_nick),
+        }
+
+
+class Achievement(object):
+    """
+    A type of achievement. Referenced implicitly in PlayerAchievement.
+    """
+
+    def __repr__(self):
+        return "<Achievement({0.achievement_cd}, {0.descr}, {0.limit})>".format(self)
+
+    def to_dict(self):
+        return {
+            'achievement_cd': self.achievement_cd,
+            'name': self.descr,
+            'limit':self.limit,
+        }
+
+
+class PlayerAchievement(object):
+    """
+    Achievements a player has earned.
+    """
+
+    def __repr__(self):
+        return "<PlayerAchievement({0.player_id}, {0.achievement_cd})>".format(self)
+
+    def to_dict(self):
+        return {
+            'player_id': self.player_id,
+            'achievement_cd': self.achievement_cd,
+        }
+
+
+class Hashkey(object):
+    """
+    A player's identifying key from the d0_blind_id library.
+    """
+
+    def __init__(self, player_id=None, hashkey=None):
+        self.player_id = player_id
+        self.hashkey = hashkey
+
+    def __repr__(self):
+        return "<Hashkey({0.player_id}, {0.hashkey})>".format(self)
+
+    def to_dict(self):
+        return {
+            'player_id': self.player_id,
+            'hashkey': self.hashkey
+        }
+
+
+class PlayerNick(object):
+    """
+    A single nickname a player has used in a game.
+    """
+
+    def __repr__(self):
+        return "<PlayerNick({0.player_id}, {0.stripped_nick})>".format(self)
+
+    def to_dict(self):
+        return {
+            'player_id': self.player_id,
+            'name': qfont_decode(self.stripped_nick),
+        }
+
+
+class PlayerElo(object):
+    """
+    A player's skill for a particular game type, as determined by a modified Elo algorithm.
+    """
+
+    def __init__(self, player_id=None, game_type_cd=None, elo=None):
+        self.player_id = player_id
+        self.game_type_cd = game_type_cd
+        self.elo = elo
+        self.score = 0
+        self.games = 0
+
+    def __repr__(self):
+        return ("<PlayerElo(pid={0.player_id}, gametype={0.game_type_cd}, elo={0.elo}, "
+                "games={0.games})>".format(self))
+
+    def to_dict(self):
+        return {
+            'player_id': self.player_id,
+            'game_type_cd': self.game_type_cd,
+            'elo': self.elo,
+            'games': self.games,
+        }
+
+
+class PlayerRank(NickColorsMixin):
+    """
+    A player's rank for a given game type.
+    """
+
+    def __repr__(self):
+        return ("<PlayerRank(pid={0.player_id}, gametype={0.game_type_cd}, rank={0.rank})>"
+                .format(self))
+
+    def to_dict(self):
+        return {
+            'player_id': self.player_id,
+            'game_type_cd': self.game_type_cd,
+            'rank': self.rank
+        }
+
+
+class PlayerCaptime(FuzzyDateMixin, EpochMixin):
+    """
+    A flag capture time for a player on a given map.
+    """
+
+    def __init__(self, player_id=None, game_id=None, map_id=None, fastest_cap=None, mod=None):
+        self.player_id = player_id
+        self.game_id = game_id
+        self.map_id = map_id
+        self.fastest_cap = fastest_cap
+        self.mod = mod
+
+    def __repr__(self):
+        return "<PlayerCaptime(pid={0.player_id}, map_id={0.map_id}, mod={0.mod})>".format(self)
+
+
+class PlayerGroups(object):
+    """
+    An authorization group a player belongs to. Used to control access.
+    """
+
+    def __init__(self, player_id=None, group_name=None):
+        self.player_id  = player_id
+        self.group_name = group_name
+
+    def __repr__(self):
+        return "<PlayerGroups({0.player_id}, {0.group_name})>".format(self)
+
+
+# TODO: determine if this is a real model (it is very similar to PlayerCaptime from above)
+class PlayerCapTime(object):
+    """
+    Fastest flag capture times per player.
+    """
+
+    def __init__(self, row):
+        self.fastest_cap = row.fastest_cap
+        self.create_dt = row.create_dt
+        self.create_dt_epoch = timegm(row.create_dt.timetuple())
+        self.create_dt_fuzzy = pretty_date(row.create_dt)
+        self.player_id = row.player_id
+        self.game_id = row.game_id
+        self.map_id = row.map_id
+        self.map_name = row.map_name
+        self.server_id = row.server_id
+        self.server_name = row.server_name
+
+    def to_dict(self):
+        return {
+            "fastest_cap" : self.fastest_cap.total_seconds(),
+            "create_dt_epoch": self.create_dt_epoch,
+            "create_dt_fuzzy": self.create_dt_fuzzy,
+            "game_id":self.game_id,
+            "map_id": self.map_id,
+            "map_name": self.map_name,
+            "server_id": self.server_id,
+            "server_name": self.server_name,
+        }
+
+
+class PlayerMedal(object):
+    """
+    A medal a player has earned in a large tournament.
+    """
+
+    def __repr__(self):
+        return "<PlayerMedal(pid={0.player_id}, place={0.place}, alt={0.alt})>".format(self)
diff --git a/xonstat/models/server.py b/xonstat/models/server.py
new file mode 100644 (file)
index 0000000..40c4a0b
--- /dev/null
@@ -0,0 +1,30 @@
+"""
+Models related to servers.
+"""
+
+from datetime import datetime as dt
+
+from xonstat.models.mixins import FuzzyDateMixin, EpochMixin
+
+
+class Server(FuzzyDateMixin, EpochMixin):
+    """
+    A Xonotic server, identifiable by name and (when there's a conflict) hashkey.
+    """
+
+    def __init__(self, name=None, hashkey=None, ip_addr=None):
+        self.name = name
+        self.hashkey = hashkey
+        self.ip_addr = ip_addr
+        self.create_dt = dt.utcnow()
+
+    def __repr__(self):
+        return "<Server({}, {})>".format(self.server_id, self.name.encode('utf-8'))
+
+    def to_dict(self):
+        return {
+            'server_id': self.server_id,
+            'name': self.name,
+            'ip_addr': self.ip_addr,
+            'location': self.location,
+        }
index 6ef400f58b1a462f5e66d04a3b64ee1b08543ee8..c991f70c683398aa88b9f772c94d7ebda55aaa7a 100644 (file)
@@ -42,7 +42,7 @@
         <tr>
           <td>${player.player_id}</th>
           <td class="no-stretch"><a href="${request.route_url("player_info", id=player.player_id)}" title="Go to this player's info page">${player.nick_html_colors()|n}</a></th>
-          <td><span class="abstime" data-epoch="${player.epoch()}" title="${player.create_dt.strftime('%a, %d %b %Y %H:%M:%S UTC')}">${player.joined_pretty_date()}</span></th>
+          <td><span class="abstime" data-epoch="${player.epoch()}" title="${player.create_dt.strftime('%a, %d %b %Y %H:%M:%S UTC')}">${player.fuzzy_date()}</span></th>
           <td class="text-center">
             <a href="${request.route_url("player_game_index", player_id=player.player_id, page=1)}" title="View recent games by this player">
               <i class="fa fa-list"></i>
index 2d39e103b89e613fff6bdbf540d296ebeb9f908c..956d335f96a1f4a1e29588ae61813b25b7087e7c 100644 (file)
@@ -66,7 +66,7 @@
     </h2>
 
     <h5>
-      <i><span class="abstime" data-epoch="${player.epoch()}" title="${player.create_dt.strftime('%a, %d %b %Y %H:%M:%S UTC')}">Joined ${player.joined_pretty_date()}</span> (player #${player.player_id})</i>
+      <i><span class="abstime" data-epoch="${player.epoch()}" title="${player.create_dt.strftime('%a, %d %b %Y %H:%M:%S UTC')}">Joined ${player.fuzzy_date()}</span> (player #${player.player_id})</i>
       % if cake_day:
         <img src="/static/images/icons/24x24/cake.png" title="Happy cake day!" />
       % endif
index 45bf794897180f3e87706628eb8a645666e35176..69013b0c30943555f407537d9ce08399c7897f9b 100644 (file)
             <tr>
               <td>${player.player_id}</th>
               <td class="no-stretch"><a href="${request.route_url("player_info", id=player.player_id)}" title="Go to this player's info page">${player.nick_html_colors()|n}</a></th>
-              <td><span class="abstime" data-epoch="${player.epoch()}" title="${player.create_dt.strftime('%a, %d %b %Y %H:%M:%S UTC')}">${player.joined_pretty_date()}</span></th>
+              <td><span class="abstime" data-epoch="${player.epoch()}" title="${player.create_dt.strftime('%a, %d %b %Y %H:%M:%S UTC')}">${player.fuzzy_date()}</span></th>
               <td class="text-center">
                 <a href="${request.route_url("player_game_index", player_id=player.player_id, page=1)}" title="View recent games by this player">
                   <i class="fa fa-list"></i>
index 84ecb4d46f8771eac5fc63d42175a36dcefd0e9e..9186d5014f0c1b87f9dd58b6af9e336071657515 100644 (file)
@@ -1,9 +1,9 @@
-from pyramid.response import Response
-from pyramid.httpexceptions import HTTPForbidden, HTTPFound
-from pyramid.security import remember, forget
+from pyramid.httpexceptions import HTTPFound
+from pyramid.security import remember
 from pyramid.session import check_csrf_token
 from pyramid_persona.views import verify_login
-from xonstat.models import *
+from xonstat.models import DBSession, Player
+
 
 def forbidden(request):
     '''A simple forbidden view. Does nothing more than set the status and then
index 7af7c142957ac6b17ab40dc27e72bf19df8707b4..3df051f4e5e7b464673f149b483db8a5f9ab79e8 100644 (file)
@@ -1,18 +1,14 @@
-import datetime
 import logging
-import re
-import time
 from collections import OrderedDict
+
 from pyramid import httpexceptions
-from pyramid.response import Response
-from sqlalchemy import desc, func, over
 from sqlalchemy.orm.exc import *
-from webhelpers.paginate import Page, PageURL
-from xonstat.models import *
+from webhelpers.paginate import Page
+from xonstat.models import DBSession, Server, Map, Game, PlayerGameStat, PlayerWeaponStat
+from xonstat.models import TeamGameStat, PlayerRank, GameType, Weapon
 from xonstat.util import page_url
 from xonstat.views.helpers import RecentGame, recent_games_q
 
-
 log = logging.getLogger(__name__)
 
 
@@ -85,7 +81,7 @@ def _game_info_data(request):
         raise httpexceptions.HTTPNotFound("Could not find that game!")
 
     except Exception as e:
-        raise inst
+        raise e
 
     return {'game':game,
             'server':server,
index afaf9145c57cbaf688346e1ae8826a054e7269e7..82ae6564d02041efb69d54485ef41d757e6dea44 100644 (file)
@@ -1,9 +1,11 @@
 import logging
-import sqlalchemy.sql.expression as expr
+from calendar import timegm
 from datetime import datetime
+
+import sqlalchemy.sql.expression as expr
 from sqlalchemy.orm import aliased
-from xonstat.models import *
-from xonstat.util import *
+from xonstat.models import DBSession, Server, Map, Game, PlayerGameStat, GameType
+from xonstat.util import pretty_date, html_colors
 
 log = logging.getLogger(__name__)
 
index 98973a7a91a0b296e851b8202a2c3563c606e846..c842a554421565b605d7f0e37f8dfd2c9c4d980d 100644 (file)
@@ -1,16 +1,9 @@
 import logging
-import sqlalchemy as sa
-import sqlalchemy.sql.functions as func
-import sqlalchemy.sql.expression as expr
-from beaker.cache import cache_regions, cache_region
-from collections import namedtuple
 from datetime import datetime, timedelta
-from pyramid.response import Response
-from xonstat.models import *
-from xonstat.util import *
-from xonstat.views.helpers import RecentGame, recent_games_q
-from webhelpers.paginate import Page
 
+from beaker.cache import cache_region
+from xonstat.models import DBSession, PlayerRank, ActivePlayer, ActiveServer, ActiveMap
+from xonstat.views.helpers import RecentGame, recent_games_q
 
 log = logging.getLogger(__name__)
 
@@ -82,8 +75,8 @@ def get_summary_stats(scope="all"):
             )
 
     except Exception as e:
-        raise e
         stat_line = None
+        raise e
 
     return stat_line
 
index 506dcd97369f84723bf16a27c97138461666006f..9a9b4e76fba3c6d7d7fd044a511311c07dd0ad8d 100644 (file)
@@ -1,10 +1,13 @@
 import logging
-import sqlalchemy.sql.functions as func
-import sqlalchemy.sql.expression as expr
 from collections import namedtuple
 from datetime import datetime, timedelta
+
+import sqlalchemy.sql.expression as expr
+import sqlalchemy.sql.functions as func
+from pyramid import httpexceptions
 from webhelpers.paginate import Page
-from xonstat.models import *
+from xonstat.models import DBSession, Server, Map, Game, PlayerGameStat, Player, PlayerCaptime
+from xonstat.models.map import MapCapTime
 from xonstat.util import page_url, html_colors
 from xonstat.views.helpers import RecentGame, recent_games_q
 
@@ -186,7 +189,7 @@ def map_captimes_data(request):
                 order_by(expr.asc(PlayerCaptime.fastest_cap))
 
     except Exception as e:
-        raise pyramid.httpexceptions.HTTPNotFound
+        raise httpexceptions.HTTPNotFound
 
     map_captimes = Page(mct_q, current_page, items_per_page=20, url=page_url)
 
index ff3a33f990ffe7d85a6e380fd14874c11cd9f478..41af4f11e2d2b4a9bb177de4474c4ece67dfbaea 100644 (file)
@@ -1,17 +1,20 @@
 import datetime
 import logging
+from calendar import timegm
+from collections import namedtuple
+from urllib import unquote
+
 import pyramid.httpexceptions
 import sqlalchemy as sa
-import sqlalchemy.sql.functions as func
 import sqlalchemy.sql.expression as expr
-from calendar import timegm
-from collections import namedtuple
+import sqlalchemy.sql.functions as func
 from webhelpers.paginate import Page
-from xonstat.models import *
-from xonstat.util import page_url, to_json, pretty_date, datetime_seconds
+from xonstat.models import DBSession, Server, Map, Game, PlayerWeaponStat, Player, Hashkey
+from xonstat.models import PlayerElo, PlayerCaptime, PlayerMedal, GameType
+from xonstat.models.player import PlayerCapTime
 from xonstat.util import is_cake_day, verify_request
+from xonstat.util import page_url, to_json, pretty_date, datetime_seconds
 from xonstat.views.helpers import RecentGame, recent_games_q
-from urllib import unquote
 
 log = logging.getLogger(__name__)
 
index 13700b6ebfe95cb59218a00b50eb2585573a9327..b9d4675147918af1c1c89b1b962e8ddbbd4f6480 100644 (file)
@@ -1,16 +1,9 @@
-import datetime
 import logging
-import pyramid.httpexceptions
-import re
-import time
-from pyramid.response import Response
-from sqlalchemy import desc
-from sqlalchemy.orm.exc import MultipleResultsFound, NoResultFound
+
 from sqlalchemy import func
-from xonstat.models import *
-from xonstat.util import strip_colors, qfont_decode
-from xonstat.util import page_url, html_colors
-from webhelpers.paginate import Page, PageURL
+from webhelpers.paginate import Page
+from xonstat.models import DBSession, Server, Map, Game, PlayerGameStat, Player
+from xonstat.util import page_url
 
 log = logging.getLogger(__name__)
 
index 6aae2c3f69d5c4cf50481ccb35b7135b27a7b7f2..89d512be6d5212b9d8b4b42a49c05e3a3028cdbd 100644 (file)
@@ -86,8 +86,8 @@ class ServerTopMaps(ServerInfoBase):
             top_maps_q = DBSession.query(
                 fg.row_number().over(order_by=expr.desc(func.count())).label("rank"),
                 Game.map_id, Map.name, func.count().label("times_played"))\
-                .filter(Map.map_id==Game.map_id)\
-                .filter(Game.server_id==self.server_id)\
+                .filter(Map.map_id == Game.map_id)\
+                .filter(Game.server_id == self.server_id)\
                 .filter(Game.create_dt > (self.now - timedelta(days=self.lifetime)))\
                 .group_by(Game.map_id)\
                 .group_by(Map.name) \
@@ -157,7 +157,7 @@ class ServerTopScorers(ServerInfoBase):
                 .filter(Game.server_id == self.server_id)\
                 .filter(Player.player_id > 2)\
                 .filter(PlayerGameStat.create_dt >
-                        (self.now - timedelta(days=LEADERBOARD_LIFETIME)))\
+                        (self.now - timedelta(days=self.lifetime)))\
                 .order_by(expr.desc(func.sum(PlayerGameStat.score)))\
                 .group_by(Player.nick)\
                 .group_by(Player.player_id)
@@ -302,23 +302,13 @@ class ServerInfo(ServerInfoBase):
         except:
             raise HTTPNotFound
 
-    def raw(self):
-        """Returns the raw data shared by all renderers."""
-        return {
-            'server': self.server,
-            'top_players': self.top_players_v.top_players,
-            'top_scorers': self.top_scorers_v.top_scorers,
-            'top_maps': self.top_maps_v.top_maps,
-            'recent_games': self.recent_games,
-        }
-
     def html(self):
         """For rendering this data using something HTML-based."""
         return {
             'server': self.server,
-            'top_players': self.top_players_v.html()["top_players"],
-            'top_scorers': self.top_scorers_v.html()["top_scorers"],
-            'top_maps': self.top_maps_v.html()["top_maps"],
+            'top_players': self.top_players_v.html().get("top_players", None),
+            'top_scorers': self.top_scorers_v.html().get("top_scorers", None),
+            'top_maps': self.top_maps_v.html().get("top_maps", None),
             'recent_games': self.recent_games,
             'lifetime': self.lifetime,
         }
index f63fbd28d4063d1be43698026008a9260b930cf5..74e2d3c0e6a05c1206449d2203c1ec7b9c94e24d 100644 (file)
@@ -1,19 +1,17 @@
 import datetime
 import logging
-import os
-import pyramid.httpexceptions
 import re
-import time
+
+import pyramid.httpexceptions
 import sqlalchemy.sql.expression as expr
-from calendar import timegm
-from pyramid.response import Response
 from sqlalchemy import Sequence
 from sqlalchemy.orm.exc import MultipleResultsFound, NoResultFound
 from xonstat.elo import EloProcessor
-from xonstat.models import *
+from xonstat.models import DBSession, Server, Map, Game, PlayerGameStat, PlayerWeaponStat
+from xonstat.models import PlayerRank, PlayerCaptime
+from xonstat.models import TeamGameStat, PlayerGameAnticheat, Player, Hashkey, PlayerNick
 from xonstat.util import strip_colors, qfont_decode, verify_request, weapon_map
 
-
 log = logging.getLogger(__name__)