[Protected Lua] Assassination Rogue macro

[Pthree]

/run if GetSpellCooldown("Kick")==0 and IsSpellInRange("Kick","target") and (UnitCastingInfo("target") or UnitChannelInfo("target")) then CastSpellByName("Kick") end
/run vSnD=0 b={UnitBuff("player", "Slice and Dice")}; if b[7]~=nil then vSnD=b[7]-GetTime() end;
/run if (UnitAffectingCombat("player") or IsStealthed()==1) and UnitExists("focus") and GetSpellCooldown("Tricks of the Trade")==0 and IsSpellInRange("Tricks of the Trade","focus") then RunMacroText("/cast [@focus] Tricks of the Trade") end
/run if IsStealthed()==1 and UnitPower("player")>= 55 then CastSpellByName("Mutilate") end
/run local c=CastSpellByName; local p=UnitPower("player"); if vSnD==0 then if GetComboPoints("player")>0 then if p>=25 then c("Slice and Dice") end else if p>=55 then c("Mutilate") end end end
/run local c=CastSpellByName; local p=UnitPower("player"); if vSnD<4 then if GetComboPoints("player")>0 then if p>=35 then c("Envenom") end else if p>=55 then c("Mutilate") end end end
/run if vSnD>=4 and not UnitBuff("player","Overkill") and GetSpellCooldown("Vanish")==0 and not UnitBuff("player", "Stealth") and UnitHealth("target")>1000000 then CastSpellByName("Vanish") end
/run if vSnD>=4 and GetComboPoints("player")<4 and UnitPower("player")>=55 and (UnitHealth("Target")/UnitHealthMax("Target")>0.35) then CastSpellByName("Mutilate") end
/run if vSnD>=4 and GetComboPoints("player")<4 and UnitPower("player")>=60 and (UnitHealth("Target")/UnitHealthMax("Target")<=0.35) then CastSpellByName("Backstab") end
/run local c=CastSpellByName; local p=UnitPower("player"); local cp=GetComboPoints("player"); if vSnD>=4 and cp>=4 and p>=35 then local dp=0; local i=1; while(i<=40)do local d={UnitDebuff("target",i)}; if d[1]=="Deadly Poison" and d[8]=="player" then dp=d[4] break end i=i+1 end; if dp>=4 then if cp==5 and dp==5 and GetSpellCooldown("Cold Blood")==0 then c("Cold Blood") end c("Envenom") elseif p>=85 then c("Mutilate") end end
/run StaticPopup_Hide("MACRO_ACTION_FORBIDDEN");

Logic:
1. If target is casting and kick is off CD, then kick.
2. If you are in combat or is stealthed, and TotT is off CD, then TotT your focus.
3. If you are stealthed, then mutilate (your opener, in other words).
4. If SnD is not active, and you have at least 1 CP, then SnD. If you have no CP, then Mutilate.
5. If there is only 4sec or less left on SnD, and you have at least 1 CP, then Envenom. If you have no CP, then Mutilate.
6. If SnD is safe, you are not in stealth, you dont have Overkill and your target has atleast one million HP, then Vanish.
7. If SnD is safe, you have less than 4 CP and your target is over 35% HP, then Mutilate.
8. If SnD is safe, you have less than 4 CP and your target is under 35% HP, then Backstab.
9. If SnD is safe, you have at least 4 CP and 4 DP stacks, then Envenom. If you have 5 CP and 5 DP stacks, and Cold Blood is off CD, then Cold Blood before Envenom. If you dont have enough DP stacks and have 85 energy, then Mutilate.

I am pulling about 6k dps on the lvl 85 dummies with this (my average ilvl is 335). I wanted it to check if I was behind the target before it did backstab aswell, but I have no idea how to do that. So you have to make sure you are behind it at 35%, otherwise it wont work (you should be there at all times anyway though). And I did not include Rupture since that was a dps-loss for me, might include it later if I find it more viable then. For poisons you should have Instant poison on your MH and Deadly Poison on your OH. And you have to handle Vendetta by yourself!

To be able to run this, you must have the ability to run protected lua (which you normally cant do). In other words, you must have a hack of some sorts to enable that.

This macro is too long for a normal macro in WoW, therefor you need to download the addon "SuperDuperMacro" (you can find that on wowinterface.com for example), or some other addon that lets you create macros without the character limit.

If you want to loop this (making it so that you dont have to spamclick this macro), make a normal macro in WoW (not with an addon) that says:

/click sdb_roguerota
/run if UnitAffectingCombat("player") then RunMacroText("/in 0.1 /run RunMacro(\"ROGUE\")") end

The first line makes SDM (superdupermacro) execute the rotation macro. In this case the SDM macro is called "roguerota". The second line makes the macro execute itself, looping it (this non-addon macro is called "ROGUE"). The macro will be looped until you exit combat. You can name both of the macros whatever you want, just make sure you type the names in the non-addon macro (non-addon macro = normal macro created in WoW) correct.

[Expoited]

How unprotect lua functions?

[Pthree]

How unprotect lua functions?

It says in my post, you need a hack.

[Magick111]

So this does your rotation for you?

What is this. . . I don't even

