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

[3.3.5] "Earthgrab Totem" chases target #15651

Closed
Treeston opened this issue Oct 3, 2015 · 11 comments
Closed

[3.3.5] "Earthgrab Totem" chases target #15651

Treeston opened this issue Oct 3, 2015 · 11 comments

Comments

@Treeston
Copy link
Member

Treeston commented Oct 3, 2015

Branch 3.3.5, rev 774ea0b, TDB 335.59

Creature-summoned Earthgrab Totem will chase its target, when it should be immobile (as it's a totem, duh).

@P-Kito
Copy link
Contributor

P-Kito commented Oct 4, 2015

uhm... really? I cant believe :D

@Killyana
Copy link
Member

Killyana commented Oct 4, 2015

If the npc enter combat when it's rooted it will enter in evade mode: #15304

@ghost
Copy link

ghost commented Oct 4, 2015

Thanks for the link, Killyana. I knew I have read about this problem before, but could not find it by searching only for "Earthgrab Totem", because it is reported as Mosh'ogg Witch Doctor & Ward of Laze.

@Killyana
Copy link
Member

Killyana commented Oct 5, 2015

@Treeston can you provide the npc that summon this totem?

@Treeston
Copy link
Member Author

Treeston commented Oct 5, 2015

One type of thistleshrubs in Tanaris (I believe it is Gnarled Thistleshrub), and I also vaguely remember seeing it in Stranglethorn somewhere.

@dr-j
Copy link
Contributor

dr-j commented Jan 10, 2016

This arisen because of http://www.wowhead.com/npc=670/skullsplitter-witch-doctor a7fcd11a0/sql/old/6.x/world/01_2015_03_21/2015_07_03_05_world_2015_05_26_06.sql#L94 which fixes it for above case and since there is only one entry for the npc 'Earthgrab Totem' probably fixes all.

DELETE FROM `smart_scripts` WHERE  `entryorguid`=6066;
INSERT INTO `smart_scripts` (`entryorguid`, `source_type`, `id`, `link`, `event_type`, `event_phase_mask`, `event_chance`, `event_flags`, `event_param1`, `event_param2`, `event_param3`, `event_param4`, `action_type`, `action_param1`, `action_param2`, `action_param3`, `action_param4`, `action_param5`, `action_param6`, `target_type`, `target_param1`, `target_param2`, `target_param3`, `target_x`, `target_y`, `target_z`, `target_o`, `comment`) VALUES 
(6066, 0, 0, 0, 0, 0, 100, 0, 1000, 1000, 4800, 5200, 11, 8377, 0, 0, 0, 0, 0, 2, 0, 0, 0, 0, 0, 0, 0, 'Earthgrab Totem - In Combat - Cast \'Earthgrab\''),
(6066, 0, 1, 2, 4, 0, 100, 0, 0, 0, 0, 0, 20, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 'Earthgrab Totem - On Agro - Auto attack off'),
(6066, 0, 2, 0, 61, 0, 100, 0, 0, 0, 0, 0, 21, 0, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 'Earthgrab Totem - On Agro - Disable Combat Movement');

@Treeston
Copy link
Member Author

Root inhabittype maybe?

@ghost
Copy link

ghost commented Sep 21, 2016

Sounds plausible, worth trying. PR #17543 (Core/Creature: implement INHABIT_ROOT to apply UNIT_STATE_ROOT to creature) makes this alternative available:

-- set InhabitType INHABIT_ROOT for NPC entry 6066 Earthgrab Totem
UPDATE `creature_template` SET `InhabitType`=`InhabitType`| 8 WHERE `entry`= 6066;

@ghost ghost added the Feedback-PatchFix label Sep 21, 2016
@Keader
Copy link
Member

Keader commented Nov 1, 2016

#17543 already merged @tkrokli ... so any news on this? :)

@ghost
Copy link

ghost commented Nov 1, 2016

Thank you for reminding me about this, @Keader . Have you tested my query to see if it works?
If it doesn't work alone, will it work with dr-j's SAI in #15651 (comment) ?

@Keader
Copy link
Member

Keader commented Nov 2, 2016

Nops, just looking around some old tickets ^^

Aokromes pushed a commit to Aokromes/TrinityCore that referenced this issue Nov 3, 2016
joschiwald pushed a commit that referenced this issue Nov 26, 2017
…litter Hunter

Closes #18179
Closes #18185
By Staleness89
(cherry picked from commit 101b74b)

DB/Creature: Earthgrab Totem and Tainted Earthgrab Totem

Closes #15651
(cherry picked from commit 322f191)

DB/Gameobject: Add two missing game object

Closes #18187
(cherry picked from commit 7e07d99)

DB/Spell: Earthgrab Totem

http://www.wowhead.com/spell=8378/earthgrab-totem used by http://www.wowhead.com/npc=6066/earthgrab-totem
(cherry picked from commit ee52ba6)
Mihc3 pushed a commit to Mihc3/TrinityCore that referenced this issue Dec 16, 2017
…litter Hunter

Closes TrinityCore#18179
Closes TrinityCore#18185
By Staleness89
(cherry picked from commit 101b74b)

DB/Creature: Earthgrab Totem and Tainted Earthgrab Totem

Closes TrinityCore#15651
(cherry picked from commit 322f191)

DB/Gameobject: Add two missing game object

Closes TrinityCore#18187
(cherry picked from commit 7e07d99)

DB/Spell: Earthgrab Totem

http://www.wowhead.com/spell=8378/earthgrab-totem used by http://www.wowhead.com/npc=6066/earthgrab-totem
(cherry picked from commit ee52ba6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants