pa/eqemu-server
2
0
Fork 0
mirror of https://github.com/EQEmu/Server.git synced 2026-04-27 04:02:37 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Wrap SaveBuffs delete+replace in a transaction to prevent buff wipe on insert failure

Browse Source 
Co-authored-by: Valorith <76063792+Valorith@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot] 2026-03-08 15:47:13 +00:00
parent 0b86ffd2de
commit e359852ccf
1 changed files with 14 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
zone/zonedb.cpp
View File

@ -2877,14 +2877,6 @@ void ZoneDatabase::UpdateAltCurrencyValue(uint32 char_id, uint32 currency_id, ui
void ZoneDatabase::SaveBuffs(Client *client)
{
CharacterBuffsRepository::DeleteWhere(
database,
fmt::format(
"`character_id` = {}",
client->CharacterID()
)
);
auto buffs = client->GetBuffs();
const int max_buff_slots = client->GetMaxBuffSlots();
@ -2933,9 +2925,20 @@ void ZoneDatabase::SaveBuffs(Client *client)
v.emplace_back(e);
}
database.TransactionBegin();
CharacterBuffsRepository::DeleteWhere(
database,
fmt::format(
"`character_id` = {}",
client->CharacterID()
)
);
if (!v.empty()) {
const auto saved_count = CharacterBuffsRepository::ReplaceMany(database, v);
if (saved_count != static_cast<int>(v.size())) {
database.TransactionRollback();
LogError(
"Failed to save all buffs for character [{}] [{}]. Expected [{}] rows, saved [{}]. Verify the `character_buffs` schema is up to date.",
client->GetCleanName(),
@ -2943,8 +2946,11 @@ void ZoneDatabase::SaveBuffs(Client *client)
v.size(),
saved_count
);
return;
}
}
database.TransactionCommit();
}
void ZoneDatabase::LoadBuffs(Client *client)