GUI in-game sound library

[XxMattvdbraakXx (IayZBoy)]

So i want to make a sound library (of all my ingame sounds) for my admin panel, But all attempts have failed. It is not cloning the placeholder TextButton.
Current script:

for _,v in pairs(workspace:GetDescendants()) do
	if v:IsA("Sound") then
		local clone = script.Parent.PlaceHolder:Clone()
		local id = v.SoundId:match("%d+")
		local info = game:GetService("MarketplaceService"):GetProductInfo(id, Enum.InfoType.Asset)
		clone.Name = v.Name
		clone.Text = info.Name
		clone.MouseButton1Click:Connect(function()
			game.ReplicatedStorage.RemoteEvents.Music:FireServer(id)
		end)
		task.wait()
	end
end

Help is highly appreciated!

[majdTRM]

Don’t forget to set the parent of the clone.

e.g.

clone.Parent = script.Parent

[bonkybincer]

Did you forget to set the parent of the clone? I don’t see it anywhere in your script.

[XxMattvdbraakXx (IayZBoy)]

oml-
bruhhh
how did i forget…
adding rn

[XxMattvdbraakXx (IayZBoy)]

Result after doing so.

[XxMattvdbraakXx (IayZBoy)]

Still no cloning of the textbuttons.

[majdTRM]

Maybe they are stacked on top of each other? Check the explorer pane.

[XxMattvdbraakXx (IayZBoy)]

I am using a UIGridLayout, So that is not the issue.

[majdTRM]

Are there any errors in the Output pane? (View>Output) Try using the debugger to step through the script and see where it stops/doesn’t happen.

[XxMattvdbraakXx (IayZBoy)]

There are no errors at all, I have checked it.

[bonkybincer]

Try removing match(%d+) from local id = v.SoundId:match("%d+").

Edit: I’m playing the guessing game now, I don’t know what the culprit is anymore. It may be outside of the script provided.

[XxMattvdbraakXx (IayZBoy)]

The server script already has rbxassetid:// pre-set, So the :match(%d+) searches for the numbers only.

[bonkybincer]

You are using game:GetService("MarketplaceService"):GetProductInfo() in the local script, (which i believe requires rbxassetid:// to work properly) so it isn’t actually getting product info.

Can’t you just add id:match(%d+") to game.ReplicatedStorage.RemoteEvents.Music:FireServer(id) instead?

[XxMattvdbraakXx (IayZBoy)]

This is something i have to tell my game version, As u can see, It does not require rbxassetid://

[XxMattvdbraakXx (IayZBoy)]

Accidentally had the script disabled.