maciejpi/nordabiz
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix(forum): Delete dependent records before removing reply
Some checks are pending
NordaBiz Tests / Unit & Integration Tests (push) Waiting to run
Details
NordaBiz Tests / E2E Tests (Playwright) (push) Blocked by required conditions
Details
NordaBiz Tests / Smoke Tests (Production) (push) Blocked by required conditions
Details
NordaBiz Tests / Send Failure Notification (push) Blocked by required conditions
Details

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
Maciej Pienczyn 2026-02-14 18:09:15 +01:00
parent 8de78e14a1
commit 48b42eb8b1
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
blueprints/forum/routes.py
View File

@ -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()