[Fuck For Fun]

so yeah i set new player = cmd_target
and check it by player/client

but someone when im using

client_cmd( client, "amx_kick #%d", get_user_userid( player ) );

ERROR:
You must select a valid player.

public HandleCommands( client, const message[ 192 ], command )
{
	new cmd[ 64 ], arg[ 64 ], arg1[ 64 ], arg2[ 64 ], arg3[ 64 ], AdminSz[ 32 ], NameSz[ 32 ];
	
	parse( message, cmd, charsmax( cmd ), arg, charsmax( arg ), arg1, charsmax( arg1 ), arg2, charsmax( arg2 ), arg3, charsmax( arg3 ) );
	
	//client_print(0, print_chat, "[DEBUG] arg:%s arg1:%s arg2:%s", arg, arg1, arg2)
	new player = cmd_target( client, arg, arg1, command )
	//client_print(0, print_chat, "[DEBUG] player(%d)",)
	
	get_user_name( client, AdminSz, charsmax(AdminSz))

	if (player != 0)
		get_user_name( player, NameSz, charsmax(NameSz))
	else
		copy(NameSz, charsmax(NameSz), arg1);
		
	switch( command )
	{
		case KICK:
		{
			if (!player)
			{
				ColorChat(client, "^3Error^1: You must select a valid player.");
			} 
			else if (!is_user_connected(player))
			{
				ColorChat( client, "'^3%s^1' has can't be Kicked, Because he's ^4disconnected^1.", NameSz);
			}
			else
			{
				ColorChat( 0, "^3%s^1 has used ^4Kick^1 command.", AdminSz)
				//client_cmd( client, "amx_kick %s", arg );
				client_cmd( client, "amx_kick #%d", get_user_userid( player ) );
			}
		}
		
		case BAN:
		{
			if (!player)
			{
				ColorChat( client, "^3Error^1: You must select a valid player.");
			} 
			else if (!is_user_connected(player))
			{
				ColorChat( client, "'^3%s^1' has can't be Banned, Because he's ^4disconnected^1.", NameSz);
			}
			else
			{
				ColorChat( 0, "^3%s^1 has used ^4Ban^1 command.", AdminSz)
				client_cmd( client, "amx_ban %s %s %s", arg, arg1, arg2 );
			}
		}

And that case are working:

		case SLAY:
		{
			if (!player)
			{
				ColorChat(client, "^3Error^1: You must select a valid player.");
			}
			else if (!is_user_alive(player))
			{
				new szName[32];
				get_user_name(player, szName, charsmax(szName));
				ColorChat( client, "'^3%s^1' can't be Slay, Because he's ^4dead^1.", szName);
			}
			else if ( !( get_user_flags( player ) & ADMIN_IMMUNITY ) || player == client )
			{
				ColorChat( 0, "^3%s^1 has used ^4Slay^1 command.", AdminSz)
				
				user_kill( player );
			}
			else
			{
				if (get_user_flags(player) & ADMIN_IMMUNITY)
					ColorChat( client, "^3Error^1: You can not use this command on Owner")
				else
					ColorChat( client, "^3Error^1: Please enter a player name.")
			}
		}

		case VOTEMAP:
		{
			ColorChat(0, "^3%s^1 making a ^4Vote Map^1.",AdminSz)
			
			client_cmd( client, "amx_votemap %s %s %s %s", arg, arg1, arg2, arg3 );
		}

[OciXCrom]

new player = cmd_target( client, arg, arg1, command )

For starters, that code won't even compile because "cmd_target" has only 3 arguments, not 4.

https://www.amxmodx.org/api/amxmisc/cmd_target

Give us a working code and show the definition for all arguments involved in your "cmd_target" execution, including "message" and "command" values.

[fysiks]

You're redefining functions that already exist. Don't do that. Give your custom function a unique name (even if it does work, it is extremely confusing, especially if you're changing the arguments that get passed to it).

Write a small plugin to test just the functionality that you're having problems with. This will often make it easier to find the issue. The other benefit is that, if you can't fix it, you can post just this version of the plugin so that we can actually run it.

[Fuck For Fun]

Originally Posted by fysiks You're redefining functions that already exist. Don't do that. Give your custom function a unique name (even if it does work, it is extremely confusing, especially if you're changing the arguments that get passed to it). Write a small plugin to test just the functionality that you're having problems with. This will often make it easier to find the issue. The other benefit is that, if you can't fix it, you can post just this version of the plugin so that we can actually run it.

I understand you, you're right.

The test I did before the changes works fine with cmd_target as well
Since I set up player then like this suddenly doesn't work for me so I think wrong setting of new player or something in CASE

[fysiks]

Originally Posted by Fuck For Fun The test I did before the changes works fine with cmd_target as well Since I set up player then like this suddenly doesn't work for me so I think wrong setting of new player or something in CASE

What??? I think you might need to consider using a translator because your posts are very confusing.

Did you write a small plugin to reproduce the issue so that it's easier to debug as I mentioned before? Post that new plugin so we can help you.

[Fuck For Fun]

Originally Posted by fysiks What??? I think you might need to consider using a translator because your posts are very confusing. Did you write a small plugin to reproduce the issue so that it's easier to debug as I mentioned before? Post that new plugin so we can help you.

What? You should be more specific with your reactions

Like I said, the new I added is for Testin !player and

client_cmd( client, "amx_kick #%d", get_user_userid( player ) );