The function triggers 1 more every time it fires

[0utputt (Output)]

function addAuraToInventory(player : Player, aura)
	print("done")
	for index,slot in pairs(player.PlayerGui.MainGui.Values.Inventory:GetChildren()) do
		if slot:IsA("StringValue") then
			if slot.Value == "Empty" then
				slot.Value = aura
				local inventorygui = player.PlayerGui.MainGui.Inventory
				inventorygui.InventoryFrame[slot.Name].Text = aura
				break
			end
		end
	end
end

The function triggers 1 more every time it fires, its really annoying, can anyone help

[batteryday]

@0utputt

can anyone help

If you show us where you call the function

[0utputt (Output)]

(its inside randomizerarity function)

	player.PlayerGui.MainGui.RollFrame.EquipButton.MouseButton1Click:Connect(function()
        addAuraToInventory(player, player.PlayerGui.MainGui.Values.LastRolledAura.Value)
		replaceaura(player,player.PlayerGui.MainGui.Values.LastRolledAura.Value)
		player.PlayerGui.MainGui.SpinButton.Visible = true
		player.Values.AuraType.Value = player.PlayerGui.MainGui.Values.LastRolledAura.Value
		player.PlayerGui.MainGui.Values.LastRolledAuraReturnable.Value = false
		refreshTag(player)


	end)

[batteryday]

@0utputt

(its inside randomizerarity function)

Take it out. Every time randomizerarity is called it connects a new MouseButton1Click Event. The MouseButton1Click event should be in the outermost part of the script (meaning it is not in any other functions)

[0utputt (Output)]

Thanks! i used a remote event to fire from local script to fix it