Skip to content

Commit 8e4a25c

Browse files
committed
fix: added back some exception handling to delete_event
1 parent 5d9ce54 commit 8e4a25c

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

app/routers/event.py

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ async def eventview(request: Request, event_id: int,
7474

7575

7676
@router.delete("/{event_id}")
77-
def delete_event(event_id: int,
77+
def delete_event(request: Request, event_id: int,
7878
db: Session = Depends(get_db)):
7979
# TODO: Check if the user is the owner of the event.
8080
try:
@@ -86,9 +86,14 @@ def delete_event(event_id: int,
8686

8787
participants = get_participants_emails_by_event(db, event_id)
8888

89-
db.delete(event)
90-
db.query(UserEvent).filter_by(event_id=event_id).delete()
91-
db.commit()
89+
try:
90+
db.delete(event)
91+
db.query(UserEvent).filter_by(event_id=event_id).delete()
92+
db.commit()
93+
except (SQLAlchemyError, TypeError):
94+
return templates.TemplateResponse(
95+
"event/eventview.html", {"request": request, "event_id": event_id},
96+
status_code=status.HTTP_500_INTERNAL_SERVER_ERROR)
9297

9398
if participants and event.start > datetime.now():
9499
pass

0 commit comments

Comments
 (0)