[exvel]

Also you can get player's steamid only after OnClientAuthorized is called. The best work around will be to move all your code in this forward.

[Raver]

Okay i has try it.
Here is the Code:

public Action:Event_PlayerConnect(Handle:event, const String:name[], bool:dontBroadcast)
{
	if(GetConVarInt(DisableStandardJoinMsg))
	{
		if(!dontBroadcast)
		{
			SetEventBroadcast(event, true);
		}
	}
	if(GetConVarInt(JoinMessageEnabled) == 1)
	{
		new client = GetClientOfUserId(GetEventInt(event, "userid"));
		new String:clientname[24];
		GetClientName(client, clientname, sizeof(clientname));
		new String:ip[18];
		GetClientIP(client, ip, sizeof(ip));
		new String:country[45];
		GeoipCountry(ip, country, sizeof(country));
		
		JoinMessage(clientname, country);
	}
	return Plugin_Continue;
}

And here is the message from the Console:

L 03/08/2010 - 18:58:09: [SM] Native "GetClientIP" reported: Client index 0 is invalid
L 03/08/2010 - 18:58:09: [SM] Displaying call stack trace for plugin "msgchange.smx":
L 03/08/2010 - 18:58:09: [SM]   [0]  Line 83, E:\Server-Stuff\msgchange.sp::Event_PlayerConnect()

[Frus]

add this after you get the client number

if (!client) return Plugin_Continue;

[Raver]

Dont work...
Code:

public Action:Event_PlayerConnect(Handle:event, const String:name[], bool:dontBroadcast)
{
	if(GetConVarInt(DisableStandardJoinMsg))
	{
		if(!dontBroadcast)
		{
			SetEventBroadcast(event, true);
		}
	}
	if(GetConVarInt(JoinMessageEnabled) == 1)
	{
		new client = GetClientOfUserId(GetEventInt(event, "userid"));
		new String:clientname[24];
		GetClientName(client, clientname, sizeof(clientname));
		new String:ip[18];
		GetClientIP(client, ip, sizeof(ip));
		new String:country[45];
		GeoipCountry(ip, country, sizeof(country));
		
		JoinMessage(clientname, country);
		if (!client)
		{
			return Plugin_Continue;
		}
	}
	return Plugin_Continue;
}

L 03/08/2010 - 19:06:35: [SM] Native "GetClientIP" reported: Client index 0 is invalid
L 03/08/2010 - 19:06:35: [SM] Displaying call stack trace for plugin "msgchange.smx":
L 03/08/2010 - 19:06:35: [SM]   [0]  Line 83, E:\Server-Stuff\Smsgchange.sp::Event_PlayerConnect()

[Frus]

By 'after' I quite literally mean immediately after :S (the next line)

[asherkin]

use,

if (!client || client == 0)

and like he said, directly after.

[p3tsin]

PHP Code:

new server = 0;
if (!client || client == 0 || client == server) 


Better triple check it to be completely sure.

[cmptrwz]

PHP Code:

new server = 0;
if (!client || client == 0 || client == server || !(client > 0 && client <= MaxClients)) 


[asherkin]

lol, yea, I posted without thinking xD.
What I get for posting as I'm going out.

[exvel]

Originally Posted by p3tsin PHP Code: new server = 0; if (!client || client == 0 || client == server)  Better triple check it to be completely sure.