when clicked the buttons dont work?

this is my script everythings working fine only when i press the buttons they dont work

game.Players.PlayerAdded:connect(function(plr)
    sg = Instance.new("ScreenGui",plr.PlayerGui)
    sg.Name = "Menu Gui"
    fr = Instance.new("Frame",sg)
    fr.Size = UDim2.new(0.994, 0, 0.991,0)
    fr.Position = UDim2.new(0.002, 0,0.004, 0)
    fr.BorderSizePixel = 1
    fr.Active = true
    fr.ZIndex = 1
    fr.Name = "Menu"
    fr.Style = "Custom"
    archer = Instance.new("ImageButton",fr)
    archer.BorderSizePixel = 1
    archer.Image = "http://www.roblox.com/asset/?id=147172117"
    archer.Name = "Archer button"
    archer.Position = UDim2.new(0.02,0, 00.2, 0)
    archer.Size = UDim2.new(0.3,0,0.5,0)
    warrior = Instance.new("ImageButton",fr)
    warrior.BorderSizePixel = 1
    warrior.Image = "http://www.roblox.com/asset/?id=117293269"
    warrior.Name = "Warrior Button"
    warrior.Size = UDim2.new(0.3,0,0.5,0)
    warrior.Position = UDim2.new(.35,0,.2,0)
    mage = Instance.new("ImageButton",fr)
    mage.Name = "Mage button"
    mage.Size = UDim2.new(0.3,0,0.5,0)
    mage.Position = UDim2.new(0.67, 0,.2,0)
    mage.Image = "http://www.roblox.com/asset/?id=77523689"
    mage.MouseButton1Click:connect(function(Player)
        staff = game:GetService("Lighting").Magicstaff:clone(Player.Backpack)
        sg:Destroy()
    end)
    archer.MouseButton1Click:connect(function(Player)
        bow = game:GetService("Lighting").Bow:clone(Player.Backpack)
        sg:Destroy()
    end)
    warrior.MouseButton1Click:connect(function(Player)
        sword = game:GetService("Lighting").LinkedSword:clone(Player.Backpack)
        sg:Destroy()
    end)
end)