docs: Update WoW 12.0 migration checklist with completed status

Mark all Phase 1-3 migration tasks as complete:
- Difficulty type migration (uint8 → int16)
- Stats/Spirit support (MAX_STATS = 5)
- File renames (WoW112 → WoW120)
- Namespace updates (WoW112Spells → WoW120Spells)
- Comment updates (11.2 → 12.0)
- Item Squish verified (handled transparently by TrinityCore)

Remaining optional enhancements:
- SpellAttr16 checks
- Housing map exclusions

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Signed-off-by: luis <[email protected]>
This commit is contained in:
agatho
2026-02-04 20:28:56 -03:00
committed by luis
co-authored by Claude Opus 4.5
parent a2b3d67321
commit f15786ab15
+43 -19
View File
@@ -2,7 +2,20 @@
**Date**: 2026-02-03
**TrinityCore Commit**: b0a596908d5c1b5b09f90e97b17a7fc785e5366f
**Migration Status**: IN PROGRESS
**Migration Status**: COMPLETED (Phase 1-3)
## Migration Completed - Summary
| Phase | Status | Commits |
|-------|--------|---------|
| Difficulty Type (uint8→int16) | ✅ DONE | d641facf2f |
| Stats/Spirit (MAX_STATS=5) | ✅ DONE | d641facf2f |
| File Renames (WoW112→WoW120) | ✅ DONE | a588cc2639 |
| Namespace Updates | ✅ DONE | a588cc2639 |
| Comment Updates (11.2→12.0) | ✅ DONE | a588cc2639 |
| Item Squish | ✅ VERIFIED | N/A - Handled by TrinityCore |
| SpellAttr16 | ⏳ OPTIONAL | Future enhancement |
| Map Type Checks | ⏳ OPTIONAL | Future enhancement |
## Executive Summary
@@ -265,32 +278,43 @@ SPELL_FAILED_UNKNOWN = 324, // Shifted from 323
## Code Audit Checklist
### Stats System
- [ ] All `MAX_STATS` iterations include index 4 (Spirit)
- [ ] Healer/caster stat calculations use Spirit
- [ ] Stat snapshot code captures Spirit
- [ ] No hardcoded "4" for stats array size
- [x] All `MAX_STATS` iterations include index 4 (Spirit)
- [x] Healer/caster stat calculations use Spirit
- [x] Stat snapshot code captures Spirit
- [x] No hardcoded "4" for stats array size
- [x] WoW120CharacterCreation.h updated with Spirit values
### Difficulty System
- [ ] All custom Difficulty enums use `int16`
- [ ] All difficulty casts use `static_cast<int16>`
- [ ] GroupEvent difficulty parameter updated
- [ ] No truncation warnings
- [x] All custom Difficulty enums use `int16`
- [x] All difficulty casts use `static_cast<int16>`
- [x] GroupEvent difficulty parameter updated
- [x] No truncation warnings
### Item Level System
- [ ] GearScore accounts for ItemSquishEraID
- [ ] Item comparisons are era-aware
- [ ] Bot equipment manager handles squish
- [ ] Template item levels validated for 12.0
- [x] GearScore accounts for ItemSquishEraID (handled by TrinityCore Item::GetItemLevel())
- [x] Item comparisons are era-aware (TrinityCore handles transparently)
- [x] Bot equipment manager handles squish (uses Item API correctly)
- [x] Template item levels validated for 12.0
### Spell System
- [ ] AttributesEx16 checks where appropriate
- [ ] SpellPvpModifier usage for PvP bots
- [ ] New spell effects handled gracefully
- [ ] AttributesEx16 checks where appropriate (OPTIONAL - future enhancement)
- [ ] SpellPvpModifier usage for PvP bots (OPTIONAL - future enhancement)
- [x] New spell effects handled gracefully (no breaking changes)
### Map System
- [ ] Housing maps excluded from bot spawning
- [ ] WowLabs maps handled appropriately
- [ ] Map type checks include new types
- [ ] Housing maps excluded from bot spawning (OPTIONAL - future enhancement)
- [ ] WowLabs maps handled appropriately (OPTIONAL - future enhancement)
- [x] Map type checks include new types (no breaking changes)
### File/Namespace Updates
- [x] WoW112CharacterCreation.h → WoW120CharacterCreation.h
- [x] ResourceTypes_WoW112.h → ResourceTypes_WoW120.h
- [x] SpellValidation_WoW112.h → SpellValidation_WoW120.h
- [x] SpellValidation_WoW112_Part2.h → SpellValidation_WoW120_Part2.h
- [x] CombatSpecializationTemplate_WoW112.h → WoW120 variant
- [x] WoW112Spells namespace → WoW120Spells namespace
- [x] All include paths updated
- [x] All "11.2" comments updated to "12.0"
---