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

Stone Statues still misbehaving #20406

Closed
aletson opened this issue Sep 24, 2017 · 6 comments
Closed

Stone Statues still misbehaving #20406

aletson opened this issue Sep 24, 2017 · 6 comments

Comments

@aletson
Copy link

aletson commented Sep 24, 2017

Description: Stone Statues are still misbehaving similar to #6554 #1923

Current behaviour:

When you have a Stone Statue out (crafted object from Jewelcrafting) and attack a mob, the statue will stop healing and run to attack the mob.

Expected behaviour:

The statue should only heal. There's a comment on a wowhead page from back in 2.2.3 that indicates that it might go attack things if a Hunter uses Feign Death, but I have no way of confirming that.

Steps to reproduce the problem:

  1. Get a Rough Stone Statue in your inventory. (.additem 25498 1)
  2. Use it
  3. Attack a mob.

Branch(es): 3.3.5

TC rev. hash/commit: 7a0f1d7

TDB version: 335.63 with updates

Operating system: Ubuntu 16.04 LTS

@offl
Copy link
Contributor

offl commented Nov 1, 2017

There's a comment on a wowhead page from back in 2.2.3 that indicates that it might go attack things if a Hunter uses Feign Death, but I have no way of confirming that.

If you leave combat (For example by using feign death) the statue will attack your enemy...and get killed in 1 hit ^^
http://www.wowhead.com/item=25881/heavy-stone-statue#comments:id=277273

So adding REACT_DEFENSIVE is hack?

@offl
Copy link
Contributor

offl commented Aug 15, 2020

All of them uses NO_COMBAT difficulty flag but probably can attack if hunter uses feign death. Or Blizzard added it later to fix that bug from TBC times.
I don't like that they follows player, guardian follow movement is still used and ignores the fact that creature channels spell. And I don't want to root them.

@ghost
Copy link

ghost commented Aug 15, 2020

Oh. I thought that maybe the solution for totems would apply in this case too.

@offl
Copy link
Contributor

offl commented Nov 26, 2020

CREATURE_DIFFICULTYFLAGS_SESSILE                           = 0x00000100, // Creature is rooted
CREATURE_DIFFICULTYFLAGS_IGNORE_COMBAT                     = 0x02000000,

Are flags they uses

offl added a commit that referenced this issue Nov 28, 2020
@github-actions
Copy link

73d2472

@ghost
Copy link

ghost commented Nov 28, 2020

Nice. Good improvement.

Shauren pushed a commit that referenced this issue Mar 1, 2022
… statues & root them

Closes #20406

(cherry picked from commit 73d2472)
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

2 participants