[edon1337]

You guys are speaking no-sense. Test this code if you want.

PHP Code:

#include < amxmodx >
#include < fakemeta >
#include < hamsandwich >

#define ADMIN_FLAG ADMIN_KICK

new g_szAdminPlayerModel[ ] = "hsk_admin1"

public plugin_init( )
{
    register_plugin( "Admin Model", "1.0", "DoNii" );

    RegisterHam( Ham_Spawn, "player", "fw_HamSpawnPost", 1 );

    register_forward( FM_SetClientKeyValue, "fw_FMSetClientKeyValuePre", 0 ) ;
}

public plugin_precache( )
{
    new szBuffer[ 64 ];
    formatex( szBuffer, charsmax( szBuffer ), "models/player/%s/%s.mdl", g_szAdminPlayerModel, g_szAdminPlayerModel );

    precache_model( szBuffer );
}

public fw_HamSpawnPost( id )
{
    if( ! is_user_connected( id ) )
    return HAM_IGNORED;

    if( ~ get_user_flags( id ) & ADMIN_FLAG )
    return HAM_IGNORED;

    set_user_info( id, "model", g_szAdminPlayerModel );
    return HAM_IGNORED;
}

public fw_FMSetClientKeyValuePre( iPlayer, const szBuffer[ ], const szKey[ ] )
{
    if( equal( szKey, "model" ) )
    {    
        if( ( get_user_flags( iPlayer ) & ADMIN_FLAG ) )
        {
            set_user_info( iPlayer, "model", g_szAdminPlayerModel );
            log_to_file( "TestModel.txt", "szBuffer[] = %s | szKey = %s |", szBuffer, szKey );
            
            return FMRES_SUPERCEDE;
        }
    }
    return FMRES_IGNORED;
} 


+ I added a log file to prove you wrong.

PHP Code:

L 11/21/2017 - 22:39:52: szBuffer[] = \model\gordon\    | szKey = model |
L 11/21/2017 - 22:39:52: szBuffer[] = \model\hsk_admin1 | szKey = model |
L 11/21/2017 - 22:39:53: szBuffer[] = \model\hsk_admin1 | szKey = model |
L 11/21/2017 - 22:39:53: szBuffer[] = \model\hsk_admin1 | szKey = model | 


You see that it overrides 'gordon' model, I wrote multiple times 'model gordon' and the skin didn't change.

[aron9forever]

Originally Posted by edon1337 And what is that plugin using? FM_SetClientKeyValue.

It's also hooking the forward
register_forward(FM_SetClientKeyValue, "fw_SetClientKeyValue")

and checking every time userinfo is supposed to change (when players type model asdf in console, or when server changes it) and blocking them if they're not from the server

it's done in a neat package that handles all problems including delay to avoid overflows, that's why I recommended it, no point in reinventing the wheel



I wouldn't be talking about this if I hadn't personally tried all combinations and noticed this problem myself, which you can only do if you have a server that actually uses these functions, dickhead players willing to abuse it and high enough traffic to notice overflows when changing all skins at once



edit:: I guess I was wrong with what I understood from your suggestion, I thought it implied only changing user's info string once as that's a common error

[edon1337]

Originally Posted by aron9forever It's also hooking the forward register_forward(FM_SetClientKeyValue, "fw_SetClientKeyValue")

Originally Posted by edon1337 Use set_user_info + FM_SetClientKeyValue.

And what have I been saying all this time?

Originally Posted by aron9forever it's done in a neat package that handles all problems including delay to avoid overflows, that's why I recommended it, no point in reinventing the wheel

That plugin is more of an API, the OP needs a plugin to let user choose his skin. About the overflow, I do agree that his delay method is alright.

Originally Posted by aron9forever I wouldn't be talking about this if I hadn't personally tried all combinations and noticed this problem myself, which you can only do if you have a server that actually uses these functions, dickhead players willing to abuse it and high enough traffic to notice overflows when changing all skins at once

What problems are you talking about? I'm using almost the same method as his API, I don't get what you're trying to explain.

[aron9forever]

Originally Posted by edon1337 What problems are you talking about? I'm using almost the same method as his API, I don't get what you're trying to explain.

if you don't set a delay between model changes, and you change all player models at 32 slot they can easily get disconnected because of an overflow. The API also includes this delaying regardless of the number of clients you execute on every time you use the command.

there's no point in rewriting all of this inside another plugin when you'll have to copy it in every plugin where you want to change a model, just use the API, along with the team change API which is also as useful

[indraraj striker]

Originally Posted by Saint Sinner After chose a model , model does not change , why? #include < amxmodx > #include < cstrike > new const Plugin[] = "Models Menu"; new const Version[] = "1.0"; new const Author[] = "="; //-------------------------------------------------------------------------- new const Model_One[] = "models/player/Kloun/kloun.mdl"; new const Model_Two[] = "models/player/Naruto/naruto.mdl"; new const Model_Theree[] = "models/player/Marihuana/marihuana.mdl"; new const Model_Four[] = "models/player/Jason/jason.mdl"; new const Model_Five[] = "models/player/Jack/jack.mdl"; #define Admin_Access ADMIN_RCON //-------------------------------------------------------------------------- new g_Menu; public plugin_precache() { precache_model( Model_One ); precache_model( Model_Two ); precache_model( Model_Theree ); precache_model( Model_Four ); precache_model( Model_Five ); } public plugin_init() { register_plugin( Plugin, Version, Author ); register_clcmd( "say /skin", "ClientCommand_Menu" ); register_clcmd( "say_team /skin", "ClientCommand_Menu" ); g_Menu = menu_create( "Select your model:", "Model_Menu_Handler" ); //-------------------------------- menu_additem( g_Menu, "KloYyn" ); menu_additem( g_Menu, "Narut0o" ); menu_additem( g_Menu, "MariHuana" ); menu_additem( g_Menu, "Jaso0on" ); menu_additem( g_Menu, "JacCcKk" ); //-------------------------------- } public ClientCommand_Menu( id ) { if( get_user_flags( id ) & Admin_Access && is_user_alive( id ) && cs_get_user_team( id ) == CS_TEAM_CT ) { menu_display( id, g_Menu ); } else { client_print( id, print_chat, "This command is not valid for you!" ); } } public Model_Menu_Handler( id, menu, item ) { if( item != MENU_EXIT ) { switch( item ) { case 0: { cs_set_user_model( id, "Kloun" ); client_print( id, print_chat, "You chose the Kloun model" ); } case 1: { cs_set_user_model( id, "Naruto" ); client_print( id, print_chat, "You chose the Naruto model" ); } case 2: { cs_set_user_model( id, "Marihuana" ); client_print( id, print_chat, "You chose the Marihuana model" ); } case 3: { cs_set_user_model( id, "Jason" ); client_print( id, print_chat, "You chose the Jason model" ); } case 4: { cs_set_user_model( id, "Jack" ); client_print( id, print_chat, "You chose the Jack model" ); } } } }

update your amxmodx this issue has been fixed in latest update
Don't use Reamxmodx