help refining a simple rebuff macro

[Stickybuds]

I need help refining this with some more checks, it the timer expires and i happen to be looting a corpse, or a handful of other things, it cant cast yet the timer resets and now i have to wait 10mins before it casts again and hopefully I'm not "busy" when it does. If I notice i can manually click it but now the timer is off for the next recast. Then the worst part is if I am already using a clicky via mq2echange when the timer expires and it tries to run this it causes a DC.

Just need some help with some more checks to prevent this kind of stuff or to ensure the buff is on before starting the timer, or something. =)

Code: Select all

/if ((!${Me.Buff[Familiar: Minion of Chaos].ID}) && ${timerClickBracer}<=0) {
/interrupt
/delay 1s
/casting "Band of Eternal Chaos" -item
/varset timerClickBracer 6010
/varset timerRecovery 40
/goto :loopstart
}

[conradd]

I would wait to see if the cast is successfull, which means something like this :

Code: Select all

/if ((!${Me.Buff[Familiar: Minion of Chaos].ID}) && ${timerClickBracer}<=0) {
   /interrupt
   /delay 1s
   /casting "Band of Eternal Chaos" -item
   /delay 1s
   /if (${Cast.Result.Equal[CAST_SUCCESS]}) {
      /varset timerClickBracer 6010
      /varset timerRecovery 40
      /goto :loopstart
   } else {
      /echo Can't cast Familiar: Minion of Chaos !
   }
}

[Dreos]

I would consider using a MQ2 melee downshit to handle this type of re-buff. If not, cast success from Conradd was a great idea. You can also add in some movement checks, and cast ready checks (I'm not 100% the the cast ready check will work for looting, or how reliable it would be, worth a try though).

Kinda rusty on the syntax I believe these would work:

Code: Select all

${Me.Moving}

Code: Select all

${Cast.Ready}

You could check and if not moving and cast ready cast the spell and then use the cast success as a pre-requisite for enacting the timers.

Code: Select all

/if (!${Me.Moving} && ${Cast.Ready} && (!${Me.Buff[Familiar: Minion of Chaos].ID}) && ${timerClickBracer}<=0)

[Nilbus]

I agree with Dreos, a downshit is the easiest way to handle it. Only downside (which might exist with how you are doing it as well, didn't read it all the way through) is if the buff can be dispelled, the downshit will try to spam activate it no matter the recast.

[Stickybuds]

thanks guys, im trying to avoid mq2melee so i used conardd's example and it has been working great.

[conradd]

Cool, I'm happy it was usefull

so i used conardd's example and it has been working great.

Please be carefull with my name's spelling. Written like this, it's a french insult, some kind of jerk, dumbass

[Stickybuds]

doh sorry, im the worst when it comes to transposing letters