Skip to content

Commit

Permalink
Merge pull request #5125 from openstates/mt-events-more-duplicates-2
Browse files Browse the repository at this point in the history
MT: events: os-core does not consider location for duplicates
  • Loading branch information
jessemortenson authored Dec 10, 2024
2 parents 4007636 + 5a07bf7 commit be18467
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions scrapers/mt/events.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ def scrape_event(self, url: str):

# Check if event already exists in the self._events list
# and if so, add data to that instead of creating duplicate
existing_event = self.check_for_existing_event(title, location, when)
existing_event = self.check_for_existing_event(title, when)
if existing_event is None:
# No existing event found, create one
event = Event(
Expand Down Expand Up @@ -112,14 +112,10 @@ def scrape_event(self, url: str):
self._events.append(event)

def check_for_existing_event(
self, title: str, location_name: str, start_date: datetime.datetime.date
self, title: str, start_date: datetime.datetime.date
) -> Union[Event, None]:
for event in self._events:
if (
event.name == title
and event.location["name"] == location_name
and event.start_date == start_date
):
if event.name == title and event.start_date == start_date:
return event

return None
Expand Down

0 comments on commit be18467

Please sign in to comment.