[Bugfix] Fix infinite loop in TranslationTask (#741)

* fix infinite loop and missing return

(cherry picked from commit 13ce56c)

* fix build error (missing semicolon)

(cherry picked from commit 6f0b401)

* add translation

* change loop condition instead

* dont check max y

* Clean up

---------

Co-authored-by: DerToaster98 <38782719+DerToaster98@users.noreply.github.com>

Author: TylerS1066

Movecraft/src/main/java/net/countercraft/movecraft/async/translation/TranslationTask.java

@@ -229,13 +229,15 @@ protected void execute() throws InterruptedException, ExecutionException {
             MovecraftLocation test = new MovecraftLocation(newHitBox.getMidPoint().getX(), newHitBox.getMinY(),
                     newHitBox.getMidPoint().getZ());
             test = test.translate(0, -1, 0);
-            while (test.toBukkit(world).getBlock().getType().isAir()) {
+            // If we are out of bounds, that is technically air, but then we would have an infinite loop, so stay in Y bounds
+            while (test.toBukkit(world).getBlock().getType().isAir() && world.getMinHeight() >= test.getY()) {
                 test = test.translate(0, -1, 0);
             }
             Material testType = test.toBukkit(world).getBlock().getType();
             if (craft.getType().getMaterialSetProperty(CraftType.FORBIDDEN_HOVER_OVER_BLOCKS).contains(testType)) {
                 fail(String.format(I18nSupport.getInternationalisedString("Translation - Failed Craft over block"),
                         testType.name().toLowerCase().replace("_", " ")));
+                return;
             }
         }
         //call event