Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Implement script for escort quest 9759
Thanks to @Grz3s for providing the details Close scriptdev2/scriptdev2#188
- Loading branch information
Showing
3 changed files
with
430 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
41c43a8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
but forgotten:
INSERT INTO
creature_ai_scripts
VALUES(1767805, 17678, 1, 0, 100, 0, 1000, 1000, 0, 0, 45, 5, 100, 0, 0, 0, 0, 0, 0, 0, 0, 0, 'Sironas - Send AI Event OOC'),
(1767806, 17678, 6, 0, 100, 0, 0, 0, 0, 0, 45, 5, 50, 0, 0, 0, 0, 0, 0, 0, 0, 0, 'Sironas - Send AI Event on Death'),
(1767807, 17678, 4, 0, 100, 0, 0, 0, 0, 0, 1, -176781, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 'Sironas - Say on Aggro');
INSERT INTO creature_ai_texts (entry, content_default, type, comment) VALUES
(-176781, "Petulant children, pray to your gods for you are about to meet them!", 1, "17678");
and added abilities (on R2 TTimer)
and for summoned totems (don't let them move):
UPDATE creature_template SET MovementType = 0 WHERE entry IN (21992, 21995);
and add emotion to Sironas:
INSERT INTO
creature_template_addon
(entry
,mount
,bytes1
,b2_0_sheath
,b2_1_pvp_state
,emote
,moveflags
,auras
) VALUES (17678, 0, 0, 1, 16, 468, 0, '');41c43a8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the info.
I didn't forget the ACID part ACID-Scripts/WOTLK@33988d4
41c43a8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i updated the AI DB, The SD2 DB, SD2 Core, Core but iam still not able to see a single one "visual connection" for sironas.
41c43a8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unified-db/Database@3cf4dd7
41c43a8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xfurry
oh, I did not look into ACID =)
41c43a8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xfurry i dont see any recent change that modify stack rules. However i see that aura is removed as soon as another same aura type is casted on same target here -> https://github.com/cmangos/mangos-wotlk/blob/master/src/game/Unit.cpp#L4141
Conclusion stack rules are wrongs and need to be rewrited. But that is not a new.
41c43a8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cyberium
u should only add here
https://github.com/cmangos/mangos-wotlk/blob/master/src/game/Unit.cpp#L4137
SPELL_AURA_PERIODIC_TRIGGER_SPELL
on R2 all works very good
some details:
https://github.com/mangosR2/mangos/blob/master/src/game/Unit.cpp#L5162
https://github.com/mangosR2/mangos/blob/master/src/game/SpellMgr.cpp#L3097
41c43a8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@boxa let's ask @Grz3s to confirm the spells and timers, as he requested the sniff originally.
Also, about the spell stacking, I agree that we need to do some rework on that part. I'm struggling a lot with ICC spells that don't have the expected behavior because of stacking.