Dominos

Dominos

19M Downloads

Quest Tracker

Goranaws opened this issue ยท 6 comments

commented

When I created Dominos_Minimap, I also created a quest tracker frame to go with it. Since doing so I've decided to make the Tracker into it's own addon... I figured since Dominos has had issues with the tracker in the past, it would be a good idea. It reuses the default tracker, while enhancing it as well.

Here's a link to the code if you'd like to take a look.

https://github.com/Goranaws/Dominos_Quest/blob/master/Dominos_Quest.lua

(OH! I finally figured out how to make repositories!)

commented

Sorry for the delay in responding. It looks good to me, but I'm worried a bit about line 221: That may or may not cause tainting issues, and if so, you should probably look at replacing it with a secure hook.

Jason

On Jul 20, 2011, at 2:01 PM, Goranaws wrote:

When I created Dominos_Minimap, I also created a quest tracker frame to go with it. Since doing so I've decided to make the Tracker into it's own addon... I figured since Dominos has had issues with the tracker in the past, it would be a good idea. It reuses the default tracker, while enhancing it as well.

Here's a link to the code if you'd like to take a look.

https://github.com/Goranaws/Dominos_Quest/blob/master/Dominos_Quest.lua

(OH! I finally figured out how to make repositories!)

Reply to this email directly or view it on GitHub:
https://github.com/Tuller/Dominos/issues/64

commented

One Question... I look around online, but I just can't seem to understand how to create a secure script... Any chance you could point me somewhere that explains really well?

commented

Like this??

hooksecurefunc(WatchFrame, "SetWidth", function() return end)

If so, it doesn't work... I think it's due to the way it's written...

function WatchFrame_SetWidth(width)
--[[ do stuff ]]--
end

Instead of messing with the desired function, it messes with " WatchFrame:SetWidth() "... Needless to say, I'm lost.

commented

http://wowprogramming.com/docs/api/hooksecurefunc is the call you'd use.
Basically what you want to try to avoid doing is

Here's a brief article on the concept of secure execution and tainting:
http://www.wowpedia.org/Secure_Execution_and_Tainting

commented

When you hook a function, what you're basically doing is something like this:

local oldSetWidth = WatchFrame.SetWidth
WatchFrame.SetWidth = function(self, width)
oldSetWidth(self, width)
--your code
end

if you're trying to hook WatchFrame_SetWidth, then you probably just want to do this:
hooksecurefunc(WatchFrame_SetWidth, function(width)
--your code
end)

On Jul 23, 2011, at 3:18 PM, Goranaws wrote:

Like this??

hooksecurefunc(WatchFrame, "SetWidth", function() return end)

If so, it doesn't work... I think it's due to the way it's written...

function WatchFrame_SetWidth(width)
--[[ do stuff ]]--
end

Instead of messing with the desired function, it messes with " WatchFrame:SetWidth() "... Needless to say, I'm lost.

Reply to this email directly or view it on GitHub:
https://github.com/Tuller/Dominos/issues/64#issuecomment-1638404

commented

I'll test it out right now... I just finished reinstalling Github for Mac... Lion kinda ticked me off, had way to many defaults setting I hate all re-enabed, and also screwed up Github For Mac. :|