Move floatAt logic to AbstractPlayer, add isAllowedToFly and setFlying.

worldedit-bukkit/src/main/java/com/sk89q/worldedit/bukkit/BukkitPlayer.java

@@ -185,14 +185,13 @@ public class BukkitPlayer extends AbstractPlayerActor {
     }
 
     @Override
-    public void floatAt(int x, int y, int z, boolean alwaysGlass) {
+    public boolean isAllowedToFly() {
-        if (alwaysGlass || !player.getAllowFlight()) {
+        return player.getAllowFlight();
-            super.floatAt(x, y, z, alwaysGlass);
+    }
-            return;
-        }
 
-        setPosition(Vector3.at(x + 0.5, y, z + 0.5));
+    @Override
-        player.setFlying(true);
+    public void setFlying(boolean flying) {
+        player.setFlying(flying);
     }
 
     @Override

worldedit-core/src/main/java/com/sk89q/worldedit/entity/Player.java

@@ -187,6 +187,24 @@ public interface Player extends Entity, Actor {
      */
     void floatAt(int x, int y, int z, boolean alwaysGlass);
 
+    /**
+     * Check whether the player is allowed to fly.
+     *
+     * @return true if allowed flight
+     */
+    default boolean isAllowedToFly() {
+        return false;
+    }
+
+    /**
+     * Set whether the player is currently flying.
+     *
+     * @param flying true to fly
+     */
+    default void setFlying(boolean flying) {
+        throw new UnsupportedOperationException("setFlying unimplemented but isAllowedToFly was true (or unchecked)");
+    }
+
     /**
      * Get the point of the block that is being stood in.
      *

worldedit-core/src/main/java/com/sk89q/worldedit/extension/platform/AbstractPlayerActor.java

@@ -313,13 +313,17 @@ public abstract class AbstractPlayerActor implements Actor, Player, Cloneable {
 
     @Override
     public void floatAt(int x, int y, int z, boolean alwaysGlass) {
-        BlockVector3 spot = BlockVector3.at(x, y - 1, z);
+        if (alwaysGlass || !isAllowedToFly()) {
-        final World world = (World) getLocation().getExtent();
+            BlockVector3 spot = BlockVector3.at(x, y - 1, z);
-        if (!world.getBlock(spot).getBlockType().getMaterial().isMovementBlocker()) {
+            final World world = getWorld();
-            try (EditSession session = WorldEdit.getInstance().getEditSessionFactory().getEditSession(world, 1, this)) {
+            if (!world.getBlock(spot).getBlockType().getMaterial().isMovementBlocker()) {
-                session.setBlock(spot, BlockTypes.GLASS.getDefaultState());
+                try (EditSession session = WorldEdit.getInstance().getEditSessionFactory().getEditSession(world, 1, this)) {
-            } catch (MaxChangedBlocksException ignored) {
+                    session.setBlock(spot, BlockTypes.GLASS.getDefaultState());
+                } catch (MaxChangedBlocksException ignored) {
+                }
             }
+        } else {
+            setFlying(true);
         }
         setPosition(Vector3.at(x + 0.5, y, z + 0.5));
     }

worldedit-fabric/src/main/java/com/sk89q/worldedit/fabric/FabricPlayer.java

@@ -189,15 +189,14 @@ public class FabricPlayer extends AbstractPlayerActor {
     }
 
     @Override
-    public void floatAt(int x, int y, int z, boolean alwaysGlass) {
+    public boolean isAllowedToFly() {
-        if (alwaysGlass || !player.abilities.allowFlying) {
+        return player.abilities.allowFlying;
-            super.floatAt(x, y, z, alwaysGlass);
+    }
-            return;
-        }
 
-        setPosition(Vector3.at(x + 0.5, y, z + 0.5));
+    @Override
-        if (!player.abilities.flying) {
+    public void setFlying(boolean flying) {
-            player.abilities.flying = true;
+        if (player.abilities.flying != flying) {
+            player.abilities.flying = flying;
             player.sendAbilitiesUpdate();
         }
     }

worldedit-forge/src/main/java/com/sk89q/worldedit/forge/ForgePlayer.java

@@ -190,15 +190,14 @@ public class ForgePlayer extends AbstractPlayerActor {
     }
 
     @Override
-    public void floatAt(int x, int y, int z, boolean alwaysGlass) {
+    public boolean isAllowedToFly() {
-        if (alwaysGlass || !player.abilities.allowFlying) {
+        return player.abilities.allowFlying;
-            super.floatAt(x, y, z, alwaysGlass);
+    }
-            return;
-        }
 
-        setPosition(Vector3.at(x + 0.5, y, z + 0.5));
+    @Override
-        if (!player.abilities.isFlying) {
+    public void setFlying(boolean flying) {
-            player.abilities.isFlying = true;
+        if (player.abilities.isFlying != flying) {
+            player.abilities.isFlying = flying;
             player.sendPlayerAbilities();
         }
     }

worldedit-sponge/src/main/java/com/sk89q/worldedit/sponge/SpongePlayer.java

@@ -204,14 +204,13 @@ public class SpongePlayer extends AbstractPlayerActor {
     }
 
     @Override
-    public void floatAt(int x, int y, int z, boolean alwaysGlass) {
+    public boolean isAllowedToFly() {
-        if (alwaysGlass || !player.get(Keys.CAN_FLY).orElse(false)) {
+        return player.get(Keys.CAN_FLY).orElse(super.isAllowedToFly());
-            super.floatAt(x, y, z, alwaysGlass);
+    }
-            return;
-        }
 
-        setPosition(Vector3.at(x + 0.5, y, z + 0.5));
+    @Override
-        player.offer(Keys.IS_FLYING, true);
+    public void setFlying(boolean flying) {
+        player.offer(Keys.IS_FLYING, flying);
     }
 
     @Override