ShmooDude wrote:Quick request to have Stampede added back in (its the PvP 4 set bonus). My current gear has 4 pieces of PvP gear so it'd be nice to have it in the script. I understand that it'll be low priority as i'll be in the minority.
Can you verify what spellid is coming up for the Stampede proc (assuming that is what it is being called)?
Also, when you cast Ravage, is it Ravage or "Ravage!"? The spellid would also be helpful.
is the buff you get when Ravage is able to be used (has no duration).
There's also http://www.wowhead.com/spell=131538
which counts down how long till you can use ravage again (30 second duration).
It shows up as "Ravage!" in the combat log.
I'm assuming those are the two correct buff spell IDs as they're the only ones whose icons match.
Elvui displays spell ids in the tooltips of the buffs. Alternatively, the addon idTip will also add the spell id to the tooltip. idTip is available on Curse, but is has not been updated since 4.3, so it might not work (it will be pretty obvious). For the Ravage!, if you turn on logging and then examine the combat log that is saved, you can see what the spell id is. To toggle combat logging on/off, just type "/combatlog"
Since I don't have 4 piece pvp gear, I am going to have to rely on you to verify any code I put in, so getting the spellids correct will shorten the process.
Basically, I plan to add the following to the code:
#defines for 4 piece
SpellInfo(RAVAGEBANG inccounter=ripshreds energy=0 combo=1)
# use Ravage! before other fillers since it is free
if BuffPresent(STAMPEDEBUFF) Spell(RAVAGEBANG)
In order for this code to work, I need to be sure that 81022 is the spellid for the buff you get when you can cast the free Ravage, and also that 102545 is the spell id for Ravage! (this is different than what it used to be).
Edit: I might be able to simplify the code to just "Spell(RAVAGEBANG)" but I am not sure if that is more or less efficient for Ovale.