Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd_targetex #368

Closed
wants to merge 4 commits into from
Closed

cmd_targetex #368

wants to merge 4 commits into from

Conversation

luxxxoor
Copy link
Contributor

@luxxxoor luxxxoor commented Apr 4, 2016

This is a feature stolen from SM, and it will introduce multi-targets commands.

There is a list of all new targeting types : @ALL : target all players from server.
@alive : target all alive player from server.
@aim : target the player that the admin is currently looking at.
@Bots : target only bots.
@dead : target only dead players.
@humans : target only human players.
@me : target the admin that write the command.
@!me : target everyone in the server except the admin that use the command.
@ct : target anyone from CT team.
@t : target anyone from TERRORIST team.
@SPEC : target anyone from SPECTATOR team.
@Spectating : target the player that the admin is currently spectating.

AMXX is getting very old and we still don't have a multi targetting option... so there is my suggestion.

luxxxoor added 4 commits April 4, 2016 15:55
This is a feature stolen from SM, and it will introduce multi-targets commands.

Ex : amx_slay @ALL - will slay all the clients.

There is a list of all new targeting types : @ALL        : target all players from server. 
@alive      : target all alive player from server.
@aim        : target the player that the admin is currently looking at.
@Bots       : target only bots.
@dead       : target only dead players. 
@humans     : target only human players.
@me         : target the admin that write the command. 
@!me        : target everyone in the server except the admin that use the command. 
@ct         : target anyone from CT team.
@t          : target anyone from TERRORIST team.
@SPEC       : target anyone from SPECTATOR team.
@Spectating : target the player that the admin is currently spectating.

AMXX is getting very old and we still don't have a multi targetting option... so there is my suggestion.
@pavelthq
Copy link

pavelthq commented Apr 4, 2016

You have some commented //server_print("cmd_targetex :

@luxxxoor
Copy link
Contributor Author

luxxxoor commented Apr 5, 2016

i know that there are some bugs.. Artick(HamletEagle) startet the stock.. i tried to finish it.. but my time is not my friend... so here was our "project" we don't have time to finish it.. so ark please take it just as suggestion

@Javivi
Copy link
Contributor

Javivi commented Apr 6, 2016

@luxxxoor What are the bugs and what is missing. If you don't say anything it is harder for someone to take over and finish this.

@luxxxoor
Copy link
Contributor Author

at the moment it has only one known bug: if you'll use a command (ex: amx_slay) @ALL and cmd_targetex has CMDTARGET_NO_SPEC flag, it will work as CMDTARGET_ONLY_ALIVE, because i cannot find a proper way to implement that feature

@Javivi
Copy link
Contributor

Javivi commented Apr 29, 2016

And what would be the correct behavior ? Targeting both alive and dead players, but not spectators ?

@luxxxoor
Copy link
Contributor Author

yes

i think the name of flags are sufficiently suggestive, or they are not ?

@Javivi
Copy link
Contributor

Javivi commented Apr 30, 2016

But in certain cases, like with amx_slay, it would not make any sense targeting dead players.

What's sourcemod behaviour ?

@luxxxoor
Copy link
Contributor Author

luxxxoor commented May 1, 2016

it should not be designed only for base commands from admincmd

@Arkshine
Copy link
Member

Closed in favor of #515.

@Arkshine Arkshine closed this Aug 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants