[UNANSWERED] Anyone know what's wrong with this GUI script?

When the player sits in the seat, the GUI should clone. When the player jumps out, the GUI should destroy itself. This just isn't happening!

Please help me!

gui = script.Parent.Controls
guiClone = nil

script.Parent.ChildAdded:connect(function(SeatWeld)
    char = SeatWeld.Part1.Parent
    player = game.Players:GetPlayerFromCharacter(char)
    guiClone = gui:Clone()
    guiClone.Parent = player.PlayerGui
end)

script.Parent.ChildRemoved:connect(function(removeWeld)
    if guiClone then
        guiClone:Destroy()
    end
end)