Have to click gui multiple times?

Summary

Basically, I have an inventory system in my game. You touch the item on the ground, it copies the id of that item into the slot in your inventory. When you click on the inventory slot, the other slots fade out and an "equip" button appears. You equip the item, the gui returns to normal and it sets the "WeaponEquipped" bool value in the player to true, then the weapon is cloned into your backpack. It also sets the text of the equip button for that slot to "Unequip", so the next time the player clicks the gui, the button says unequip instead of equip. Once you press unequip, the button returns to normal and the item is removed from your backpack.

Problem

Everything works as is, however, I'm having a weird issue where I need to click the gui buttons multiple times before they ever actually do anything. I have no idea how to fix this.

Code

local equip = false
local player = game.Players.LocalPlayer

for _,buttons in pairs(script.Parent:GetChildren())

do
	if buttons:IsA("TextButton") then
	buttons.MouseButton1Click:connect(function(click)
		if equip == false then
			buttons.Equip.Visible = true
			equip = true
			for _,otherbuttons in pairs(script.Parent:GetChildren())
			do
				if otherbuttons:IsA("TextButton") then
					otherbuttons.Visible = false
				end
			end
			buttons.Visible = true
			buttons.Equip.MouseButton1Click:connect(function(cl)
				
				buttons.Equip.Visible = false
				
				local id = buttons.Item.Value 
				if buttons.Equip.Text == "Equip" then
					if id > 0 then 
						if id < 2000 then 
							if player.Status.WeaponEquipped.Value == false then 
							
							for _,tools in pairs(game.ReplicatedStorage:GetChildren())
							do
								if tools:IsA("Tool") then
									if tools.itemid.Value == id then
										print("Tool found: "..tools.Name)
										local clone = tools:Clone()
										clone.Parent = game.Players.LocalPlayer.Backpack
										player.Status.WeaponEquipped.Value = true 
										buttons.Equip.Text = "Unequip"
									end
								end
								end
								
							end
						end
					end
				
				else
					--unequip
					buttons.Equip.Text = "Equip" 
					player.Backpack:ClearAllChildren() 
					player.Status.WeaponEquipped.Value = false
				end
				
				for _,otherbuttons in pairs(script.Parent:GetChildren())
				do
					if otherbuttons:IsA("TextButton") then
						otherbuttons.Visible = true
					end
				end
				equip = false
			end)
		else
			buttons.Equip.Visible = false 
			equip = false
			for _,otherbuttons in pairs(script.Parent:GetChildren())
			do
				if otherbuttons:IsA("TextButton") then
					otherbuttons.Visible = true
				end
			end
		end
	end)	
	end
end