✓ ANSWER ACCEPTED

Attempt to concatenate upvalue 'username' (a userdata value)?

Asked Apr 30, 20156,523 views2 answers

local frame = script.Parent.Parent
local inputbox = frame:WaitForChild("TextBox")
local holder = frame.Parent
local display = holder:WaitForChild("display")
local imagelabel = display:WaitForChild("ImageLabel")

local userimg = 'http://www.roblox.com/Thumbs/Avatar.ashx?format=png&x=100&y=100&username='

local username = frame:WaitForChild("username")

script.Parent.MouseButton1Click:connect(function()
	username.Value = inputbox.Text
	wait()
	imagelabel.Image = userimg..username
end)

I'm currently working on a search system that relies on user input. Why can't it link username to userimg?

AnonymousApr 30, 2015

2 answers

✓ Accepted answer

Your problem is that username is an object, not a string and you tried to concatenate it.

Silly mistakes(:

So to fix? Concatenate the value of username.

local frame = script.Parent.Parent
local inputbox = frame:WaitForChild("TextBox")
local holder = frame.Parent
local display = holder:WaitForChild("display")
local imagelabel = display:WaitForChild("ImageLabel")

local userimg = 'http://www.roblox.com/Thumbs/Avatar.ashx?format=png&x=100&y=100&username='

local username = frame:WaitForChild("username")

script.Parent.MouseButton1Click:connect(function()
	username.Value = inputbox.Text
	wait()
	imagelabel.Image = userimg..username.Value --Index the value right here
end)

AnonymousApr 30, 2015Permalink

1 comment

Whoops! I usually never make that mistake. Thanks for pointing that out!

AnonymousApr 30, 2015#

As far as I can tell, username is a value object, and you can't concatenate an object.

You need to get its value or name or some other useful property so that you can properly combine strings.

userimg..username.Value
--or
userimg..username.Name
--or some other property

You also might want to look into the tostring() method in cases where you need to convert something into a string before concatenating.

AnonymousApr 30, 2015Edited Apr 30, 2015Permalink

3 comments

Ninja'd

AnonymousApr 30, 2015#

Also, the tostring function isn't needed here. Username.Value is going to be a StringValue or else it wouldn't ever work. Concatenating strings with numbers in them is fien.

AnonymousApr 30, 2015#

I know it probably isn't needed here, as was implied in that sentence. 'in cases where..." It was given for future reference.

AnonymousMay 1, 2015#

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