[Pthree]

So this does your rotation for you?

I spam this macro, and yes it does my rotation.

What is this. . . I don't even

This is a macro using protected lua. You don't even what?

[k4hn]

Pthree I appreciate you releasing this. But, your macro is not working.

I've pasted the exact same code into Super Duper Macro, and yes I am running a nifty little program that let's me bypass the protected LUA's.

When I press the macro, nothing happens. Nothing..

[Pthree]

Pthree I appreciate you releasing this. But, your macro is not working.

I've pasted the exact same code into Super Duper Macro, and yes I am running a nifty little program that let's me bypass the protected LUA's.

When I press the macro, nothing happens. Nothing..

It works fine for me. Do you click the macro once or spam it? You have to spam it unless you did the loop-macro.

Also, SDM isn't all that updated. But if you have the latest version, and have applied this fix: SuperDuperMacro WoW 4.x fix 3 : Plug-Ins & Patches : WoWInterface AddOns
It should be alright. To be sure everything is set up right:
Name your SDM macro "roguerota" (without the "").
Create a normal macro containing only: /click sdb_roguerota
Drag the normal macro to your actionbar, and spam it.

The "/click sdb_roguerota" tells the addon SDM to run the macro named "roguerota". I did it like this since SDM wouldn't let me drag the SDM-macros to my actionbar, it just came up an error message. But creating a normal macro that tells SDM to run a SDM-macro works just as well.
Why are we even using SDM you might ask, and that is because the rotation macro is too big to fit in a normal macro which has a limit of only 255 characters.

I hope you understand what I mean, I'm not really a good explainer and it gets messy quite easy

[k4hn]

Hey Pthree, thanks for the quick answer.

I do have the fixes for the SDM. I'm also using other rotation macros for my paladin. But this oddly doesn't seem to work for me.

This is what I did:

Open SDM. Pressed 'New'. Had the tick boxes on 'Button Macro' and 'Global', named the SDM macro to 'roguerota'. Pasted the entire code into the empty field after removing '# Enter macro text here.'.
Pressed 'Save'. Closed the window and opened the normal macro menu. Created a macro called 'a' with the macro commands '/click sdb_roguerota'. Dragged the macro 'a' to my hotbar at button 2. Pressed button 2 while targeting and standing behind a Training Dummy and nothing happens.

This is the current build I have with the rogue: Talent Calculator - World of Warcraft

Are you completely sure that the code is correct? The first lines maybe?
Do you know if there's an addon that could possible clash with this SDM macro? making it non working?
Also if its to any help, the rogue I got is a Worgen.

[Pthree]

Yeah I'm sure the code is correct, it works very well for me.

Though you could try to make the SDM-macro Character specific, thats what I did. But I dont think that should matter.
The only thing I can think of that could be wrong, is that you say "Pressed button 2 while targeting and standing behind a Training Dummy and nothing happens.",
do you only click it once? You have to spam it!

What could interfere (but shouldn't) is that I use the Mutiliate glyph, bringing the cost down to 55 energy. Do you have that?
Also, for the rotation to work as it should you need to have the talents "Murderous Intent" and "Cut to the Chase". So this mainly is for PvE (your specc seems to be PvP?). But even if you dont have those, something should happen in the beginning.
Is your language in WoW english? I don't know if it would work with for example a german client, since the spellnames are different.

Thats all I can think of I'm afraid

[k4hn]

Well I tried spamming also of course

Gonna try and disable some addons, deleting some old cache etc and see if it works. I'll come back with a reply later.

Do you happen to know any more rotation macros working for cataclysm? for example a fury warrior macro?

hade gött så länge

Edit; Still doesn't work but when standing behind a Training Dummy and pressing Backstab once then go back spamming the macro it launches Slice and Dice. Nothing else goes on after that.

Edit2: IT'S WORKING!!! My hat goes off for you! Got it working after deleting the WTF folder and repaired WoW.exe. Don't really know what the problem was :x

[Pthree]

Ah I'm glad it works! I have no idea what could've been wrong, probably some old junk stuffed in the WTF :P

No sorry, I mainly play rogue right now, so I haven't created anything for other classes.

[k4hn]

Pthree could you please do an alternative where Rupture is up all the time? Would be greatful! =)

[k4hn]

bumping ^^

would really love a alternate with rupture up at all times :x

[Ultraviolence]

Add this anywhere in the macro for Rupture usage:

/run if not UnitDebuff("target", "Rupture", unitCaster~="player") and UnitPower("player")>=25 and GetComboPoints("player", "target")>0 and IsUsableSpell("Rupture")==1 and GetSpellCooldown("Rupture")==0 and IsSpellInRange("Rupture", "target")==1 then CastSpellByName("Rupture") end

[k4hn]

Add this anywhere in the macro for Rupture usage:

/run if not UnitDebuff("target", "Rupture", unitCaster~="player") and UnitPower("player")>=25 and GetComboPoints("player", "target")>0 and IsUsableSpell("Rupture")==1 and GetSpellCooldown("Rupture")==0 and IsSpellInRange("Rupture", "target")==1 then CastSpellByName("Rupture") end

gonna try this. +rep if it works good