Merge pull request #411 from datamade/dwe/attribute-errors

handling some AttributeErrors

chicago/feeds.py

@@ -6,6 +6,7 @@
 from django.utils.feedgenerator import Rss201rev2Feed
 from django.urls import reverse, reverse_lazy
 from django.conf import settings
+from sentry_sdk import capture_exception
 
 from .models import ChicagoPerson, Bill, Organization, Event
 from .utils import to_datetime
@@ -74,7 +75,11 @@ def item_link(self, bill):
  return reverse("bill_detail", args=(bill.slug,))
 
  def item_pubdate(self, bill):
- return to_datetime(bill.last_action_date)
+ try:
+ return to_datetime(bill.last_action_date)
+ except AttributeError as e:
+ capture_exception(e)
+ return None
 
  def description(self, obj):
  return "Bills returned from search"

chicago/views.py

@@ -20,6 +20,8 @@
 from django.views.generic import TemplateView, DetailView, ListView
 from haystack.generic_views import FacetedSearchView
 from haystack.forms import FacetedSearchForm
+from sentry_sdk import capture_exception
+
 from opencivicdata.core.models import Membership
 from opencivicdata.legislative.models import (
  BillSponsorship,
@@ -614,10 +616,14 @@ def get_context_data(self, **kwargs):
  event_orgs = event.participants.filter(entity_type="organization")
  context["event_orgs"] = event_orgs
  for event_org in event_orgs:
- org_members = event_org.organization.memberships.filter(
- start_date__lte=event.start_time, end_date__gte=event.start_time
- ).select_related("person__councilmatic_person")
- expected_attendees.update([m.person for m in org_members])
+ try:
+ org_members = event_org.organization.memberships.filter(
+ start_date__lte=event.start_time, end_date__gte=event.start_time
+ ).select_related("person__councilmatic_person")
+ expected_attendees.update([m.person for m in org_members])
+ except AttributeError as e:
+ # found an event without an organization
+ capture_exception(e)
 
  attendees = set()
  for event_person in event.attendance: