From c5ea2a512e81f85c2c7fd9db5e419dfaff213860 Mon Sep 17 00:00:00 2001
From: Cameron Lamb <cameron.lamb@digital.trade.gov.uk>
Date: Wed, 6 Nov 2024 13:33:49 +0000
Subject: [PATCH] Bugfix: Events register button incorrectly showing (#796)

---
 src/events/models.py                        | 14 +++++++++++---
 src/events/templates/events/event_page.html | 10 +++++-----
 2 files changed, 16 insertions(+), 8 deletions(-)

diff --git a/src/events/models.py b/src/events/models.py
index 6359768c3..211374737 100644
--- a/src/events/models.py
+++ b/src/events/models.py
@@ -199,10 +199,18 @@ def get_template(self, request, *args, **kwargs):
     def get_context(self, request, *args, **kwargs):
         context = super().get_context(request, *args, **kwargs)
 
+        is_online = self.event_type in {
+            types.EventType.ONLINE,
+            types.EventType.HYBRID,
+        }
+        is_in_person = self.event_type in {
+            types.EventType.IN_PERSON,
+            types.EventType.HYBRID,
+        }
+
         context.update(
-            is_online=self.event_type == types.EventType.ONLINE,
-            is_in_person=self.event_type == types.EventType.IN_PERSON,
-            is_hybrid=self.event_type == types.EventType.HYBRID,
+            is_online=is_online,
+            is_in_person=is_in_person,
             event_date_range=get_event_datetime_display_string(self),
         )
 
diff --git a/src/events/templates/events/event_page.html b/src/events/templates/events/event_page.html
index f49153144..1b0be5064 100644
--- a/src/events/templates/events/event_page.html
+++ b/src/events/templates/events/event_page.html
@@ -52,20 +52,20 @@
                                 {% endif %}
                                 <dt>Event type</dt>
                                 <dd>
-                                    {% if is_online or is_in_person %}
-                                        {{ page.get_event_type_display }}
-                                    {% else %}
+                                    {% if is_online and is_in_person %}
                                         In person or online
+                                    {% else %}
+                                        {{ page.get_event_type_display }}
                                     {% endif %}
                                 </dd>
                             </dl>
                             {% if not page.is_past_event %}
-                                {% if page.offline_event_url and is_in_person or is_hybrid %}
+                                {% if page.offline_event_url and is_in_person %}
                                     <a href="{{ page.offline_event_url }}"
                                        class="dwds-button dwds-button--inline dwds-button--secondary"
                                        target="_blank">Register to join</a>
                                 {% endif %}
-                                {% if page.online_event_url and is_online or is_hybrid %}
+                                {% if page.online_event_url and is_online %}
                                     <a href="{{ page.online_event_url }}"
                                        class="dwds-button dwds-button--inline dwds-button--secondary"
                                        target="_blank">Watch event online</a>