From 37a9ba3517873cf14cd7fc944f2af82a8398886f Mon Sep 17 00:00:00 2001 From: Stephen Adamson Date: Mon, 22 Jun 2026 17:06:18 +0100 Subject: [PATCH] fix(bookings): use transaction context for discount check in approve edit Use tx.QueryRow instead of db.Conn.QueryRow for discount count check in AdminApproveEditRequestHandler to ensure consistency within the transaction. Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus --- backend/handlers/bookings/manage.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/backend/handlers/bookings/manage.go b/backend/handlers/bookings/manage.go index 80883e1..e364d15 100644 --- a/backend/handlers/bookings/manage.go +++ b/backend/handlers/bookings/manage.go @@ -1729,7 +1729,7 @@ func AdminApproveEditRequestHandler(w http.ResponseWriter, r *http.Request) { // Check for applied discounts (admin warning only — discounts remain locked in) var discountCount int - if err := db.Conn.QueryRow(r.Context(), `SELECT COUNT(*) FROM booking_discounts WHERE booking_id = $1`, bookingID).Scan(&discountCount); err != nil { + if err := tx.QueryRow(r.Context(), `SELECT COUNT(*) FROM booking_discounts WHERE booking_id = $1`, bookingID).Scan(&discountCount); err != nil { log.Printf("ADMIN APPROVE EDIT: Failed to check discounts: %v", err) } if discountCount > 0 {