Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mogu'Shan Vaults Raid #93

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Mogu'Shan Vaults Raid #93

wants to merge 1 commit into from

Conversation

zTerragor
Copy link
Member

Mogu'Shan Vaults Raid

Need test and applicable to our core

Mogu'Shan Vaults Raid

Need test  and applicable to our core

DELETE FROM npc_text WHERE ID BETWEEN @NPCTEXTID+0 AND @NPCTEXTID+3;
INSERT INTO npc_text (ID, Probability0, Probability1, Probability2, Probability3, Probability4, Probability5, Probability6, Probability7, BroadcastTextId0, BroadcastTextId1, BroadcastTextId2, BroadcastTextId3, BroadcastTextId4, BroadcastTextId5, BroadcastTextId6, BroadcastTextId7, VerifiedBuild) VALUES
(@NPCTEXTID+0, 1, 0, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 46366), -- 64825 (Leng Windstaff)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All NPCTEXTID should has a BroadcastTextID

UPDATE creature_template SET HealthScalingExpansion=9, VerifiedBuild=46366 WHERE entry=304; -- Felsteed
UPDATE creature_template SET HealthScalingExpansion=9, VerifiedBuild=46366 WHERE entry=305; -- White Stallion
UPDATE creature_template SET HealthScalingExpansion=9, VerifiedBuild=46366 WHERE entry=32640; -- Traveler's Tundra Mammoth
UPDATE creature_template SET HealthScalingExpansion=9, VerifiedBuild=46366 WHERE entry=59481; -- World Trigger
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HealthScalingExpansion non exist on Wow 5.4.8

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i can fix this

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need test current script behavior.

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.

3 participants