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

[Magisters Terrace] Vexallus can skip summoning Pure Energy #21258

Closed
amed80 opened this issue Jan 24, 2025 · 2 comments · Fixed by #21266
Closed

[Magisters Terrace] Vexallus can skip summoning Pure Energy #21258

amed80 opened this issue Jan 24, 2025 · 2 comments · Fixed by #21266

Comments

@amed80
Copy link
Contributor

amed80 commented Jan 24, 2025

Current Behaviour

After this PR #21239 , Vexallus started skipping summoning Pure Energy if you reach next threshold while the summoning spell still in cooldown.

Expected Blizzlike Behaviour

Vexallus should keep count of each threshold he has went through and summon Pure Energy for each threshold with 5 seconds cooldown.

Source

Retail Wotlk Starting from 1:37, you can see 5 different spawns for Pure Energy

Retail Wotlk Starting from 3:12 , Pure Energy spawns 5 different times.

Wotlk Classic at 0;44 , Vexallus reaches the 25% threshold while still in the 5 seconds cooldown from his previous summons , 3 seconds later he summons Pure Energy that were intended for 25% threshold.

TBC Classic at 8:37 , Vexallus summons Pure Energy for reaching 70% threshold , at 8:39 he reaches 55% threshold but wait until 8:42 and summons Pure Energy there were intended for 55% threshold and similar behavior happens with 40% and 25% threshold.

Steps to reproduce the problem

  1. .go c id 24744
  2. .damage 21600 (in Normal mode) or .damage 29300 (in Heroic Mode) on Vexallus
  3. After Vexallus summons Pure Energy do another .damage 21600 (29300 in heroic mode) before the 5 seconds cooldown are over.
  4. Wait until the cooldown is over > he doesn't summons Pure Energy

Extra Notes

No response

AC rev. hash/commit

AzerothCore rev. a13c2ff 2025-01-23 17:30:31 +0000 (master branch) (Win64, RelWithDebInfo, Static)

Operating system

Windows 11 x64

Custom changes or Modules

No response

@blinkysc
Copy link
Contributor

Working on PR

@blinkysc
Copy link
Contributor

pr #21266

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

Successfully merging a pull request may close this issue.

3 participants