mod-paragon: combined Arcane Torrent that refunds mana, energy, and runic power

Building on the previous fix that hid the rogue and DK Arcane Torrent variants
for Paragon Blood Elves: instead of just dropping the duplicates, turn the
remaining mana variant (28730) into a single combined racial that refunds
whichever resource pool the character is using at the moment.

Add SpellScript spell_paragon_arcane_torrent in modules/mod-paragon/src/
Paragon_SC.cpp. Hooks AfterCast on 28730: when the caster is class 12 the
script EnergizeBySpell's 15 energy and 150 internal runic power (= 15 displayed,
matching stock 25046 / 50613 amounts) on top of the spell's stock mana effect.
ModifyPower no-ops on pools the player has no max for, so it is safe even
before the Paragon picks up energy- or RP-using abilities. Non-Paragon Blood
Elves are untouched and keep learning their stock racial.

Update migration 2026_05_10_03.sql to also register the script binding via
spell_script_names (28730 -> 'spell_paragon_arcane_torrent'). Idempotent
DELETE + INSERT.

Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
Docker Build
2026-05-09 15:44:42 -04:00
parent d96123e661
commit 16717acdd3
2 changed files with 66 additions and 3 deletions
@@ -4,10 +4,24 @@
-- every SkillLineAbility delta from patch-enUS-4, so Paragon Blood Elves
-- auto-learned all three and the spellbook showed three identical entries.
--
-- Paragon uses mana as its primary display power; keep only the mana
-- variant (28730) for class 12. IDs 13338 / 17510 match stock WotLK
-- SkillLineAbility rows for 25046 / 50613 on skill line 756.
-- Paragon should learn a single combined Arcane Torrent that refunds mana,
-- energy, AND runic power -- whichever pool the character is using at the
-- moment. We keep spell 28730 as the in-book entry for class 12 and attach
-- the SpellScript spell_paragon_arcane_torrent (modules/mod-paragon/src/
-- Paragon_SC.cpp) so casts by a Paragon also EnergizeBySpell energy + RP on
-- top of the stock mana effect. Other classes' Blood Elves are unaffected.
--
-- IDs 13338 / 17510 match stock WotLK SkillLineAbility rows for spells 25046
-- / 50613 on skill line 756.
UPDATE `skilllineability_dbc`
SET `ClassMask` = `ClassMask` & ~2048
WHERE `ID` IN (13338, 17510);
-- Bind spell_paragon_arcane_torrent (defined in Paragon_SC.cpp) to spell
-- 28730. AC's `spell_script_names` is the standard mapping: script name on
-- the right, spell id on the left. Idempotent via DELETE + INSERT.
DELETE FROM `spell_script_names`
WHERE `spell_id` = 28730 AND `ScriptName` = 'spell_paragon_arcane_torrent';
INSERT INTO `spell_script_names` (`spell_id`, `ScriptName`) VALUES
(28730, 'spell_paragon_arcane_torrent');