re-order checks in Torches module

block#getType is likely the expensive call here(?)
MLG-Fortress · Mar 12, 2017 · 37aada7 · 37aada7 · erik1988 · Mar 12, 2017
1 parent 4736b7c
commit 37aada7
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/main/java/com/extrahardmode/features/Torches.java b/src/main/java/com/extrahardmode/features/Torches.java
@@ -93,7 +93,7 @@ public void onBlockPlace(BlockPlaceEvent placeEvent)
         final boolean playerBypasses = playerModule.playerBypasses(player, Feature.TORCHES);
 
         // FEATURE: players can't attach torches to common "soft" blocks
-        if (block.getType().equals(Material.TORCH) && limitedTorchPlacement && !playerBypasses)
+        if (limitedTorchPlacement && !playerBypasses && block.getType().equals(Material.TORCH))
         {
             Torch torch = new Torch(Material.TORCH, block.getData());
             Material attachmentMaterial = block.getRelative(torch.getAttachedFace()).getType();