(Joke Addon) How do I make it so that when you bonus roll AP, your WoW does ForceQuit

[Buttonmasher]

I want to make a quick joke addon because somebody said, "i want to write an addon that forces wow to close whenever i bonus roll artifact power."

I can't test it because I don't have any bonus rolls and can't guarantee they'd hit AP, so I wanted to verify that this indeed would work with more knowledgeable people:

local frame = CreateFrame("FRAME");

function frame:OnEvent(event, arg1)
	local rewardType
	if (event == "BONUS_ROLL_RESULT") then
		if rewardType == "artifact_power" then
			ForceQuit()
		end
	end
end


frame:RegisterEvent("BONUS_ROLL_RESULT");

[Deleted]

I would've expected ForceQuit to require a hardware event, but that doesn't seem to be the case, so I guess this should work.

local frame = CreateFrame("Frame")

frame:RegisterEvent("BONUS_ROLL_RESULT")

frame:SetScript("OnEvent", function(self, event, rewardType)
	if rewardType == "artifact_power" then
		ForceQuit()
	end
end)

I'm not sure about the possible values of rewardType, but I assume that you did your research and "artifact_power" is right.

Event handlers (OnShow, OnUpdate, OnEvent, etc.) are set with the SetScript method, or hooked with HookScript. The corresponding arguments are passed to the function. Since there's only 1 event registered, there's no need to see which event triggered the function.

local rewardType

This just creates a local variable with nil value. Even if rewardType had some value in the global scope, it would now be overwritten by the nil value in the local scope. It's equivalent to:

local rewardType = nil

[SpaceDuck]

Try this:

local frame = CreateFrame("FRAME");

function frame:OnEvent(self, event, ...)
	if (event == "BONUS_ROLL_RESULT") then
           local rewardType = ...;
		if rewardType == "artifact_power" then
			ForceQuit()
		end
	end
end


frame:RegisterEvent("BONUS_ROLL_RESULT");

[Deleted]

Try this:

local frame = CreateFrame("FRAME");

function frame:OnEvent(self, event, ...)
	if (event == "BONUS_ROLL_RESULT") then
           local rewardType = ...;
		if rewardType == "artifact_power" then
			ForceQuit()
		end
	end
end


frame:RegisterEvent("BONUS_ROLL_RESULT");

I think you meant:

local frame = CreateFrame("FRAME");

function frame:OnEvent(self, event, ...)
  if (self == "BONUS_ROLL_RESULT") then
    local rewardType = event;
    if rewardType == "artifact_power" then
      ForceQuit()
    end
  end
end

frame:SetScript("OnEvent", frame.OnEvent);

frame:RegisterEvent("BONUS_ROLL_RESULT");

I'll let you figure out yourself why this would work, but what you posted wouldn't. It has something to do with explicit method declarations and syntactic sugar.

[Kanegasi]

I think you meant:

local frame = CreateFrame("FRAME");

function frame:OnEvent(self, event, ...)
  if (self == "BONUS_ROLL_RESULT") then
    local rewardType = event;
    if rewardType == "artifact_power" then
      ForceQuit()
    end
  end
end

frame:SetScript("OnEvent", frame.OnEvent);

frame:RegisterEvent("BONUS_ROLL_RESULT");

I'll let you figure out yourself why this would work, but what you posted wouldn't. It has something to do with explicit method declarations and syntactic sugar.

I'm going to take your comment as a challenge, so I think you meant this:

local f=CreateFrame('frame')
f:RegisterEvent('BONUS_ROLL_RESULT')
f:SetScript('OnEvent',function(_,_,rewardtype)
    if rewardtype=='artifact_power' then ForceQuit() end
end)

[Deleted]

I'm going to take your comment as a challenge, so I think you meant this:

local f=CreateFrame('frame')
f:RegisterEvent('BONUS_ROLL_RESULT')
f:SetScript('OnEvent',function(_,_,rewardtype)
    if rewardtype=='artifact_power' then ForceQuit() end
end)

Well, if you look at the first reply to this thread, that is basically exactly the same thing I posted, although in a slightly more compact form.

In my later reply I was just jokingly making a point about how the arguments in the code SpaceDuck posted are seemingly shifted (thus self == "BONUS_ROLL_RESULT" and rewardType = event).

[Kanegasi]

Well, if you look at the first reply to this thread, that is basically exactly the same thing I posted, although in a slightly more compact form.

In my later reply I was just jokingly making a point about how the arguments in the code SpaceDuck posted are seemingly shifted (thus self == "BONUS_ROLL_RESULT" and rewardType = event).

Ahh, I see. I need to read further before replying next time.