Address deprecation warnings

Author: me4502

src/main/java/com/sk89q/craftbook/mechanics/Ammeter.java

@@ -117,6 +117,6 @@ else if (data > 0)
     public void loadConfiguration (YAMLProcessor config, String path) {
 
         config.setComment(path + "item", "Set the item that is the ammeter tool.");
-        item = BukkitAdapter.asItemType(ItemSyntax.getItem(config.getString(path + "item", ItemTypes.COAL.getId())).getType());
+        item = BukkitAdapter.asItemType(ItemSyntax.getItem(config.getString(path + "item", ItemTypes.COAL.id())).getType());
     }
 }

src/main/java/com/sk89q/craftbook/mechanics/BetterPistons.java

@@ -453,12 +453,12 @@ public static boolean isEnabled(Types type) {
 
     public static List<String> getDefaultBlacklist() {
         return Lists.newArrayList(
-                BlockTypes.OBSIDIAN.getId(),
-                BlockTypes.BEDROCK.getId(),
-                BlockTypes.NETHER_PORTAL.getId(),
-                BlockTypes.END_PORTAL.getId(),
-                BlockTypes.END_PORTAL_FRAME.getId(),
-                BlockTypes.END_GATEWAY.getId()
+                BlockTypes.OBSIDIAN.id(),
+                BlockTypes.BEDROCK.id(),
+                BlockTypes.NETHER_PORTAL.id(),
+                BlockTypes.END_PORTAL.id(),
+                BlockTypes.END_PORTAL_FRAME.id(),
+                BlockTypes.END_GATEWAY.id()
         );
     }
 

src/main/java/com/sk89q/craftbook/mechanics/BounceBlocks.java

@@ -39,7 +39,7 @@ public class BounceBlocks extends AbstractCraftBookMechanic {
     public void loadConfiguration (YAMLProcessor config, String path) {
 
         config.setComment(path + "blocks", "A list of blocks that can be jumped on.");
-        blocks = BlockSyntax.getBlocks(config.getStringList(path + "blocks", Collections.singletonList(BlockTypes.DIAMOND_BLOCK.getId())), true);
+        blocks = BlockSyntax.getBlocks(config.getStringList(path + "blocks", Collections.singletonList(BlockTypes.DIAMOND_BLOCK.id())), true);
 
         config.setComment(path + "sensitivity", "The sensitivity of jumping.");
         sensitivity = config.getDouble(path + "sensitivity", 0.1);

src/main/java/com/sk89q/craftbook/mechanics/GlowStone.java

@@ -73,6 +73,6 @@ public void onBlockBreak(BlockBreakEvent event) {
     public void loadConfiguration (YAMLProcessor config, String path) {
 
         config.setComment(path + "glowstone-off-block", "Sets the block that the redstone glowstone mechanic turns into when turned off.");
-        offBlock = BlockSyntax.getBlock(config.getString(path + "glowstone-off-block", BlockTypes.GLASS.getId()), true);
+        offBlock = BlockSyntax.getBlock(config.getString(path + "glowstone-off-block", BlockTypes.GLASS.id()), true);
     }
 }

src/main/java/com/sk89q/craftbook/mechanics/LightStone.java

@@ -87,6 +87,6 @@ private static String getLightLine(int data) {
     public void loadConfiguration (YAMLProcessor config, String path) {
 
         config.setComment(path + "item", "The item that the lightstone mechanic uses.");
-        item = BukkitAdapter.asItemType(ItemSyntax.getItem(config.getString(path + "item", ItemTypes.GLOWSTONE_DUST.getId())).getType());
+        item = BukkitAdapter.asItemType(ItemSyntax.getItem(config.getString(path + "item", ItemTypes.GLOWSTONE_DUST.id())).getType());
     }
 }

src/main/java/com/sk89q/craftbook/mechanics/Snow.java

@@ -563,14 +563,14 @@ public int hashCode() {
 
     private static List<String> getDefaultReplacables() {
         return Lists.newArrayList(
-                BlockTypes.DEAD_BUSH.getId(),
-                BlockTypes.SHORT_GRASS.getId(),
-                BlockTypes.DANDELION.getId(),
-                BlockTypes.POPPY.getId(),
-                BlockTypes.BROWN_MUSHROOM.getId(),
-                BlockTypes.RED_MUSHROOM.getId(),
-                BlockTypes.FIRE.getId(),
-                BlockTypes.FERN.getId());
+                BlockTypes.DEAD_BUSH.id(),
+                BlockTypes.SHORT_GRASS.id(),
+                BlockTypes.DANDELION.id(),
+                BlockTypes.POPPY.id(),
+                BlockTypes.BROWN_MUSHROOM.id(),
+                BlockTypes.RED_MUSHROOM.id(),
+                BlockTypes.FIRE.id(),
+                BlockTypes.FERN.id());
     }
 
     @Override

src/main/java/com/sk89q/craftbook/mechanics/TreeLopper.java

@@ -178,11 +178,11 @@ else if (data instanceof Tree)
     public void loadConfiguration (YAMLProcessor config, String path) {
 
         config.setComment(path + "block-list", "A list of log blocks. This can be modified to include more logs. (for mod support etc)");
-        enabledBlocks = BlockSyntax.getBlocks(config.getStringList(path + "block-list", BlockCategories.LOGS.getAll().stream().map(BlockType::getId).sorted(String::compareToIgnoreCase).collect(Collectors.toList())), true);
+        enabledBlocks = BlockSyntax.getBlocks(config.getStringList(path + "block-list", BlockCategories.LOGS.getAll().stream().map(BlockType::id).sorted(String::compareToIgnoreCase).collect(Collectors.toList())), true);
 
         config.setComment(path + "tool-list", "A list of tools that can trigger the TreeLopper mechanic.");
-        enabledItems = config.getStringList(path + "tool-list", Arrays.asList(ItemTypes.IRON_AXE.getId(), ItemTypes.WOODEN_AXE.getId(),
-                ItemTypes.STONE_AXE.getId(), ItemTypes.DIAMOND_AXE.getId(), ItemTypes.GOLDEN_AXE.getId()))
+        enabledItems = config.getStringList(path + "tool-list", Arrays.asList(ItemTypes.IRON_AXE.id(), ItemTypes.WOODEN_AXE.id(),
+                ItemTypes.STONE_AXE.id(), ItemTypes.DIAMOND_AXE.id(), ItemTypes.GOLDEN_AXE.id()))
                 .stream().map(ItemSyntax::getItem).map(ItemStack::getType).map(BukkitAdapter::asItemType).collect(Collectors.toList());
 
         config.setComment(path + "max-size", "The maximum amount of blocks the TreeLopper can break.");

src/main/java/com/sk89q/craftbook/mechanics/XPStorer.java

@@ -297,7 +297,7 @@ public void loadConfiguration(YAMLProcessor config, String path) {
         xpPerBottle = config.getInt(path + "xp-per-bottle", 16);
 
         config.setComment(path + "block", "The block that is an XP Storer.");
-        block = BlockSyntax.getBlock(config.getString(path + "block", BlockTypes.SPAWNER.getId()), true);
+        block = BlockSyntax.getBlock(config.getString(path + "block", BlockTypes.SPAWNER.id()), true);
 
         config.setComment(path + "require-sneaking-state", "Sets how the player must be sneaking in order to use the XP Storer.");
         sneakingState = TernaryState.getFromString(config.getString(path + "require-sneaking-state", "no"));

src/main/java/com/sk89q/craftbook/mechanics/area/AreaCommands.java

@@ -90,7 +90,7 @@ public void saveArea(CommandContext context, CommandSender sender) throws Comman
             BlockVector3 size = max.subtract(min).add(1, 1, 1);
 
             // Check maximum size
-            if (Area.instance.maxAreaSize != -1 && size.getBlockX() * size.getBlockY() * size.getBlockZ()
+            if (Area.instance.maxAreaSize != -1 && size.x() * size.y() * size.z()
                     > Area.instance.maxAreaSize) {
                 throw new CommandException("Area is larger than allowed " + Area.instance.maxAreaSize + " blocks.");
             }

src/main/java/com/sk89q/craftbook/mechanics/area/simple/Bridge.java

@@ -317,10 +317,10 @@ public boolean isApplicableSign(String line) {
 
     public List<String> getDefaultBlocks() {
         List<String> materials = new ArrayList<>();
-        materials.add(BlockTypes.COBBLESTONE.getId());
-        materials.add(BlockTypes.GLASS.getId());
-        materials.addAll(BlockCategories.PLANKS.getAll().stream().map(BlockType::getId).collect(Collectors.toList()));
-        materials.addAll(BlockCategories.SLABS.getAll().stream().map(BlockType::getId).collect(Collectors.toList()));
+        materials.add(BlockTypes.COBBLESTONE.id());
+        materials.add(BlockTypes.GLASS.id());
+        materials.addAll(BlockCategories.PLANKS.getAll().stream().map(BlockType::id).toList());
+        materials.addAll(BlockCategories.SLABS.getAll().stream().map(BlockType::id).toList());
         return materials;
     }