SEServerExtender / EssentialsPlugin

Dedicated Essentials Plugin
GNU Lesser General Public License v3.0
18 stars 18 forks source link

New commands parser implications. #15

Closed LeShred closed 9 years ago

LeShred commented 9 years ago

Removed the calls to General.SplitString() in order to have a consistent and reliable commands parsing.

I tested it with most commands that can use composed names (player or ship names) and also with timed and triggered cleanups. Everything seems to be working fine!

On a side note: I hesitated to modify the ChatHandlerBase.HandleCommand() virtual method in order to take List instead of string[] but as the commands and parameters should not be modified it's perhaps not worth the modifications.