The creature spawn table uses a single `id` column. Commit #114 switched
all module creature queries to `id1`/`id2`/`id3`, which do not exist in the
acore_world.creature schema, so the world auto-updater aborted with
"Unknown column 'id1' in 'where clause'".
Revert id1 -> id across all creature UPDATE/DELETE/SELECT statements and
drop the id2/id3 columns (and their values) from the creature INSERTs.
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
* fix(DB): Ahune RDF satchel gives Valor not Frost in phase 1
The Satchel of Chilled Goods (54536) inherits base loot that grants two
Emblem of Frost, the phase-4 ICC currency. Override it at the first WotLK
bracket to give Emblem of Valor instead, matching the phase-1 RDF daily
reward.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
* fix(DB): Gate Ahune satchel scythe behind ToC phase
The ilvl-232 Frostscythe of Lord Ahune (54806) was a 1.4% drop in the RDF
satchel (54536) from the first WotLK phase, the same phase-3 item level that
#478 gated out of the Ice Chest.
- 80_1_1 drops the scythe from the satchel.
- 80_3 (Trial of the Crusader) restores it, matching the Ice Chest gating.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
The WotLK Ahune revert restored the ilvl-232 cloaks (54801-54805) at the
first level-80 bracket (Naxxramas, 80_1_1). That item level matches phase-3
normal raid gear, so the cloaks were available two phases early.
- 80_1_1 now restores only the WotLK encounter (level 80, the level-75 daily
and emblem path, seasonal cosmetics); the epic cloaks are withheld.
- 80_3 (Trial of the Crusader) unlocks the ilvl-232 cloaks, where their item
level fits.
- Also adds the WotLK Frostscythe of Lord Ahune (54806, ilvl 232) to the 80_3
loot. The previous revert dropped no weapon at all once a realm reached 80;
it is now part of the same guaranteed epic group as the cloaks.
References: https://wowgaming.altervista.org/aowow/?item=54536
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Pre-3.1 Wrath awarded the Plagued and Black Proto-Drake for
Glory of the Raider (10/25 player). The rewards were stripped
from `achievement_reward` when Ulduar released, and AzerothCore's
base data reflects the later state, so progression servers on
bracket 80_1 currently can't grant the mounts.
- Add `progression_80_1_achievement_reward_glory_of_the_raider.sql`
in Bracket_80_1_1 inserting rows for achievements 2137/2138
(items 44175 and 44164, sender 26917 Alexstrasza). Mail text
restored from wiki preservation of the original letter.
- Add matching `_down` file in Bracket_80_2 that deletes the
rows when Ulduar unlocks, matching retail's Patch 3.1 removal.
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Keep 10-man Obsidian Sanctum drakes, Sartharion 10M, and the 10M
Satchel of Spoils at Emblem of Heroism during the Naxxramas bracket;
only 25-man encounters are upgraded to Emblem of Valor. The Ulduar
downgrade is adjusted to revert the correct source emblem per entry.
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
DBUpdater requires unique filenames across all brackets. Append the
hosting bracket suffix to disambiguate.
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Merge small SQL files that target the same database table and share the
same bracket prefix into consolidated files. This reduces file count by
22 (from 230 to 208) while preserving all SQL content and comments.
Document SQL file naming conventions and organization rules in README.
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Move `scale` to `creature_template_model.DisplayScale` for Lord Kazzak
(entry 12397) as `scale` column was removed from `creature_template`.
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The SQL statements for restoring TBC profession trainer items have been commented out, effectively disabling their execution. This may be for testing, rollback, or to temporarily prevent these changes from being applied.
The DELETE statements for removing specific TBC items and Brilliant Glass from trainers have been commented out, likely to temporarily retain these items in trainers or for testing purposes.
All SQL statements modifying npc_trainer for flying and flight forms have been commented out, effectively disabling these changes without deleting them.
Removed redundant INSERT statements from progression_80_1_2_pvp_vendors.sql and added a corresponding down migration script in Bracket_80_2. This improves migration management and separation of up/down SQL logic.
Changed the configuration key from 'ProgressionSystem.Bracket_80_1' to 'ProgressionSystem.Bracket_80_1_1' in Bracket_50_59_2_loader.cpp and Bracket_80_1_loader.cpp for consistency and correct feature gating.
Renamed the function AddBracket_80_1_Scripts to AddBracket_80_1_A_Scripts in both its definition and all usages to improve naming consistency and clarity.
Introduces the AddBracket_80_1_B_Scripts function declaration and ensures it is called within Addmod_progression_systemScripts. This allows scripts for the 80_1_B bracket to be properly loaded.
* Updated min count to match max count
For this bracket should only 2 tokens!
* Update progression_80_1_disables.sql
* Modify disables and update creature_template for quests
Update gossip menu and npcflag for Archmage Lan'dalock.
* Revert whaterevr bs github keeps doing
* Update progression_80_4_1_disables.sql
* The Forge of Souls quest available
* Trial of the Crusader quest avaiable
* Ruby santcum quest
* Forbid players for getting quest that require an instance
* Adds back Proof of Demise: The Black Knight
Implemented with the Call of the Crusade and gets removed with Fall of the Lich King
* Update progression_80_4_1_disables.sql
* Removed the wrong instance from ulduar bracket
* Remove Raids weekely
Todo re-add them later phases
* Revert "Remove Raids weekely"
This reverts commit 6f94d6532b.
* Update progression_80_1_disables.sql
* Remove proof demise
* Update progression_80_3_disables.sql
---------
Co-authored-by: FlyingArowana <TheSCREWEDSoftware@users.noreply.github.com>
* Updated min count to match max count
For this bracket should only 2 tokens!
* Added ulduar text to the tier
* Adds inscribed rings with ulduar release
* Update progression_80_2_inscribed_kirin_tor_rings.sql
* Commment for Trial of the Crusader with the tier
* Kirin tor Etched rings with trial of the crusader
* Use the patch name
* Added runed rings for icc bracket
* Revert "Updated min count to match max count"
This reverts commit 08db8646b7.
I've no clue why this ona fresh repo is here as a old change.
* Added delete statement
* Remove frozo
* Re-add frozo on drop
* Add frozo and his shop back in fall of lich king
* Proper progression done to frozo no need to phase
* Reverted 8bc8bcb074
* Instead of delete and add, rephase to 1
* Update progression_80_1_frozo_the_renowned.sql
* Update progression_80_1_frozo_the_renowned_down.sql
---------
Co-authored-by: FlyingArowana <TheSCREWEDSoftware@users.noreply.github.com>
