Can someone help me with this?

Can someone help? I'm using a GUI and this script doesn't work for it.

Here's the script:

script.Parent.MouseButton1Click:connect(function() MainShop = script.Parent.Parent.MainShop if mainShop.Visible == false then mainShop.Visible = true script.Parent.Text = "Close" else mainShop.Visible = false script.Parent.Text = "Open" end end)

Whenever I click the button, it doesn't open up. Is this script outdated? Someone please help me.