Please add a cooldown for purple potion throwing roughly equivalent to what a human can manually throw. This keeps the original intent of purple pots while at the same time mitigating abuse by pot throwing macros.

Here's how I'm writing this - somewhat generic, with controls.

Add 'next throw time' variable to PM (nothing to save here - set to mintime initially).
Add SetThrowDelay(delay) method which sets the above var to Now+delay
Add CanThrowNow() method which checks above var vs Now

Add double to CoreAI for delay for Explosion potion (initially set to 2.1)

To base explosion potion, on throw (drink), check CoreAI value, if > 0, check CanThrowNow
On target potion, call SetThrowDelay(delay)


Rationale:
There's really three phases (?) of using a purple potion:
1) arm
2) throw
3) explode

The way things work now is player doubleclicks and arms the potion. Timer begins (between 3 and 5 seconds) and player gets a targetting cursor.
Player targets at some point (if they are just targetting some non-moving object they'll throw it immediately - in PvP, they'll wait until 1 and target it, then they can use another right away). Then when the timer ends, the potion explodes and causes damage.

With this change, if a player targets right away (when timer is 2-5) there is no real change. They can click another one immediately.
However, if they wait until they get the timing right, there is a delay after.

It's also set up so that if we want to, we can add other thrown things to the delay. E.g. if we wanted to delay throws after getting smashed by a warhammer or after you drink a cure potion, we could do that.


I still have to test and make sure that it functions well.


Possible other changes/ideas:
1) make delay variable (e.g. random 1.7-3.1 seconds)
2) make delay dependent on stamina or dexterity (e.g. scales from 2.1 - 4.3)
3) make delay dependent on some other skill value or skill check (*shrug*)

Committed to SVN. Check it out and see if it's what you were envisioning.

Cool. I'll do that.
What did it feel like to you?

So it feels ok to me. Requires a little more paying attention if you are using purple pots heavily.
With the CoreAI dial we can dial it up or down as needed.

Okay, I'll tag you in the patch notes so folks can tag you with questions.
I'm hoping they don't have too many questions, but I think it is good you be part of the convo.

note: the way alchy mages use these is to arm the potion, cancel the targetting cursor, cast a spell, target the spell, double-click the potion again to get the targetting cursor back and target the potion throw.

How dies Pix's change affect this, and are they compatible?

interesting - I don't think it should (since the 'cooldown' starts on target) but we should test.

Closing this unless we get some negative feedback.