GUI Shop Open And Close Script Not Working? (SOLVED)

So I am making a GUI shop for my game but when I am trying to make an open/close script, it never seems to work. Am I missing something or doing something wrong? Here is what my localscript contains:

local button = game.StarterGui.Shop.openclose.openclose
local Opened = false
local ShopFrame = button.Parent.Parent.main

button.MouseButton1Click:Connect(function()
	if Opened == false then
		ShopFrame.Visible = true
		Opened = true
	else
		ShopFrame.Visible = false
		Opened = false
	end
end)

Answer: Basically I changed the script so it would work. This is what I got:

local frame = script.Parent.Parent.main
local open = false

script.Parent.openclose.MouseButton1Click:connect(function()
	if frame.Visible == false then
		frame.Visible = true
	else
		frame.Visible = false
	end
end)