Commands cannot be used on players with quotes in their name.

[starrypy]

Commands cannot be used on players with quotes in their name.

[rubellyte]

This does appear to be an actual issue; the regex used for joining quoted arguments together doesn't handle escapes. Let me see if I can quickly fix this.

[GermaniumSystem]

Thanks for the quick response! I came to the same conclusion and gave shlex a shot. It seems to work really well, but I'm not familiar enough with the project to know if adding it would have unintended consequences.

Here's a diff:

diff --git a/plugins/command_dispatcher.py b/plugins/command_dispatcher.py
index bc356c3..3629ac3 100644
--- a/plugins/command_dispatcher.py
+++ b/plugins/command_dispatcher.py
@@ -13,9 +13,10 @@ Updated for release: kharidiron
 
 import asyncio
 import packets
+import shlex
 
 from base_plugin import BasePlugin
-from utilities import extractor, get_syntax, send_message
+from utilities import get_syntax, send_message
 from data_parser import ChatSent
 from pparser import build_packet
 
@@ -55,8 +56,8 @@ class CommandDispatcher(BasePlugin):
                 full = build_packet(packets.packets['chat_sent'], pkt)
                 yield from connection.client_raw_write(full)
                 return False
-            to_parse = data['parsed']['message'][len(
-                self.plugin_config.command_prefix):].split()
+            to_parse = shlex.split(data['parsed']['message'][len(
+                self.plugin_config.command_prefix):])
 
             try:
                 command = to_parse[0]
@@ -140,8 +141,7 @@ class CommandDispatcher(BasePlugin):
                 General Exception error as a last-resort catch-all.
         """
         try:
-            yield from self.commands[command](extractor(to_parse),
-                                              connection)
+            yield from self.commands[command](to_parse,connection)
         except SyntaxWarning as e:
             self._send_syntax_error(command, e, connection)
         except NameError as e:
diff --git a/utilities.py b/utilities.py
index c0a13e7..bbda05b 100644
--- a/utilities.py
+++ b/utilities.py
@@ -235,18 +235,6 @@ def read_signed_vlq(reader):
         return -((v >> 1) + 1), d
 
 
-def extractor(*args):
-    """
-    Extracts quoted arguments and puts them as a single argument in the
-    passed iterator.
-    """
-    # It's not elegant, but it's the best way to do it as far as I can tell.
-    # My regex-fu isn't strong though, so if someone can come up with a
-    # better way, great.
-    x = re.split(r"(?:([^\"]\S*)|\"(.+?)\")\s*", " ".join(*args))
-    return [x for x in filter(None, x)]
-
-
 @asyncio.coroutine
 def read_packet(reader, direction):
     """