Source.Python forums

i've found

http://www.sourcepython.com/showthread.php?478-Help-with-Client-Command-Filter-to-force-players-to-join-specified-team&highlight=Client+Command+Filter

but all the wiki posted are not available anymore

here is what i have right now



hello does not show up ~

tested game: CSGO

You are trying to compare the Command object with a string. This will always result in False.
http://wiki.sourcepython.com/index.php/commands#Command
http://wiki.sourcepython.com/index.php/commands.client

You should use the ClientCommandFilter decorator.

Ayuto wrote:You are trying to compare the Command object with a string. This will always result in False.
http://wiki.sourcepython.com/index.php/commands#Command
http://wiki.sourcepython.com/index.php/commands.client

Syntax: Select all

def client_command_filter(player, command):
    if command[0] == 'jointeam':
        print('hello')

thanks got it!

L'In20Cible wrote:You should use the ClientCommandFilter decorator.

whats the difference if i may ask

i tried both way and both works

Yes, the result is the same, but if you use the decorator the code is shorter (50%) and less error prone as you don't need to unregister the filter when you unload the plugin. This will be done automatically.

Just compare these two scripts and you will understand.
As you can see the second script is much easier to use.

I use something like that:

Ayuto wrote:Yes, the result is the same, but if you use the decorator the code is shorter (50%) and less error prone as you don't need to unregister the filter when you unload the plugin. This will be done automatically.

Just compare these two scripts and you will understand.

Syntax: Select all

from commands.client import client_command_manager

def load():
    client_command_manager.register_filter(client_command_filter)
 
def unload(): 
    client_command_manager.unregister_filter(client_command_filter)
	
def client_command_filter(player, command):
    pass

Syntax: Select all

from commands.client import ClientCommandFilter

@ClientCommandFilter
def client_command_filter(player, command):
    pass

As you can see the second script is much easier to use.

Yep code is shorter i see :)

BackRaw wrote:I use something like that:

Syntax: Select all

from commands import CommandReturn
from commands.client import ClientCommandFilter

# 'team_manager' is a dictionary containing teamid values for "t" and "ct", identical to:
team_manager = {
    "t": 2,
    "ct": 3
}


@ClientCommandFilter
def jointeam_filter(playerinfo, command):
    """
    Gets called whenever a client command is executed.
    """

    # was 'jointeam' the command?
    if command[0] != "jointeam":

        # if not, allow the action
        return CommandReturn.CONTINUE

    # get the player count of the T team
    count_t = len(tuple(playeriter_t))

    # get the selected TeamID
    teamid = int(command[1])

    # grab the player's index
    index = index_from_playerinfo(playerinfo)

    # are there players on T?
    if count_t < 1 and teamid != team_manager["t"]:

        # if not, tell the player that they must join T
        tell(index, "There are no players on the Terrorist side, please join there.")

        # and don't allow the previous action
        return CommandReturn.BLOCK

    # if yes ...
    if count_t > 0:

        # ... did we chose the T team?
        if teamid == team_manager["t"]:

            # if yes, tell a message
            tell(index, "There are players on the Terrorist side, but none on the Counter-Terrorist side.")
            tell(index, "Please join there.")

            # and don't allow the previous action
            return CommandReturn.BLOCK

        # if not, is this player currently on the T team and wants to switch to the CT team?
        if playerinfo.get_team_index() == team_manager["t"]:

            tell(index, "Players are auto-switched by the script.")

            return CommandReturn.BLOCK

    # if not, allow the action
    return CommandReturn.CONTINUE


# 'tell' is a wrapper function to make telling players a message easy
# like es.tell() if you're familiar with EventScripts.

Hi why return CommandReturn.CONTINUE and CommandReturn.BLOCK?? I just use return True and return False
any difference?

btw is there an equivalent of

for SP?

-------------------------- edit ----------------------

nvm got it

Technically, you should be using CommandReturn to block or allow the command. However, True/False work just the same. You can also return None (which also works by not returning any value) and it will work as True/CommandReturn.CONTINUE. If I remember correctly, only returning a False value or CommandReturn.BLOCK will block the command.