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

[6.x] Mount spells with attribute SPELL_ATTR0_DONT_AFFECT_SHEATH_STATE appear as debuff + spell targeting broken #18072

Closed
nerv234 opened this issue Oct 11, 2016 · 3 comments

Comments

@nerv234
Copy link

nerv234 commented Oct 11, 2016

Description: Mount spells with attribute SPELL_ATTR0_DONT_AFFECT_SHEATH_STATE appears as debuff instead of buff and also some spells targets caster instead of target. I think it is related to d34020d and this is why i made single issue about it.

Steps to reproduce the problem:

  1. .cast 171850 (http://www.wowhead.com/spell=171850/llothien-prowler).
  2. You will see mount aura as debuff instead of buff.

Steps to reproduce second problem:

  1. .cast 80240 (warlock spell http://www.wowhead.com/spell=80240/havoc)
  2. You'll see debuff is applied to caster instead of target.

Branch(es): 6.x

TC hash/commit: ac4cb2b

TDB version: Latest (not related to db)

Operating system: win7 (not related)

Btw. Do you thought about renaming 6.x branch to sth. like 7.x?

@ghost ghost changed the title [6.x] Mess with the spells :) (dunno how to name it) [6.x] Mount spells with attribute SPELL_ATTR0_DONT_AFFECT_SHEATH_STATE appear as debuff Oct 11, 2016
@ghost
Copy link

ghost commented Oct 11, 2016

About "renaming 6.x branch to sth. like 7.x" - it would be better to create a new branch named 'master' and move any open issues and Pull Requests to the new branch, because renaming the current 6.x branch will cause all the existing open issues and PRs to close (it would unlink the existing issues and Pull Requests from the existing branch they are linked to). But that change will also create more work than most TC developers currently can find the time to do, because none of them are paid to maintain it.

Secondly, the ideal way of reporting issues is to report only 1 specific problem in each new issue (given that there are no already open issue tickets about that problem).

@Magoxx
Copy link

Magoxx commented Oct 11, 2016

confirm not only mounts, many spells do damage to caster instead enemy.

@nerv234 nerv234 changed the title [6.x] Mount spells with attribute SPELL_ATTR0_DONT_AFFECT_SHEATH_STATE appear as debuff [6.x] Mount spells with attribute SPELL_ATTR0_DONT_AFFECT_SHEATH_STATE appear as debuff + spell targeting broken Oct 12, 2016
@joschiwald
Copy link
Contributor

joschiwald commented Oct 15, 2016

target is loaded from db2 as 1 where 0 is expected

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

5 participants