simple rollonloot

ssbattousai · 10-08-12, 10:17 PM

Problem

/run rollonloot(rollID,1|2|3);

doesn't work, does anyone have a working macro that 'needs on the loot' rather than clicking on it.

Please no addons and no redirection to sites.

Then also a /run ConfirmLootRoll(rollID, 1|2|3); to complete the macro

Any ideas would be greatly appreciated.

Thank you all!

Phanx · 10-08-12, 10:19 PM

Is that your actual macro? Lua (the programming language WoW's interface uses) is case-sensitive, so "rollonloot" is not the same as "RollOnLoot".

ssbattousai · 10-08-12, 10:29 PM

Originally Posted by Phanx

Is that your actual macro? Lua (the programming language WoW's interface uses) is case-sensitive, so "rollonloot" is not the same as "RollOnLoot".

Yep, sorry I was writing it off the top of my head, here is the correct question

/script RollOnLoot(1, 2);
/script ConfirmLootRoll(1, 2);

I want that to work, any ideas?

Torhal · 10-08-12, 11:56 PM

You can't make that work in a macro. Macros execute in order, without any wait between commands. You would need to have an AddOn which listens for the CONFIRM_LOOT_ROLL event, at which point your ConFirmLootRoll(id) would be executed.

Sharparam · 10-09-12, 10:07 AM

Might be a possible with a macro such as this:

lua Code:

/run local f=CreateFrame"Frame"f:SetScript("OnEvent",function(s,e,i,r)ConfirmLootRoll(i,r)s:UnregisterEvent(e)s:SetScript("OnEvent",nil)end)f:RegisterEvent"CONFIRM_LOOT_ROLL"RollOnLoot(1,2)

This could have unintended side-effects though since the script confirms the first CONFIRM_LOOT_ROLL event, which *should* be the RollOnLoot executed right after frame creation, but it *might* not be.

Then I'm not sure about performance of creating a new frame every time the macro is called.

SDPhantom · 10-09-12, 10:35 AM

You can modify the code to store the frame in a global and check for the global before trying to create the frame. something like the following.

Lua Code:

/run local f=CRF or CreateFrame("Frame","CRF")f:SetScript("OnEvent",function(s,e,i,r)ConfirmLootRoll(i,r)s:UnregisterEvent(e)s:SetScript("OnEvent",nil)end)f:RegisterEvent"CONFIRM_LOOT_ROLL"RollOnLoot(1,2)