Uses non-blank email instead of non-blank first_name for hotel info c…

…riteria, also filters out refunded badges
magfest · Aug 31, 2018 · 03531dc · 03531dc
1 parent 784cd5d
commit 03531dc
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/uber/site_sections/hotel_assignments.py b/uber/site_sections/hotel_assignments.py
@@ -286,8 +286,8 @@ def requested_hotel_info(self, out, session):
             eligibility_filters.append(Attendee.registered <= c.PREREG_HOTEL_ELIGIBILITY_CUTOFF)
 
         hotel_query = session.query(Attendee).filter(*eligibility_filters).filter(
-            Attendee.badge_status != c.INVALID_STATUS,
-            Attendee.first_name != '',
+            Attendee.badge_status.notin_([c.INVALID_STATUS, c.REFUNDED_STATUS]),
+            Attendee.email != '',
         )  # noqa: E712
 
         attendees_without_hotel_pin = hotel_query.filter(*eligibility_filters).filter(or_(