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

[Mechanics] Effects slowing down cast speed also affect GCD #763

Closed
Jerrythemagicdog opened this issue Mar 6, 2019 · 3 comments
Closed

Comments

@Jerrythemagicdog
Copy link
Contributor

Jerrythemagicdog commented Mar 6, 2019

Description:

Effects like Mind-Numbing Poison (Item ID: 9186) also slow down Global Cooldown which they shouldn't.
(They already can't reduce the gcd above 1.5 and that is working)

Expected behaviour:

Global Cooldowns should be unaffected by spells like Slow, Mind-Numbing poison effect and Curse of Tongues. It's a known bug across many private servers.

** Reproduce steps:**

  • Cast Icy Veins on mage
  • Check GCD for an instant spell
  • Apply mind numbing on mage
  • Check GCD for an instant spell, and see that it has increased

Some related issues and conclusions:
ccshiro/cc-buglist#248
(kelno: this bug report is actually about getting above 1.5 and not about this exact problem)

rev. hash/commit:
rev. 10603

@kelno
Copy link
Member

kelno commented Mar 7, 2019

(from chat)
-- Kelno
We never send the gcd to the client, what we do is
We update the player field UNIT_MOD_CAST_SPEED whenever a buff/debuff is applied
Which is a multiplier, 1 by default
If you get icy veins it gets down to 0.8333 for example
And what this does affect that I can see so far is:

  • The GCD
  • The shown cast time on tooltips

So I find this very strange
Because changing this would also break the cast time on tooltips
-- Jerry
so two things are controlled by one thing
maybe GCD shouldn't be in UNIT_MOD_CAST_SPEED at all
-- Kelno
Yes, maybe there are more than two but whatever
Ah well that's not something we can control
That's just how the client decides the GCD
-- Jerry
okay, so how did Warmane do it
hmmm
-- Kelno
Maybe they broke the tooltip in that case

@kelno
Copy link
Member

kelno commented Mar 7, 2019

Apparently it's not the case on warmane either (tested by jerry):
https://streamable.com/ami5k

https://docs.google.com/document/d/17EZjEeuVQ_oziqph88E_in1WlmTmA5n1ZcJuEbdCr7Q/edit?usp=sharing

@kelno
Copy link
Member

kelno commented Mar 7, 2019

Conclusion here is: It should be affected.

@kelno kelno closed this as completed Mar 7, 2019
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

2 participants