[DazhLarsson]

Why isnt this working for me?
No errors, just dont work, i dont get kicked when i join the server, and my steamid isnt in the database :s?

#pragma semicolon 1

#define DEBUG

#include <sourcemod>
#include <sdktools>

#pragma newdecls required

Database hDatabase = null;

public Plugin myinfo =
{
name = "Database test",
author = "Morten",
description = "database tester",
version = "1.0",
url = "none"
};

public void OnPluginStart()
{
Database.Connect(GotDatabase);
}


public void GotDatabase(Database db, const char[] error, any data)
{
if (db == null)
{
LogError("Database failure: %s", error);
}
else
{
hDatabase = db;
}
}

public void CheckSteamID(int userid, const char[] auth)
{
char query[255];
FormatEx(query, sizeof(query), "SELECT steamid FROM player WHERE steamid = '%s'", auth);
hDatabase.Query(T_CheckSteamID, query, userid);
}

public void T_CheckSteamID(Database db, DBResultSet results, const char[] error, any data)
{
int client = 0;

/* Make sure the client didn't disconnect while the thread was running */
if ((client = GetClientOfUserId(data)) == 0)
{
return;
}

if (results == null)
{
LogError("Query failed! %s", error);
KickClient(client, "Authorization failed");
} else if (results.RowCount == 0) {
KickClient(client, "You are not a member");
}
}

[Balimbanana]

It might depend on when CheckSteamID is being called. If you need to check SteamID's, I think the earliest you can check is by the forward:

public void OnClientAuthorized(int client, const char[] auth)

Then call CheckSteamID inside OnClientAuthorized.
Could also check what ID is being returned by having a PrintToServer("CheckSteamID %s",auth); inside CheckSteamID to see if it retrieved correctly.

[Ilusion9]

Use PHP tags for code examples.
What is CheckSteamID function?
Use OnClientPostAdminCheck function instead of CheckSteamID

[DazhLarsson]

I tried this, but i get the error: Exception reported: Error fetching data from field 1
whyyyyyyy

PHP Code:

#pragma semicolon 1

#define DEBUG

#include <sourcemod>
#include <sdktools>

#pragma newdecls required

char dbconfig[] = "MyDB";
Database g_DB;

public Plugin myinfo =
{
name = "Database test",
author = "Morten",
description = "database tester",
version = "1.0",
url = "none"
};

public void OnPluginStart()
{
    char error[255];
    g_DB = SQL_Connect(dbconfig, true, error, sizeof(error));
    SQL_SetCharset(g_DB, "utf8");
    RegConsoleCmd("sm_Morten", Morten);
}

public Action Morten(int client, int args)
{
char query[255], steamid[32];
GetClientAuthId(client, AuthId_Steam2,steamid, sizeof(steamid) );
FormatEx(query, sizeof(query), "SELECT steamid FROM player WHERE steamid = '%s'", steamid);
SQL_TQuery(g_DB, MortenCallBack, query, client);
PrintToChatAll("Test 1");
return Plugin_Handled;
}



public void MortenCallBack(Handle owner, Handle hndl, const char[] error, any data)
{
    char p1[128];
    char p2[128];
    char p3[64];
    while (SQL_FetchRow(hndl)) {
        SQL_FetchString(hndl, 1, p1, sizeof(p1));
        SQL_FetchString(hndl, 2, p2, sizeof(p2));
        SQL_FetchString(hndl, 3, p3, sizeof(p3));
        PrintToChatAll("Database : %s", p1);
}
} 


[Ilusion9]

Because you get only one field in (you are selecting only only steamid):

PHP Code:

SELECT steamid FROM player WHERE steamid = '%s' 


and you are fetching 3 fields:

PHP Code:

        SQL_FetchString(hndl, 1, p1, sizeof(p1));
        SQL_FetchString(hndl, 2, p2, sizeof(p2));
        SQL_FetchString(hndl, 3, p3, sizeof(p3)); 


Also, the first field has index 0.
Do this:

PHP Code:


public void MortenCallBack(Database db, DBResultSet rs, const char[] error, any data)
{
    if (!rs)  return;

    char steamid[65];
    if (rs.FetchRow())
    {
        rs.FetchString(0, steamid, sizeof(steamid));
        PrintToChatAll("Database : %s", steamid);
    }
} 


[DazhLarsson]

Thx for the answer, but should i do if i want to Fetch the index 2 field?

[Balimbanana]

Select all columns with:

FormatEx(query, sizeof(query), "SELECT * FROM player WHERE steamid = '%s'", steamid);

Then do the while (SQL_FetchRow(hndl)) and start from index 0. Unless index 0 is the steamid column, in which case, you could probably skip 0.

If there are a lot of columns, and you only want a few specific ones, then you should probably set up:

Transaction SQL_CreateTransaction()
int SQL_AddQuery(Transaction txn, const char[] query, any data)
void SQL_ExecuteTransaction(Handle db, Transaction txn, SQLTxnSuccess onSuccess, SQLTxnFailure onError, any data, DBPriority priority)

[DazhLarsson]

Thank you<3333