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] Honor among thieves breaks stealth #18553

Closed
VincentVanclef opened this issue Dec 22, 2016 · 6 comments
Closed

[3.3.5] Honor among thieves breaks stealth #18553

VincentVanclef opened this issue Dec 22, 2016 · 6 comments

Comments

@VincentVanclef
Copy link

Description: If i use rupture on a target and go stealth, when rupture crits it breaks stealth, only on crits. same happends on honor among thieves procs.

Current behaviour: (Tell us what happens.) ^

Expected behaviour: (Tell us what should happen instead.) Neither rupture crits nor honor should break stealth.

Steps to reproduce the problem:

  1. get combo points
  2. apply rupture to any npc
  3. vanish and notice when it crits you destealth if you have honor among thieves in talents

Branch(es): 3.3.5 / master (Tell us which branch(es) this issue affects.)

TC rev. hash/commit: dbd5677

TDB version: 65

Operating system: unbuntu 16.04

@VincentVanclef
Copy link
Author

added these values in spell_custom_atr and it fixed it:

51698 64
51699 64
51700 64
51701 64
52916 64
1943 64

@djbarrueco
Copy link

djbarrueco commented Dec 22, 2016

Maybe the same problem in #18421

@VincentVanclef
Copy link
Author

yeah try add those abilities into the spell_custom_Atr like i did and test should fix it

@lmdelval
Copy link

@persephone can you email me to delvalmorell8606@gmail.com with a brief explanation about spell proc system, masks, flags... please? ^^

@VincentVanclef
Copy link
Author

@Dalgurakk I dont know much about it, just that attribute 64 is "does not break sttealth"

@Keader
Copy link
Member

Keader commented Dec 23, 2016

@Dalgurakk look: #14824 (about this custom_attr)

@ariel- ariel- self-assigned this Dec 29, 2016
ariel- added a commit that referenced this issue Dec 29, 2016
- They have damage class melee and proc PROC_FLAG_TAKEN_SPELL_MELEE_DMG_CLASS
- Sated/Exhaustion are negative auras, counts as negative spell flag, and stealth drops too

Closes #18421
Closes #18553
Aokromes pushed a commit to Aokromes/TrinityCore that referenced this issue Dec 30, 2016
- They have damage class melee and proc PROC_FLAG_TAKEN_SPELL_MELEE_DMG_CLASS
- Sated/Exhaustion are negative auras, counts as negative spell flag, and stealth drops too

Closes TrinityCore#18421
Closes TrinityCore#18553
joschiwald pushed a commit that referenced this issue Apr 29, 2018
- They have damage class melee and proc PROC_FLAG_TAKEN_SPELL_MELEE_DMG_CLASS
- Sated/Exhaustion are negative auras, counts as negative spell flag, and stealth drops too

Closes #18421
Closes #18553

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

6 participants