From 48b42eb8b167107641cdb6dda78b93a8b6c02b3b Mon Sep 17 00:00:00 2001 From: Maciej Pienczyn Date: Sat, 14 Feb 2026 18:09:15 +0100 Subject: [PATCH] fix(forum): Delete dependent records before removing reply Explicitly delete reply reads, attachments, reports and edit history before deleting the reply to avoid NotNullViolation on forum_reply_reads. Co-Authored-By: Claude Opus 4.6 --- blueprints/forum/routes.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/blueprints/forum/routes.py b/blueprints/forum/routes.py index b211603..af9b574 100644 --- a/blueprints/forum/routes.py +++ b/blueprints/forum/routes.py @@ -586,6 +586,11 @@ def admin_forum_delete_reply(reply_id): return jsonify({'success': False, 'error': 'Odpowiedź nie istnieje'}), 404 topic_id = reply.topic_id + # Delete dependent records first (reply reads, attachments, reports, edit history) + db.query(ForumReplyRead).filter(ForumReplyRead.reply_id == reply_id).delete() + db.query(ForumAttachment).filter(ForumAttachment.reply_id == reply_id).delete() + db.query(ForumReport).filter(ForumReport.reply_id == reply_id).delete() + db.query(ForumEditHistory).filter(ForumEditHistory.reply_id == reply_id).delete() db.delete(reply) db.commit()