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

Core/Pets: summon animation on loading warlock pet #19135

Open
sogladev opened this issue Jun 23, 2024 · 0 comments
Open

Core/Pets: summon animation on loading warlock pet #19135

sogladev opened this issue Jun 23, 2024 · 0 comments

Comments

@sogladev
Copy link
Contributor

Current Behaviour

Upon dismount or loading a warlock pet, the summon spell is being fake casted. This results in a visual/animation/sound that is not blizzlike

pet_summon.mp4

Expected Blizzlike Behaviour

No summon animation on loading pet

Source

I believe this was cherry-picked from TC in TrinityCore/TrinityCore@30d0b2e

This fix send a fake-cast if pet is created by a spell. This fixes raise dead, however most spells do not have a cooldown and do not need this fake-cast fix to be send

original wrath pvp warlock footage showing lack of dismount animation
https://www.youtube.com/watch?v=hD5xBPkxd2c
https://www.youtube.com/watch?v=qeY6_mNVZSQ

Steps to reproduce the problem

.cheat cast
.learn 697 summon voidwalker
.add 6265 32 add soul shards
.learn 46584 raise dead
.learn 23228 learn a mount

make a death knight with master of ghouls talent so make ghoul a pet

image

  1. summon voidwalker
  2. mount
  3. dismount, observe summon animation

Extra Notes

master...sogladev:azerothcore-wotlk:fix(Core/Pet)LoadPetFromDB

potential fix: add if (petInfo->CreatedBySpellId && spellInfo && spellInfo->CategoryRecoveryTime > 0)

could also check for spellInfo->RecoveryTime but these 2 seem to be 0 for most summon spells besides raise dead

if (petInfo->CreatedBySpellId)

AC rev. hash/commit

b055faf

Operating system

Ubuntu 24.04

Custom changes or Modules

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant