[8guawong]

PHP Code:

public MenuHandler(Handle:menu, MenuAction:action, param1, param2)
{
    if (action == MenuAction_Select)
    {
        new Handle:data = CreateDataPack();
        WritePackCell(data, param1);
        SQL_TQuery(db, CheckDB "some query", data);
    }
}

public CheckDB (Handle:owner, Handle:hndl, const String:error[], any:data)
{
    new client = ReadPackCell(data);
    PrintToChat(data, "test");
} 


[SM] Native "PrintToChat" reported: Client index 431227078 is invalid

Why? How to fix?

crap nvm i was using wrong variable

[WildCard65]

you forgot to reset the pack. Also your passing a handle to a native expecting client index, I'm assuming you meant to use the variable client instead of data.

[Drixevel]

Originally Posted by 8guawong PHP Code: public MenuHandler(Handle:menu, MenuAction:action, param1, param2) {     if (action == MenuAction_Select)     {         new Handle:data = CreateDataPack();         WritePackCell(data, param1);         SQL_TQuery(db, CheckDB "some query", data);     } } public CheckDB (Handle:owner, Handle:hndl, const String:error[], any:data) {     new client = ReadPackCell(data);     PrintToChat(data, "test"); }  [SM] Native "PrintToChat" reported: Client index 431227078 is invalid Why? How to fix? crap nvm i was using wrong variable

Here's how to fix that issue:

public MenuHandler(Handle:menu, MenuAction:action, param1, param2)
{
	if (action == MenuAction_Select)
	{
		new Handle:data = CreateDataPack();
		WritePackCell(data, GetClientUserId(param1));
		SQL_TQuery(db, CheckDB "some query", data);
	}
}

public CheckDB (Handle:owner, Handle:hndl, const String:error[], any:data)
{
	ResetPack(data);
	new client = GetClientOfUserId(ReadPackCell(data));
	CloseHandle(data);
	PrintToChat(client, "test");
}

IF that's the only variable you're wanting to pass through, just pass that alone: (You probably already know this but It can help others)

public MenuHandler(Handle:menu, MenuAction:action, param1, param2)
{
	if (action == MenuAction_Select)
	{
		SQL_TQuery(db, CheckDB "some query", GetClientUserid(param1));
	}
}

public CheckDB (Handle:owner, Handle:hndl, const String:error[], any:data)
{
	new client = GetClientOfUserId(data);
	PrintToChat(client, "test");
}

[Powerlord]

As a general rule, I do ResetPack as soon as I write the last data value. That way, even if you forget to ResetPack before reading, you've already reset it.

[8guawong]

Originally Posted by r3dw3r3w0lf Here's how to fix that issue: public MenuHandler(Handle:menu, MenuAction:action, param1, param2) { if (action == MenuAction_Select) { new Handle:data = CreateDataPack(); WritePackCell(data, GetClientUserId(param1)); SQL_TQuery(db, CheckDB "some query", data); } } public CheckDB (Handle:owner, Handle:hndl, const String:error[], any:data) { ResetPack(data); new client = GetClientOfUserId(ReadPackCell(data)); CloseHandle(data); PrintToChat(client, "test"); } IF that's the only variable you're wanting to pass through, just pass that alone: (You probably already know this but It can help others) public MenuHandler(Handle:menu, MenuAction:action, param1, param2) { if (action == MenuAction_Select) { SQL_TQuery(db, CheckDB "some query", GetClientUserid(param1)); } } public CheckDB (Handle:owner, Handle:hndl, const String:error[], any:data) { new client = GetClientOfUserId(data); PrintToChat(client, "test"); }

param1 is client already

[Drixevel]

Originally Posted by 8guawong param1 is client already

During a Threaded Query, the response time isn't instant so in that situation, It's best to get the client user ID and and revert it back on the other end just in case the client disconnects during the query.

[8guawong]

Originally Posted by r3dw3r3w0lf During a Threaded Query, the response time isn't instant so in that situation, It's best to get the client user ID and and revert it back on the other end just in case the client disconnects during the query.

oh ok i'll change it just incase , thanks!