Ramdom complain about: Stillwater mutant

From testing, Confused's bug is 100% visual.

The way Confused works is that it adds the ability back to the champion at the same CD that it was at when it was removed (if anyone is seeing otherwise, do let me know).

However, the client doesn't register the CD value for whatever reason.

Either way, it is still serious as the player cannot tell what is going on.

 

It doesn't seem counterintuitive to me, no. It seems weirder if CDs are removed by Confused such that if a unit has an ability on CD8 and it is removed, it would come back at CD0.

We have (and have had) this problem in other areas too and it's usually considered a fairly broken interaction such as with Conqueror and the Tortalleon Transport.

 

Now, personally, I'd prefer if Confused could "suppress" abilities so they stay on the champion, but wouldn't trigger or be activated, but still decrement CD, etc. but that's new functionality Pox doesn't have.

 

That seems pretty awkward, we'd also have to decide if it rounds up or down too.

While we investigate the fix for Confused itself, I'll ask him if the CDs can be stored and manually decremented by the ability before putting it back on, but this is getting kind of hacky.

 

Hmm... nice idea I think and potentially possible. It does make the effect more complex, but would resolve the issue. It seems like it'd work and SHOULD bypass the current bug.

ANyway, we are investigating several ways to fix it, so I will add this one to list, it seems promising.

 

I don't consider it being increased. I consider it being maintained. In the case of your idea, there are also considerations like if it is Cleansed early, and adding/subtracting CDs can be a problem, as there are some technical edge cases such as CDs going negative that are a problem. There is also condition stacking rules so if you get Confused again then what happens to this CD reduction check, etc. It's messier than it seems on the surface, partly because of how conditions work in Pox.

 

The problem is actually with your specific wording as well as just how Confused works since it'd basically just be holding onto a bunch of numbers, and the fact that it is a condition with stacking rules.

In the Salaman's case it is very simple, trigger = -1 CD on things that are currently on CD - no holding numbers, no calculations, no "what if it is removed early" considerations.

In this case, if we are trying to calculate how much CD to remove based on the duration of the condition, it is a bit messier.

What might be easier is for the Confused effect to try and manually decrements the CD each turn so that there's a trigger point for when the CD decrements happens, instead of it trying to be calculated.

 

Salaman Ability:

On relic/equipment deploy
Find all activated abilities on friendlies
Reduce CD by 1 if on CD

Straight-forward, easy.

So first implement of Confused (hard-coded value):

On application
Collect current CDs of activated abilities
Calculate new CDs of activated abilities based on ORIGINAL duration of condition
Remove abilities
On removal
Add abilities at 0 CD
Adjust CDs of abilities by the calculated value

So, here we have the problem of... what happens if Confused is cleansed? Ok, so instead of using the original duration, we need to do some calculations on original - remaining duration. It's messy, but probably doable.

What about if the champion gets Confused a second time? Now we have to take the original CDs of the abilities... calculate how much SHOULD have decremented... then use THOSE values on the new calculations of the second Confused..

And yea... I don't think it's even possible right now to pull the correct numbers and transfer them between the two conditions (I could be wrong on this).

Note: Even the manual decrement would have this problem, so the ideal fix is the Suppression system (but that's not happening anytime soon). My money is on DMR"s solution right now.