[Annihilator]

I've been trying to log client name and auth using OnClientPostAdminCheck(client) and I ran into a strange error.

code I'm using:

PHP Code:

public OnClientPostAdminCheck(client)
{
    decl String:auth[32];
    decl String:clientName[MAX_NAME_LENGTH];
    GetClientAuthString(client, auth, sizeof(auth));
    LogError("%s", auth);
    GetClientName(client, clientName, sizeof(clientName));
    LogError("%N", clientName);
} 


error:

PHP Code:

L 08/12/2012 - 07:40:23: [test.smx] STEAM_0:0:27319953
L 08/12/2012 - 07:40:23: [SM] Native "LogError" reported: Client index 7302741 is invalid
L 08/12/2012 - 07:40:23: [SM] Displaying call stack trace for plugin "test.smx":
L 08/12/2012 - 07:40:23: [SM]   [0]  Line 213, test.sp::OnClientPostAdminCheck() 


And I have absolutely no idea why I can retrieve steamid, but not the clients name :/
Any tips? ^^

Edit: Ups, found the misplaced %N

[Dr. McKay]

PHP Code:

public OnClientPostAdminCheck(client)
{
    decl String:auth[32];
    GetClientAuthString(client, auth, sizeof(auth));
    LogError(auth);
    LogError("%N", client);
} 


When using %N, you specify a client index, not the client's actual name.

Also, LogError("%s", auth) is silly, just specify auth directly.

[Bacardi]

Originally Posted by Dr. McKay Also, LogError("%s", auth) is silly, just specify auth directly.

out of topic.
If string contain formats like "qwerty %i %s %d asd",
you can print output this way example, LogError("%s", auth)

But this way it give you error LogError(auth), it need input values for those.

carry on

[Skyy]

you would want:

LogError("%s", clientName);

because you've stored the clients name in a string. If you wanted to use %N, you'd use the arg passed through to the function and its respecting name, i.e.:
void function functName(arg1);

LogError("%N", client);

but why not just do:

LogError("%s (%N)", auth, client);

[TnTSCS]

LogError("%L", client) ??

[11530]

TnTSCS's right (as always). %L was specifically made for logging more details about the client.

Format Class Functions:

%L: Requires a client index; expands to 1<2><3><> where 1 is the player's name, 2 is the player's userid, and 3 is the player's Steam ID. If the client index is 0, the string will be: Console<0><Console><Console>