Bad Argument @ 'getPlayerName'

[Seif]

This is killing me. This specific line is giving me the Bad Argument error yet I see nothing wrong

local pname = getPlayerName(playerSource)

It's in a command. playerSource is the first parameter in addCommandHandler so it's the player.

For some reason, pname is a boolean. This debug line tells me that I can't concatenate pname because it's a boolean value.

outputChatBox("Name: "..pname..".", playerSource)

I can't find the problem at all.

EDIT: OK this is awkward. I think it's due to me disabling /me that it's doing this, because I pasted the same lines in another command and it worked. In that case, here's how I disabled the original /me:

addEventHandler("onPlayerChat", getRootElement(), 
function (message, messageType)
outputChatBox(message)
if messageType == 1 then
cancelEvent()
triggerEvent("commandMe", source, "/me", message)
end
end
)
 
--then I added commandMe as an event so I can trigger it
addEvent("commandMe", true)
addEventHandler("commandMe", getRootElement(), commandMe)

Edited July 21, 2010 by Guest

[MaddDogg]

Is it clientside?

Then the first parameter of a command handler function is the command text.

Maybe post the whole code.

Also try a type(playerSource) and a tostring(playerSource) to see, what it really is and post results.

[Seif]

Is it clientside?

Then the first parameter of a command handler function is the command text.

Maybe post the whole code.

Also try a type(playerSource) and a tostring(playerSource) to see, what it really is and post results.

I just edited my post. And no, it's server side.

[MaddDogg]

There seems to be nothing wrong with your /me snippet.

Did you try to uncomment just these lines and see, if it worked?

I still would like to see your command code and the result from Also type(playerSource) and tostring(playerSource).

[Seif]

function commandMe(playerSource, command, param)
local arg = tostring(param)
local pname = getPlayerName(playerSource)
outputChatBox("Name: "..pname..".", playerSource)
if not param then
outputChatBox("#B4B5B7USAGE: /me [action]", playerSource, 0, 0, 0, true)
return false
end
local cmd = string.sub(arg, 1)
outputRadiusChat(playerSource, "* "..pname.." "..cmd, 20.0, 188, 94, 255, false)
end
addCommandHandler("me", commandMe)

I'm also struggling with string.sub. Isn't it suppose to help me extend param so it doesn't only catch 1 word, but all?

[Remp]

the second argument of triggerEvent is the element to trigger on (the element that will be source in any functions attached to it)

the first argument in your commandMe function will be '/me', not the player

this strikes me as quite an odd way to do it though, why not call the function directly rather than using an event? or simply even add your commandMe code into your onPlayerChat handle. there is no need to add your own /me command handler

[Seif]

I know, I forgot about executeCommandHandler. But how about my second problem?

I'm also struggling with string.sub. Isn't it suppose to help me extend param so it doesn't only catch 1 word, but all?

[MaddDogg]

Correct, but that really seems to be a popular mistake.

The source, which you provide, is never an argument, although it is in some way a "hidden" argument, if you will.

But as R3mp said, you should directly call the function rather than using triggerEvent.

Regarding your second problem, I don't really get, what you're trying to do.

But anyway, you're using string.sub incorrectly.

The second argument is not a word index, but rather a character index from where to start.

If arg would be "blah", your string.sub would return "lah".

http://www.lua.org/manual/5.1/manual.ht ... string.sub

Edited July 21, 2010 by Guest

[Remp]

string.sub splits a string at a given position ( http://lua-users.org/wiki/StringLibraryTutorial ),

to gather all input from a command you will need to use '...' ( check server example 4 here )

but again, you dont need to create your own function to handle this:

addEventHandler("onPlayerChat", getRootElement(), 
function (message, messageType)
outputChatBox(message)
if messageType == 1 then
cancelEvent()
        outputRadiusChat(source, "* "..getPlayerName(source).." "..message, 20.0, 188, 94, 255, false)
end
end
)

[Seif]

Thanks for the help, guys, I got it working well.