From 73a175c101c76feed59dcba793d94e3ef1e63f6d Mon Sep 17 00:00:00 2001 From: Ant Zucaro Date: Tue, 18 Oct 2011 18:28:54 -0400 Subject: [PATCH] Add a minimum required players function --- xonstat/views/submission.py | 29 ++++++++++++++++++++++------- 1 file changed, 22 insertions(+), 7 deletions(-) diff --git a/xonstat/views/submission.py b/xonstat/views/submission.py index a196610..88c4bb2 100755 --- a/xonstat/views/submission.py +++ b/xonstat/views/submission.py @@ -9,6 +9,27 @@ from xonstat.util import strip_colors log = logging.getLogger(__name__) + +def has_minimum_real_players(player_events): + """ + Determines if the collection of player events has enough "real" players + to store in the database. The minimum setting comes from the config file + under the setting xonstat.minimum_real_players. + """ + flg_has_min_real_players = True + + real_players = 0 + for events in player_events: + if is_real_player(events): + real_players += 1 + + #TODO: put this into a config setting in the ini file? + if real_players < 1: + flg_has_min_real_players = False + + return flg_has_min_real_players + + def has_required_metadata(metadata): """ Determines if a give set of metadata has enough data to create a game, @@ -411,13 +432,7 @@ def stats_submit(request): "Can't continue.") raise Exception("Required game meta fields (T, G, M, or S) missing.") - real_players = 0 - for events in players: - if is_real_player(events): - real_players += 1 - - #TODO: put this into a config setting in the ini file? - if real_players < 1: + if not has_minimum_real_players(players): raise Exception("The number of real players is below the minimum. "\ "Stats will be ignored.") -- 2.39.2