From c0e43dd48eb09bcb562b58175fd98ff79b7d01cd Mon Sep 17 00:00:00 2001
From: Adam Goldsmith <adam@adamgoldsmith.name>
Date: Thu, 25 May 2023 23:05:06 -0400
Subject: [PATCH] upcomingEvents: Improve error messages for events missing
 attributes

---
 memberPlumbing/upcomingEvents.py | 68 ++++++++++++++++++--------------
 1 file changed, 39 insertions(+), 29 deletions(-)

diff --git a/memberPlumbing/upcomingEvents.py b/memberPlumbing/upcomingEvents.py
index cd7e4e3..5d6b5f0 100644
--- a/memberPlumbing/upcomingEvents.py
+++ b/memberPlumbing/upcomingEvents.py
@@ -9,30 +9,34 @@ from .config import Config
 
 
 def format_event(event_details, truncate: bool):
-    url = (
-        "https://claremontmakerspace.org/events/#!event/register/"
-        + event_details["url"]
-    )
-    if "lgo" in event_details:
-        img = (
-            f"""<img class="alignleft" width="400" src="{event_details['lgo']['l']}">"""
+    try:
+        url = (
+            "https://claremontmakerspace.org/events/#!event/register/"
+            + event_details["url"]
         )
-    else:
-        img = ""
-    # print(json.dumps(event_details))
-    out = f"""<h2 style="text-align: center;">
+        if "lgo" in event_details:
+            img = f"""<img class="alignleft" width="400" src="{event_details['lgo']['l']}">"""
+        else:
+            img = ""
+        # print(json.dumps(event_details))
+        out = f"""<h2 style="text-align: center;">
   <a href="{url}">{img}{event_details['ttl']}</a>
 </h2>
 <div><i>{event_details['szp']} &mdash; {event_details['ezp']}</i></div>
 """
-    if not truncate:
-        out += f"""
+        if not truncate:
+            out += f"""
 <div>
   {event_details['dtl']}
 </div>
 
 <a href="{url}">Register for this class now!</a>"""
-    return out
+        return out
+    except KeyError as e:
+        print(
+            f"Event '{event_details.get('ttl')}' missing required property: '{e.args[0]}'"
+        )
+        raise
 
 
 def format_section(title: str, blurb: str, events, truncate: bool):
@@ -62,22 +66,28 @@ def generate_post():
     full_events = []
     upcoming_events = []
     for event in events["evt"]:
-        # ignore hidden events
-        if event["cal"] == 0:
-            continue
-        event_details = membershipworks.get_event_by_eid(event["eid"])
+        try:
+            # ignore hidden events
+            if event["cal"] == 0:
+                continue
+            event_details = membershipworks.get_event_by_eid(event["eid"])
 
-        # registration has already ended
-        if (
-            "erd" in event_details
-            and datetime.fromtimestamp(event_details["erd"]) < now
-        ):
-            ongoing_events.append(event_details)
-        # class is full
-        elif event_details["cnt"] >= event_details["cap"]:
-            full_events.append(event_details)
-        else:
-            upcoming_events.append(event_details)
+            # registration has already ended
+            if (
+                "erd" in event_details
+                and datetime.fromtimestamp(event_details["erd"]) < now
+            ):
+                ongoing_events.append(event_details)
+            # class is full
+            elif event_details["cnt"] >= event_details["cap"]:
+                full_events.append(event_details)
+            else:
+                upcoming_events.append(event_details)
+        except KeyError as e:
+            print(
+                f"Event '{event.get('ttl')}' missing required property: '{e.args[0]}'"
+            )
+            raise
 
     header = """<p><img class="aligncenter size-medium wp-image-2319" src="https://claremontmakerspace.org/wp-content/uploads/2019/03/CMS-Logo-b-y-g-300x168.png" alt="" width="300" height="168" /></a></p>
 <p>Greetings Upper Valley Makers:</p>