cs2-retakes-allocator icon indicating copy to clipboard operation
cs2-retakes-allocator copied to clipboard

Command System Remake

Open GL1TCH1337 opened this issue 1 year ago • 5 comments

I have removed player chat event based command system and integrated it to CS# command system. By doing this players can change prefix of command by CS# settings and no need to add different prefixes to this plugin config.

This also completely solves issue #128

Since this is a big change. Server owners should configure their config by new default config.

Waiting your feedbacks.

GL1TCH1337 avatar Apr 24 '24 09:04 GL1TCH1337

I need some time to think about this. Im not sure this is really necessary, maybe I'm not understanding though

yonilerner avatar Apr 24 '24 17:04 yonilerner

This PR is more "standard" but it gives less flexibility to the commands themselves, so Im hesitant to change it. Does CS# support case insensitive commands maybe?

yonilerner avatar Apr 25 '24 12:04 yonilerner

CS# command system commands work case insensitive. It will work anyways. !GuNs or !GUNS dont matter.

(another plugin of me but uses cs# command system)

image image image

GL1TCH1337 avatar Apr 26 '24 12:04 GL1TCH1337

CS# command system commands work case insensitive. It will work anyways. !GuNs or !GUNS dont matter.

(another plugin of me but uses cs# command system)

image image image

you removing "gunmenu" ".gunmenu" some people want it without !

oqyh avatar Apr 26 '24 13:04 oqyh

if you want to solve https://github.com/yonilerner/cs2-retakes-allocator/issues/128 you can just change

if (CenterMenuCommands.Any(cmd => cmd.Equals(message, StringComparison.OrdinalIgnoreCase))) to if (CenterMenuCommands.Any(cmd => cmd.Equals(message.TrimStart('/'), StringComparison.OrdinalIgnoreCase)))

oqyh avatar Apr 26 '24 13:04 oqyh