[sebxx4]

Hello everyone.
I got a problem - I want to connect to sql and check if player has acccess flags. Here's my example code:

PHP Code:

#include <amxmodx>
#include <amxmisc>
#include <sqlx>

#define PLUGIN "sqlVIPloader"
#define VERSION "1.0"
#define AUTHOR "Sebxx"

new c_db_host, c_db_user, c_db_pass, c_db_name, c_serv_ip
new db_host[33], db_user[33], db_pass[33], db_name[33], serv_ip[23]
new Handle:g_SqlTuple

public plugin_init() 
{
    register_plugin(PLUGIN, VERSION, AUTHOR)

    c_db_host = register_cvar("shop_sql_host", "127.0.0.0")
    c_db_user = register_cvar("shop_sql_user", "root")
    c_db_pass = register_cvar("shop_sql_pass", "")
    c_db_name = register_cvar("shop_sql_db", "shop")
    c_serv_ip = register_cvar("this_serv_ip", "127.127.127.127:27015")

    new cfg_loc[33]
    get_configsdir( cfg_loc, charsmax(cfg_loc) )
    server_cmd( "exec %s/st_shop_sql.cfg", cfg_loc )
}

public client_authorized( id, const authid[] )
{
    if ( !is_user_bot(id) && !is_user_hltv(id) )
    {
        get_pcvar_string( c_db_host, db_host, 32 )
        get_pcvar_string( c_db_user, db_user, 32 )
        get_pcvar_string( c_db_pass, db_pass, 32 )
        get_pcvar_string( c_db_name, db_name, 32 )
        get_pcvar_string( c_serv_ip, serv_ip, 22 )

        g_SqlTuple = SQL_MakeDbTuple( db_host, db_user, db_pass, db_name )

        new query[512], user_ip[32], name[32], steamid[35], pw_info[32], pw_hash[34]

        get_user_ip( id, user_ip, charsmax(user_ip), 1 )
        get_user_name( id, name, charsmax(name) )
        get_user_authid( id, steamid, charsmax(steamid) )
        get_user_info( id, "_pw", pw_info, 31 )
        hash_string( pw_info, Hash_Md5, pw_hash, charsmax(pw_hash) )

        formatex(query, 511, "SELECT sc.access \
                                FROM shop_services AS sc, shop_servers AS sv \
                                WHERE sv.serv_ip = '%s' \
                                AND sc.server_id = sv.serv_id \
                                AND ( ( sc.auth_id = '%s' AND sc.auth = 1 ) OR ( sc.auth_id = '%s' AND sc.password = '%s' AND sc.auth = 2 ) OR ( sc.auth_id = '%s' AND sc.password = '%s' AND sc.auth = 3 ) ) \
                                AND sc.expire > %d", serv_ip, steamid, name, pw_hash, user_ip, pw_hash, get_systime() )

        new Data[1]
        Data[0] = id

        SQL_ThreadQuery(g_SqlTuple, "do_query", query, Data, 1)
    }
}

public do_query( FailState, Handle:Query, Error[], Errcode, Data[], DataSize )
{
    if ( FailState == TQUERY_CONNECT_FAILED )
    {
        log_amx("Could not connect to SQL database.")

        if ( Errcode )
        {
            log_amx("Error: %s", Error)
        }

        return PLUGIN_CONTINUE
    }
    else if ( FailState == TQUERY_QUERY_FAILED )
    {
        log_amx("Query failed.")

        if ( Errcode )
        {
            log_amx("Error: %s", Error)
        }

        return PLUGIN_CONTINUE
    }

    new id = Data[0]

    if ( SQL_NumResults(Query) )
    {
        new flags[32]
        
        while( SQL_MoreResults( Query ) )
        {
            new qcolAccess = SQL_FieldNameToNum( Query, "access" )
            SQL_ReadResult( Query, qcolAccess, flags, 31 )
            //log_amx( "Flag: %s", flags )
            set_user_flags( id, read_flags( flags ) )
            SQL_NextRow(Query)
        }
    }

    SQL_FreeHandle(g_SqlTuple)
    return PLUGIN_CONTINUE
} 


The problem is, set_user_flags don't set it. SQL query works well (I see in the console flags get from sql server) but player don't have permissions. What's wrong?

[Bugsy]

If you've confirmed all the plumbing is correct and you see the log_amx() line print, maybe try adding a small delay when setting flags.

            //set_user_flags( id, read_flags( flags ) )
            set_task( 0.2 , "DelayFlags" , id , flags , sizeof( flags ) );
            
            SQL_NextRow(Query)
        }
    }
    
    SQL_FreeHandle(g_SqlTuple)
    
    return PLUGIN_CONTINUE
}

public DelayFlags( const szFlags[] , id )
{
    set_user_flags( id , read_flags( szFlags ) );
}

[sebxx4]

Now, I got it like this:

PHP Code:

    new id = Data[0]
    new flags[32], output[1024]

    if ( SQL_NumResults(Query) )
    {
        while( SQL_MoreResults( Query ) )
        {
            new qcolAccess = SQL_FieldNameToNum( Query, "access" )
            SQL_ReadResult( Query, qcolAccess, flags, 31 )
            add( output, sizeof(output), flags, sizeof(flags) )
            SQL_NextRow(Query)
        }
    }

    SQL_FreeHandle(g_SqlTuple)

    set_task( 0.2, "set_flags", id, output, sizeof(output) );

    return PLUGIN_CONTINUE
}

public set_flags( const flags[], id  )
{
    set_user_flags( id, read_flags( flags ) )
    log_amx( "Set flags: %s to player: #%d", flags, id )
} 


But it still don't set access flags :/ I have logged in console "Set flags: t to player: #1" so it should work...

[sebxx4]

EDIT:
I found there is something wrong with one of my servers. When testing on another one, it works well. And if I use SQL_PrepareQuery instead of SQL_ThreadQuery it works well on all servers too.
I think the problem is in set_user_flags function bacause even if set constant values ( eg: set_user_flags(1, read_flags("t")) ) it doesn't work. What you think about it?

[Bugsy]

Remember that set_user_flags() is actually add user flags. If you want to set what a player has, first use remove_user_flags() then set_user_flags().

For example, if a player has "abcdef" and you call set_user_flags( id , "t" ), his flags will then be "abcdeft"

Is this your issue?

Do you have any other plugins that may be adjusting flags?

[sebxx4]

I know it only adds flags. That's I want it to do.
I think I found the problem, but still don't know how to deal with it. The problem is, most of plugins checks user flags in client_authorized() function. But my plugin do only sql query in client_authorized(), but no set_user_flags() in this function too. And bacause of it, rest of plugins don't see user flags. They're updated in sql function after client_authorized.

[Bugsy]

Ok, you may need to edit those plugins to delay the flag checking.

[sebxx4]

Well, I'd rather avoid editing every plugin I use because of this one.

[Bugsy]

You've really got no choice. The earliest that you can identify a player with their authid is client authorized. Another option is to load flags with something more instantaneous, like nVault, and place this plugin before all others in plugins.ini

[CrazY.]

Rather than putting a delay in every plugin, create a forward and trigger it once the user flags are loaded, you will still need to edit the plugins though, there is no other way around it.