From 2e62fb37bf4c7d489e7683372132424548a480f3 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 8 Mar 2026 17:56:47 +0000
Subject: [PATCH] Fix zonedb.cpp: use commit_result.Success() instead of
 !commit_result for MySQLRequestResult

Co-authored-by: Valorith <76063792+Valorith@users.noreply.github.com>
---
 zone/zonedb.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/zone/zonedb.cpp b/zone/zonedb.cpp
index 5cac91b4c..9f0fa0033 100644
--- a/zone/zonedb.cpp
+++ b/zone/zonedb.cpp
@@ -2951,12 +2951,13 @@ void ZoneDatabase::SaveBuffs(Client *client)
 	}
 
 	const auto commit_result = database.TransactionCommit();
-	if (!commit_result) {
+	if (!commit_result.Success()) {
 		database.TransactionRollback();
 		LogError(
-			"Failed to commit buff save transaction for character [{}] [{}].",
+			"Failed to commit buff save transaction for character [{}] [{}]: {}",
 			client->GetCleanName(),
-			client->CharacterID()
+			client->CharacterID(),
+			commit_result.ErrorMessage()
 		);
 	}
 }