[boynedmaster]

Here's my code.

public OnClientAuthorized(client, const String:auth[]){
	PrintToServer("error 1");
	
	if(!IsFakeClient(client)){
		PrintToServer("error 2");
		GetClientAuthString(client, clientauth, sizeof(clientauth));
		
		Format(query, sizeof(query), "SELECT * FROM accounts WHERE `steamid`='%s'", clientauth);
		
		new Handle:q = SQL_Query(db, query);
		
		if(q == INVALID_HANDLE){
			SQL_GetError(db, error, sizeof(error));
			PrintToServer("Query fail: %s", error);
			PrintToServer("error 2.5");
		}
		
		PrintToServer("error 3");
		
		if(SQL_GetRowCount(q) == 0){
			PrintToServer("error 4");
			Format(query, sizeof(query), "INSERT INTO accounts(`steamid`) VALUES('%s')", clientauth);
			PrintToChat(client, "Thanks for joining! You've been put in our database.");
		}
	}
}

I only get "error 1" and "error 2".
I don't get any errors in the server. What's happening?

[boynedmaster]

do we have some top notch programmers here or what

[Dr. Greg House]

Sorry we don't even have Notch.

[blaacky]

If error 3 doesn't show up and you have no errors then it probably means your server is freezing from that query.

[friagram]

You should use threaded queries, and your print message will probably fail, since authorize can happen before they are in game.

[Bacardi]

...don't add `these` in your sql query
you get only confused, even MySQL or else show output like that way.
When parameter is string, then use 'these'

PHP Code:

Format(query, sizeof(query), "SELECT * FROM accounts WHERE `steamid`='%s'", clientauth); 


PHP Code:

Format(query, sizeof(query), "SELECT * FROM accounts WHERE steamid='%s'", clientauth); 


Try first that.

*edit
Maybe problem is your connection to database.

[boynedmaster]

Originally Posted by Bacardi ...don't add `these` in your sql query you get only confused, even MySQL or else show output like that way. When parameter is string, then use 'these' PHP Code: Format(query, sizeof(query), "SELECT * FROM accounts WHERE `steamid`='%s'", clientauth);  PHP Code: Format(query, sizeof(query), "SELECT * FROM accounts WHERE steamid='%s'", clientauth);  Try first that. *edit Maybe problem is your connection to database.

Sourcemod doesn't say any connection error. Also, I've been using `` in MySQL forever. It's fixed so many errors for me.

[Doodil]

Have you checked if you actually recieve a auth string? I remember having problems with OnClientAuthorized (years ago) and had to use OnClientPostAdminCheck() to retrieve the steam-id.

My guess is your clientauth is empty and breaks your query (btw, is that a single global variable for every client?).

Check if GetClientAuthString() returns true and report back

[xf117]

Why do you even GetClientAuthString in OnClientAuthorized? You already have auth in that callback available for you.

[Bittersweet]

That can't be the complete code. db is not defined, nor the error string. You should use threaded queries. I have a plugin that does the same thing, I use threaded queries, don't use `` around fields in the query, and I do the check after a 1 second timer in OnClientPostAdminCheck.

You said you don't get any errors in the server, what about the error logs?