fix(nhlgt): errors due to missing period data

ver(nhlgt): 2.0.9 ver(platform): 2024.3.1
toddrob99 · Mar 8, 2024 · dc2de9d · dc2de9d
1 parent dcc813d
commit dc2de9d
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 8 deletions.
diff --git a/bots/nhl_game_threads/__init__.py b/bots/nhl_game_threads/__init__.py
@@ -32,7 +32,7 @@
 
 import praw
 
-__version__ = "2.0.8"
+__version__ = "2.0.9"
 
 DATA_LOCK = threading.Lock()
 

diff --git a/bots/nhl_game_threads/templates/game_thread.mako b/bots/nhl_game_threads/templates/game_thread.mako
@@ -53,7 +53,7 @@ Shootout! ${data["game"]["awayTeam"]["name"]["default"]}: ${shootout_data.get("a
 %   elif data["game"].get("clock", {}).get("inIntermission"):
 Intermission
 %   else:
-${periodOrd[data["game_boxscore"]["period"]]}${' Period' if data["game_boxscore"]["period"] <= 3 else ''} - ${data["game"]["clock"]["timeRemaining"]} \
+${periodOrd[data["game_boxscore"].get("period", data["game_boxscore"].get("periodDescriptor", {}).get("number"))]}${' Period' if data["game_boxscore"].get("period", data["game_boxscore"].get("periodDescriptor", {}).get("number")) <= 3 else ''} - ${data["game"]["clock"]["timeRemaining"]} \
 ##%       if data["game"]["liveData"]["linescore"]["teams"]["away"]["powerPlay"]:
 ##- ${data["game"]["gameData"]["teams"]["away"]["teamName"]} Power Play (${data["game"]["liveData"]["linescore"]["teams"]["away"]["numSkaters"]} on ${data["game"]["liveData"]["linescore"]["teams"]["home"]["numSkaters"]}) \
 ##%       elif data["game"]["liveData"]["linescore"]["teams"]["home"]["powerPlay"]:
@@ -70,7 +70,7 @@ ${periodOrd[data["game_boxscore"]["period"]]}${' Period' if data["game_boxscore"
 %elif result == "postponed":
 ${'##'} Game Status: Postponed
 %elif result:
-${'##'} Final${f'/{periodOrd[data["game_boxscore"]["period"]]}' if data["game_boxscore"]["period"] > 3 else ""}: \
+${'##'} Final${f'/{periodOrd[data["game_boxscore"].get("period", data["game_boxscore"].get("periodDescriptor", {}).get("number"))]}' if data["game_boxscore"].get("period", data["game_boxscore"].get("periodDescriptor", {}).get("number")) > 3 else ""}: \
 ${maxScore} - ${minScore} \
 %   if result == "tie":
 TIE

diff --git a/bots/nhl_game_threads/templates/linescore.mako b/bots/nhl_game_threads/templates/linescore.mako
@@ -16,7 +16,7 @@
         awayLine = f'|[{awayTeam["commonName"]["default"]}]({data["teamSubs"][awayTeam["abbrev"]]})|'
         homeLine = f'|[{homeTeam["commonName"]["default"]}]({data["teamSubs"][homeTeam["abbrev"]]})|'
         for period in linescore:
-            headerLine += f'{periodOrd[period["period"]]}|'
+            headerLine += f'{periodOrd[period.get("period", period.get("periodDescriptor", {}).get("number"))]}|'
             alignmentLine += ":--|"
             awayLine += f'{period["away"]}|'
             homeLine += f'{period["home"]}|'

diff --git a/bots/nhl_game_threads/templates/penalties.mako b/bots/nhl_game_threads/templates/penalties.mako
@@ -29,7 +29,7 @@ ${'##'} Penalty Summary
     if p.get("servedBy"):
         desc += f"Served by {p['servedBy']}."
 %>\
-|${periodOrd[per.get('period')]}\
+|${periodOrd[per.get("period", per.get("periodDescriptor", {}).get("number"))]}\
 ${(' ' + p.get('timeInPeriod')) if per.get('periodDescriptor', {}).get('periodType') != 'SO' else ''}|\
 [${p['teamAbbrev']}](${data['teamSubs'].get(p['teamAbbrev'], '')})|\
 ${p['duration']}:00 ${sevs.get(p['type'], "")}|\

diff --git a/bots/nhl_game_threads/templates/post_thread.mako b/bots/nhl_game_threads/templates/post_thread.mako
@@ -44,7 +44,7 @@ ${'##'} [${awayTeam["name"]["default"]}](${data["teamSubs"][awayTeam["abbrev"]]}
 %if result == "postponed":
 ${'##'} Game Status: Postponed
 %elif result != "":
-${'##'} Final${f'/{periodOrd[data["game_boxscore"]["period"]]}' if data["game_boxscore"]["period"] > 3 else ""}: \
+${'##'} Final${f'/{periodOrd[data["game_boxscore"].get("period", data["game_boxscore"].get("periodDescriptor", {}).get("number"))]}' if data["game_boxscore"].get("period", data["game_boxscore"].get("periodDescriptor", {}).get("number")) > 3 else ""}: \
 ${maxScore} - ${minScore} \
 %   if result == "tie":
 TIE

diff --git a/bots/nhl_game_threads/templates/scoring_summary.mako b/bots/nhl_game_threads/templates/scoring_summary.mako
@@ -37,7 +37,7 @@ ${'##'} Scoring Summary
     if p.get("highlightClip"):
         desc = f"[{desc}]({highlight_url(p['highlightClip'])})"
 %>\
-|${periodOrd[per.get('period')]}\
+|${periodOrd[per.get("period", per.get("periodDescriptor", {}).get("number"))]}\
 ${(' ' + p.get('timeInPeriod')) if per.get('periodDescriptor', {}).get('periodType') != 'SO' else ''}|\
 [${p.get('teamAbbrev', {}).get('default')}](${data['teamSubs'].get(p.get('teamAbbrev', {}).get('default'), '')})|\
 ${desc}|\

diff --git a/redball/version.py b/redball/version.py
@@ -1,3 +1,3 @@
 #!/usr/bin/env python
 
-VERSION = "2024.1.1"
+VERSION = "2024.3.1"