vmangos / core

Progressive Vanilla Core aimed at all versions from 1.2 to 1.12
GNU General Public License v2.0
646 stars 529 forks source link

🐞 [Bug] Rogue's spa should not trigger damage spells, it can now be triggered #2652

Closed yjfyy closed 3 weeks ago

yjfyy commented 3 weeks ago

Client Version: 1.12.1.5875 Commit Hash: https://github.com/vmangos/core/tree/ae4d5f7fe9660af7c3b8ef91900963764b253b37 OS Client: Win10 OS Server: Win10

Gamemechanicwow commented 3 weeks ago

Any source?

ratkosrb commented 3 weeks ago

You are wrong. This has been discussed already multiple times. Just because its inconvenient for players doesn't mean it's gonna get changed. The behavior is correct for vanilla.

https://github.com/vmangos/core/issues/2514

ShiyoKozuki commented 3 weeks ago

I'd like to see a video of a damaging proc proccing on gouge/blind/sap, not just a non-damaging one, from Classic / Vanilla.

Wall-core commented 3 weeks ago

@ShiyoKozuki It's posted right above you https://github.com/vmangos/core/issues/2514

ShiyoKozuki commented 3 weeks ago

That's cripple, not a damaging poison

Gamemechanicwow commented 3 weeks ago

Patch 2.1.0: Gouge: This ability will no longer trigger poisons (and thus break itself).

This is also the same patch where sap got changed so the rogue remains in stealth.

That's cripple, not a damaging poison

Videos with sap: PVP Videos with damage poison on mh: PVE 😅

Wall-core commented 3 weeks ago

That's cripple, not a damaging poison

it very clearly says "Instant point" in the chatbox...