fix #elytra reset bypassing y checks

Author: babbaj

src/main/java/baritone/command/defaults/ElytraCommand.java

@@ -82,7 +82,11 @@ public void execute(String label, IArgConsumer args) throws CommandException {
         final String action = args.getString();
         switch (action) {
             case "reset": {
-                elytra.resetState();
+                try {
+                    elytra.resetState();
+                } catch (IllegalArgumentException ex) {
+                    throw new CommandInvalidStateException(ex.getMessage());
+                }
                 logDirect("Reset state but still flying to same goal");
                 break;
             }

src/main/java/baritone/process/ElytraProcess.java

@@ -331,6 +331,17 @@ public BlockPos currentDestination() {
 
     @Override
     public void pathTo(BlockPos destination) {
+        int minY = ctx.world().dimensionType().minY();
+        int maxY = (ctx.world().dimension() == Level.NETHER && !Baritone.settings().elytraAllowAboveRoof.value) ? 127 : Math.min(minY + 384, ctx.world().dimensionType().height() + minY);
+        if (destination.getY() < minY || destination.getY() >= maxY) {
+            throw new IllegalArgumentException("The goal must have a y value between " + minY + " and " + maxY);
+        }
+
+        int playerY = (int)ctx.player().getY();
+        if (playerY < minY || playerY >= maxY) {
+            throw new IllegalArgumentException("The player must have a y value between " + minY + " and " + maxY);
+        }
+
         this.pathTo0(destination, false);
     }
 
@@ -368,17 +379,6 @@ public void pathTo(Goal iGoal) {
             throw new IllegalArgumentException("The goal must be a GoalXZ or GoalBlock");
         }
 
-        int minY = ctx.world().dimensionType().minY();
-        int maxY = (ctx.world().dimension() == Level.NETHER && !Baritone.settings().elytraAllowAboveRoof.value) ? 127 : Math.min(minY + 384, ctx.world().dimensionType().height() + minY);
-        if (y < minY || y >= maxY) {
-            throw new IllegalArgumentException("The goal must have a y value between " + minY + " and " + maxY);
-        }
-
-        int playerY = (int)ctx.player().getY();
-        if (playerY < minY || playerY >= maxY) {
-            throw new IllegalArgumentException("The player must have a y value between " + minY + " and " + maxY);
-        }
-
         this.pathTo(new BlockPos(x, y, z));
     }