one more time on the this and arg stuff :)

[Grimsin]

I know i know we have gone over this before but one more time. I think i have the idea though. So in the code that will follow the mouse click is arg1 and a frame is this. These things have to be defined now via a local somthing = is how it works now right? How would you do that then? or can i ditch the arg1 being the mouse function and then define just the frame?

this is the code im attempting to change.

frame:SetScript("OnMouseDown",function(this,arg1) 
        if arg1=="LeftButton" and not self:AreBagsLocked() then 
            this:StartMoving() 
        end 
    end)

[Aznamir]

If I remember correctly, you need to replace "this" with "self" and "arg1" with "button"

[Maul]

Script handlers define their own locals implicitly. They just exist, you can say. This list will help you understand which script handler define which locals for itself -

http://www.wowwiki.com/Widget_handlers

OnMouseDown defines the frame it belongs to as "self" and which button on the mouse was clicked -

OnMouseDown(self,button)

[Seerah]

You're mixing your "this"es and "self"s. Pick one.

/edit: I just posted this earlier - http://www.wowinterface.com/forums/s...ad.php?t=35401

[SDPhantom]

Originally Posted by Aznamir If I remember correctly, you need to replace "this" with "self" and "arg1" with "button"

Defining function arguments is the exact same as choosing a name for a variable, it doesn't matter how you name it as long as it's a valid name.

[Xubera]

When your frame picks up on the MouseDown event, it calls the function you have defined.

local myFrame = CreateFrame("Frame")
myFrame:SetScript("OnMouseDown", function(self, button)
  print(self, button)
end)

every OnMouseDown your frame captures will send 2 arguments. The frame that picked up the event (myFrame in this example), and which mouse button called the event (which mouse button was pushed down)

what you call these variables above is irrelevant.

myFrame:SetScript("OnMouseDown", function(bob, steve)
print(bob, steve)
end)
should have the same output as the code above.

The big change was that some code came out like

myFrame:SetScript("OnMouseDown", function()
print(self, button)
end)

because self, button, arg1 - arg13 were stored in the global environment. Now they are not, so its up to the programmer to make sure they define the variables.