Fix long-range build tool.

Blocks were always placed around 0,0,0 since the trace direction was being used as a position. Also the message was backwards.
EngineHub · Feb 4, 2019 · dddf2b9 · dddf2b9
1 parent f3ec5bb
commit dddf2b9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/worldedit-core/src/main/java/com/sk89q/worldedit/command/ToolCommands.java b/worldedit-core/src/main/java/com/sk89q/worldedit/command/ToolCommands.java
@@ -205,7 +205,7 @@ public void longrangebuildtool(Player player, LocalSession session, Pattern seco
         if (secondary instanceof BlockPattern) {
             secondaryName = ((BlockPattern) secondary).getBlock().getBlockType().getName();
         }
-        player.print("Left-click set to " + secondaryName + "; right-click set to "
-                + primaryName + ".");
+        player.print("Left-click set to " + primaryName + "; right-click set to "
+                + secondaryName + ".");
     }
 }
diff --git a/worldedit-core/src/main/java/com/sk89q/worldedit/command/tool/LongRangeBuildTool.java b/worldedit-core/src/main/java/com/sk89q/worldedit/command/tool/LongRangeBuildTool.java
@@ -61,7 +61,7 @@ public boolean actSecondary(Platform server, LocalConfiguration config, Player p
             if (applied.getBlockType().getMaterial().isAir()) {
                 eS.setBlock(blockPoint, secondary);
             } else {
-                eS.setBlock(pos.getDirection().toBlockPoint(), secondary);
+                eS.setBlock(pos.toVector().subtract(pos.getDirection()).toBlockPoint(), secondary);
             }
             return true;
         } catch (MaxChangedBlocksException e) {
@@ -82,7 +82,7 @@ public boolean actPrimary(Platform server, LocalConfiguration config, Player pla
             if (applied.getBlockType().getMaterial().isAir()) {
                 eS.setBlock(blockPoint, primary);
             } else {
-                eS.setBlock(pos.getDirection().toBlockPoint(), primary);
+                eS.setBlock(pos.toVector().subtract(pos.getDirection()).toBlockPoint(), primary);
             }
             return true;
         } catch (MaxChangedBlocksException e) {