RippedAdvancedWarlockStrategy

3 - Add Support for Glyph of Conflagration

Hi,

I wondered if it would be possible to add support for the Glyph of Conflagrate to the Destro Strategy. I guess the basic changes would be to ignore the Immolate/Conflagrate timer, at the same time it would be helpful to cast the first Conflagrate within the first 5 seconds of Immolate so you can cast it a second time while Immolate is still ticking and use the possible availability of the Fire and Brimstone talent.

I tried to patch it myself, but had no time to test it yet. I added a boolean for the availability of the glyph and then check in RA_DestructionStrategy:canConflagrate wether it is set to true or not. If it is false I use the old conflagrateCD <= state.currentTime and immolateEnds>0 and state.currentTime<immolateEnds and state.currentTime >= immolateEnds - self:GetConflagrateWindow() If it is true I just drop the last check as the ConflagrateWindow is no longer interesting. This does not solve the "urgency" to cast the first Conflagrate within the first 5 seconds of Immolate, but since Conflagrate has the highest priority it should be no problem with the default settings.

Regards, Simon

User When Change
iceblox Tue, 19 May 2009 09:38:35 Create

You must login to post a comment. Don't have an account? Register to get one!

Facts

Last updated on
19 May 2009
Reported on
19 May 2009
Status
New - Issue has not had initial review yet.
Type
Enhancement - A change which is intended to better the project in some way
Priority
Medium - Normal priority.

Reported by

Possible assignees