2 Commits
Author SHA1 Message Date
Denis Prikhodko 0c99ffd34e Delves/TheDreadPit: add boss Vik'tis, fix spell ID conflict, clean DB errors
Adds Under-Lord Vik'tis (NPC 220158) boss for The Dread Pit delve (map 2684,
scenario 3210, LFGDungeons 2437). Three abilities loaded from serverside_spell:
  - Rupture (1260001): frontal cone physical damage + knockback
  - Burrowing Tremors (1260031): channeled periodic AoE + snare
  - Stinging Swarm (1260003): periodic nature damage to all players

1260002 (original Burrowing Tremors ID) conflicts with an existing client
Spell.db2 entry, so the spell was renumbered to 1260031. The serverside_spell
entries were restored accordingly. spell_script_names updated to match the
actual ScriptMgr registration (DreadPit::spell_delve_rupture).

Also cleans up medium-priority DB audit warnings:
  - removes 9 areatrigger GUIDs with unsupported difficulty 0 on map 1
  - removes lfg_dungeon_template rows for non-existent dungeon IDs (12,8,6,1)
  - removes access_requirement rows for maps without difficulty 1 (43,36,34,33)

Signed-off-by: FrauAndMann <[email protected]>
2026-09-13 19:21:22 -03:00
Denis Prikhodko 89faba7a1d Scripts/Delves: drop hardcoded tier 1 and rely on core tier adoption
The entrance tier mechanism (gossip/tiered-entrance UI -> m_delveSelectedTier
-> first entrant locks tier) already lives in DelveInstance::OnPlayerEnter,
which validates the selection against the delve's map id. The duplicate
adoption block in DelveInstanceScript::OnPlayerEnter skipped the map check
and could let a stale selection from another delve leak in - removed it.

DelveInstanceScript now takes the initial tier via a documented
DEFAULT_DELVE_TIER; the 13 per-delve instance scripts stop hardcoding
1 with a stale TODO.
2026-09-11 18:37:35 -03:00