Skip to content

fix(DB): Gate WotLK Ahune 232 loot behind ToC phase#478

Merged
Nyeriah merged 1 commit into
azerothcore:mainfrom
Nyeriah:fix/ahune-wotlk-loot-phasing
Jun 20, 2026
Merged

fix(DB): Gate WotLK Ahune 232 loot behind ToC phase#478
Nyeriah merged 1 commit into
azerothcore:mainfrom
Nyeriah:fix/ahune-wotlk-loot-phasing

Conversation

@Nyeriah

@Nyeriah Nyeriah commented Jun 20, 2026

Copy link
Copy Markdown
Member

Problem

The WotLK Ahune revert (progression_1_19_world_event_Ahune_down.sql) restored the ilvl-232 cloaks (54801-54805) at the first level-80 bracket (80_1_1, Naxxramas/EoE/OS). Item level 232 matches phase-3 normal-mode raid gear (Trial of the Crusader), so the cloaks were obtainable two phases early.

Separately, that revert restored no weapon at all: the TBC Frostscythe (35514) stopped dropping on transition to level 80, and the WotLK Frostscythe (54806, ilvl 232) was never added.

Change

  • 80_1_1 now restores only the WotLK encounter: level-80 creatures, the level-75 daily (Shards of Ahune → Frost Emblems, the correct phase-1 currency), and the seasonal/cosmetic drops. The epic cloaks are withheld.
  • 80_3 (Trial of the Crusader) unlocks the ilvl-232 cloaks, where their item level fits.
  • The WotLK Frostscythe of Lord Ahune (54806, ilvl 232) is added to the 80_3 loot, in the same guaranteed epic group as the cloaks (one random epic per kill, matching retail).

Result

Bracket Ahune Ice Chest gear
1_19 (TBC) ilvl-110 cloaks/necks + scythe 35514, Badge of Justice
80_1_1 (Naxx, p1) none — daily/emblems + seasonal cosmetics only
80_3 (ToC, p3) ilvl-232 cloaks 54801-54805 + scythe 54806

References: https://wowgaming.altervista.org/aowow/?item=54801

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • Game Balance
    • Adjusted Ahune seasonal event encounter loot progression across different game phases to align with intended progression rewards.

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 <[email protected]>
@coderabbitai

coderabbitai Bot commented Jun 20, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 09f0a8f2-c514-4ede-bf52-ef976e3b7630

📥 Commits

Reviewing files that changed from the base of the PR and between 41e56e4 and fcd311a.

📒 Files selected for processing (2)
  • src/Bracket_80_1_1/sql/world/progression_1_19_world_event_Ahune_down.sql
  • src/Bracket_80_3/sql/world/progression_80_3_world_event_Ahune_cloaks.sql

📝 Walkthrough

Walkthrough

Two SQL scripts split Ahune's ice chest loot across progression brackets: cloak items (54801–54805) are removed from the Bracket_80_1_1 script and placed into a new Bracket_80_3 script, which clears and re-inserts the full gameobject_loot_template rows for entries 28682 and 28683 with phase-appropriate rewards.

Changes

Ahune Cloak Phase-Gating

Layer / File(s) Summary
Remove cloaks from Bracket_80_1_1 loot
src/Bracket_80_1_1/sql/world/progression_1_19_world_event_Ahune_down.sql
Header comments updated to note cloaks are deferred to Bracket_80_3. The insert for entries 28682 and 28683 is narrowed to only non-cloak items (35498, 35557, 35720, 35723); cloak items 54801–54805 are removed.
Add cloaks in Bracket_80_3
src/Bracket_80_3/sql/world/progression_80_3_world_event_Ahune_cloaks.sql
New script deletes all existing loot rows for entries 28682 and 28683, then inserts the complete set including cloak/shroud items (54801–54805, Chance=0, group-based) and non-cloak rewards with per-entry Chance/count differences.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐇 Hop, hop, the cloaks must wait,
Until the Crusaders open the gate!
The ice chest holds only shards and snow,
While ilvl-232 waits to bestow.
Phase by phase the bunny knows best — ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@Nyeriah
Nyeriah merged commit cc21cbf into azerothcore:main Jun 20, 2026
1 of 2 checks passed
Nyeriah added a commit to Nyeriah/mod-progression-system that referenced this pull request Jun 20, 2026
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
azerothcore#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 <[email protected]>
Nyeriah added a commit that referenced this pull request Jun 20, 2026
* 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 <[email protected]>

* 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 <[email protected]>

---------

Co-authored-by: Claude Opus 4.8 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant