From 875412353574a47c01ff1a938737111e3564094e Mon Sep 17 00:00:00 2001
From: Luck <git@lucko.me>
Date: Sat, 30 May 2020 02:39:37 +0100
Subject: [PATCH] Fix parsing second optional argument in editor command

---
 .../common/commands/misc/EditorCommand.java    | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/common/src/main/java/me/lucko/luckperms/common/commands/misc/EditorCommand.java b/common/src/main/java/me/lucko/luckperms/common/commands/misc/EditorCommand.java
index 5b52e0836..48edc1a7e 100644
--- a/common/src/main/java/me/lucko/luckperms/common/commands/misc/EditorCommand.java
+++ b/common/src/main/java/me/lucko/luckperms/common/commands/misc/EditorCommand.java
@@ -69,19 +69,23 @@ public class EditorCommand extends SingleCommand {
     @Override
     public CommandResult execute(LuckPermsPlugin plugin, Sender sender, List<String> args, String label) {
         Type type = Type.ALL;
+        String filter = null;
 
-        // parse type
-        String typeString = ArgumentParser.parseStringOrElse(0, args, null);
-        if (typeString != null) {
+        // attempt to parse type
+        String arg0 = ArgumentParser.parseStringOrElse(0, args, null);
+        if (arg0 != null) {
             try {
-                type = Type.valueOf(typeString.toUpperCase());
+                type = Type.valueOf(arg0.toUpperCase());
             } catch (IllegalArgumentException e) {
-                // ignored
+                // assume they meant it as a filter
+                filter = arg0;
+            }
+
+            if (filter == null) {
+                filter = ArgumentParser.parseStringOrElse(1, args, null);
             }
         }
 
-        String filter = ArgumentParser.parseStringOrElse(1, args, null);
-
         // run a sync task
         plugin.getSyncTaskBuffer().requestDirectly();