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

[Spells] Visual bug after interrupting. #1560

Closed
markvetwow opened this issue Apr 22, 2017 · 5 comments
Closed

[Spells] Visual bug after interrupting. #1560

markvetwow opened this issue Apr 22, 2017 · 5 comments

Comments

@markvetwow
Copy link

markvetwow commented Apr 22, 2017

Revision: 3797+

Operating system: Win 8.1 x64

Current behavior:

When you interrupting a target spell by any cause (kick,stun,fear etc.). Visual part of a spell doesn't disappear. So,on client side it looks,like creature is still casting,even after spell interrupting.

Expected behavior:

Visual part of a spell must not shown,when creature don't cast.

Steps to reproduce the problem:

  1. Pull any mob, i choosed Shadowmoon Warlock from BF (id:17371). Wait until Shadow Bolt or any other spell starts casting.
  2. Interrupt spell by any source.
  3. :(

A little video appliaction to issue:

YouTube.com Link

@AbraKabastard
Copy link
Collaborator

This one is really bugging me!

@PadreWoW
Copy link

I have no this bug on my server, but i can not find my fix :) trying.... :)

@PadreWoW
Copy link

error somewhere in spell.cpp i'm shure! tested!

@PadreWoW
Copy link

Found!

Bug is HERE:

void Spell::SendInterrupted(uint8 result)
{
WorldPacket data(SMSG_SPELL_FAILURE, (8 + 4 + 1));
data << m_caster->GetObjectGUID();
data << m_spellInfo->Id;
data << result;
m_caster->SendMessageToSet(&data, true);

data.Initialize(SMSG_SPELL_FAILED_OTHER, (8 + 4));
data << m_caster->GetObjectGUID(); <-------- ERROR!
data << m_spellInfo->Id;
m_caster->SendMessageToSet(&data, true);

}

Should be data << m_caster->GetPackGUID();

@AbraKabastard
Copy link
Collaborator

Freakin' amazing @PadreWoW

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

3 participants