✓ ANSWER ACCEPTED

How would I change a SoundID using a textbox entry?

Asked Mar 16, 2020318 views3 answers

So I have a GUI in which a player can enter their sound ID for it to change the sound of the horn. However, it isn't working and is only changing the sound ID to the "rbxassetid://" part and doesn't include what the user entered. Here is my script:

repeat wait() until script:FindFirstAncestorOfClass("PlayerGui")
local player = script:FindFirstAncestorOfClass("Player").Name
local userboat = script.Parent.Parent.Parent.Ship.Value
local seat = userboat.VehicleSeat
local musicID = script.Parent.Parent.TextBox.Text

script.Parent.MouseButton1Click:Connect(function()
	print("submitted sound")
	seat.Horn.SoundId = "rbxassetid://" .. musicID
	print("sound submitted")
end)

AnonymousMar 16, 2020

1 comment

please refrain from using excessive .Parent climbs.

AnonymousMar 16, 2020#

3 answers

✓ Accepted answer

Your issue is the fact that you declared musicID as a constant; meaning that it will capture the state of which it was in after immediately being loaded, and will not reconfigure to any new input. Instead, you should capture the Text upon the MouseButton1Click signal:

--// Ensure this is a LocalScript within the GUI
local TextBoxInput = script:FindFirstAncestorOfClass("TextBox")
local Button = script.Parent

local Seat = script:FindFirstAncestor("Ship").Value.VehicleSeat
--// This pathway is redundant, please reformat

Button.MouseButton1Click:Connect(function()
    local SoundID = tonumber(TextBoxInput.Text)
    Seat.Horn.SoundId = "rbxassetid://"..SoundID
end)

AnonymousMar 16, 2020Edited Mar 16, 2020Permalink

1 comment

Thank you so much!!! It worked

AnonymousMar 17, 2020#

Server scripts cannot obtain info from TextBoxes. You'll need to use RemoteFunctions and a LocalScript to do this.

AnonymousMar 16, 2020Permalink

1 comment

I suggest using a remote event in this case because it looks like the player submits a value rather than the server requesting one.

AnonymousMar 16, 2020#

Edited

repeat wait() until script:FindFirstAncestorOfClass("PlayerGui")
local player = script:FindFirstAncestorOfClass("Player").Name
local userboat = script.Parent.Parent.Parent.Ship.Value
local seat = userboat.VehicleSeat


script.Parent.MouseButton1Click:Connect(function()
    local musicID = script.Parent.Parent.TextBox.Text
    print("submitted sound")
    seat.Horn.SoundId = "rbxassetid://"..musicID -- You put spaces here
    print("New horn sound ID: "..musicID)
end)

If this helped please accept this answer.

All the best,

PrismaticFruits

AnonymousMar 16, 2020Edited Mar 17, 2020Permalink

1 comment

It still only shows like this: https://cdn.discordapp.com/attachments/630185635663183889/689234576467427328/unknown.png

AnonymousMar 16, 2020#

This discussion is preserved as it was. New answers, comments, and votes are closed.