From 463ee4a3112992661bbaaee7ead1d4dd939d55ce Mon Sep 17 00:00:00 2001 From: Kelsi Date: Mon, 9 Feb 2026 22:19:13 -0800 Subject: [PATCH] Fix misleading comment in TrainerSpell state field The comment incorrectly stated state values as: 0=known(green), 1=available, 2=unavailable(red) Actual correct values are: 0=unavailable(grey), 1=available(green), 2=known(green) This was causing confusion when debugging trainer issues. --- include/game/world_packets.hpp | 2 +- src/game/game_handler.cpp | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/include/game/world_packets.hpp b/include/game/world_packets.hpp index 4f9b85a5..7f6ae490 100644 --- a/include/game/world_packets.hpp +++ b/include/game/world_packets.hpp @@ -1723,7 +1723,7 @@ public: struct TrainerSpell { uint32_t spellId = 0; - uint8_t state = 0; // 0=known(green), 1=available, 2=unavailable(red) + uint8_t state = 0; // 0=unavailable(grey), 1=available(green), 2=known(green) uint32_t spellCost = 0; // copper uint32_t profDialog = 0; uint32_t profButton = 0; diff --git a/src/game/game_handler.cpp b/src/game/game_handler.cpp index d12f12b3..c29ad5e8 100644 --- a/src/game/game_handler.cpp +++ b/src/game/game_handler.cpp @@ -3988,7 +3988,8 @@ void GameHandler::handleInitialSpells(network::Packet& packet) { // Debug: check if specific spells are in initial spells bool has527 = std::find(knownSpells.begin(), knownSpells.end(), 527u) != knownSpells.end(); bool has988 = std::find(knownSpells.begin(), knownSpells.end(), 988u) != knownSpells.end(); - LOG_INFO("Initial spells include: 527=", has527, " 988=", has988); + bool has1180 = std::find(knownSpells.begin(), knownSpells.end(), 1180u) != knownSpells.end(); + LOG_INFO("Initial spells include: 527=", has527, " 988=", has988, " 1180=", has1180); // Ensure Attack (6603) and Hearthstone (8690) are always present if (std::find(knownSpells.begin(), knownSpells.end(), 6603u) == knownSpells.end()) {