Argument 1 missing or nil while trying to get Sound name

Help and Feedback Scripting Support
, , , ,
#4

As you can see it writes id right, but still has argument issue:

image

#5

What is the first nil about in the output?

2 Likes
#6

Same results and still nothing:

image

#7

its about this stroke:

text2.Text = game:GetService("MarketplaceService"):GetProductInfo(id,Enum.InfoType.Asset).Name
#8

So the problem is, that the name doesn’t show up?

1 Like
#9

Little tip, you don’t have to create an entirely new function when using pcalls.

local marketplaceService = game:GetService('MarketplaceService')
local success, info = pcall(marketplaceService.GetProductInfo, marketplaceService, soundId, Enum.InfoType.Asset)
1 Like
#10

You could try it without the Enum.Infotype argument.

1 Like
#11

yes that’s the problem

30ch

1 Like
#12

id isn’t being assigned a value. Try printing sound.SoundId, then printing id after :match().

1 Like
#13

image

#14

Also, I’m not smart enough to understand, what :match() does.

Could anyone explain?

1 Like
#15

isn’t this how it works?

local id = Sound.SoundId:match("%d+")
#16

Like, what does it change about the id?

1 Like
#17

I thimk this post might explain. I took it from here: How to get Sound name from Sound.SoundId? - #7 by Flubberlutsch

#18

Yes but for whatever reason it’s returning nil which is why your function is erroring.

    print(Sound.SoundId)
    local id = Sound.SoundId:match("%d+")
    print(id)

Also, you should switch Sound.Changed to Sound:GetPropertyChangedSignal(‘SoundId’) as .Changed is going to listen for any property which might cause it to fire prematurely.

2 Likes
#19

As you can see it write id, but still has argument issue:
image

#20

I think you could try it without the :match()?

1 Like
#21

I see, it’s only erroring the first time because it doesn’t have a value when .Changed is initially fired.

Add this to your function to check that the property has a value:

if not sound.SoundId then
    return
end
2 Likes
#22

I think it might be an empty string, not nil, so I would check for the empty string instead. Checking both wouldn’t hurt though.

Sound.Changed:Connect(function()
	if not Sound.SoundId or Sound.SoundId == "" then
		return
	end

	-- ...
end)
2 Likes
#23

It worked with it, thank you so much for helping

2 Likes