Update to 1.20 (#2420)

build.gradle

@@ -34,8 +34,8 @@ repositories {
 
 dependencies {
     implementation 'net.bytebuddy:byte-buddy:1.14.3'
-    compileOnly 'org.spigotmc:spigot-api:1.19.4-R0.1-SNAPSHOT'
-    compileOnly 'org.spigotmc:spigot:1.19.4-R0.1-SNAPSHOT'
+    compileOnly 'org.spigotmc:spigot-api:1.20-R0.1-SNAPSHOT'
+    compileOnly 'org.spigotmc:spigot:1.20-R0.1-SNAPSHOT'
     compileOnly 'io.netty:netty-all:4.0.23.Final'
     compileOnly 'net.kyori:adventure-text-serializer-gson:4.13.0'
     compileOnly 'com.googlecode.json-simple:json-simple:1.1.1'
@@ -46,7 +46,7 @@ dependencies {
     testImplementation 'org.mockito:mockito-inline:4.11.0'
     testImplementation 'io.netty:netty-common:4.1.77.Final'
     testImplementation 'io.netty:netty-transport:4.1.77.Final'
-    testImplementation 'org.spigotmc:spigot:1.19.4-R0.1-SNAPSHOT'
+    testImplementation 'org.spigotmc:spigot:1.20-R0.1-SNAPSHOT'
     testImplementation 'net.kyori:adventure-text-serializer-gson:4.13.0'
     testImplementation 'net.kyori:adventure-text-serializer-plain:4.13.1'
 }

src/main/java/com/comphenix/protocol/utility/MinecraftVersion.java

@@ -37,6 +37,11 @@
  */
 public final class MinecraftVersion implements Comparable<MinecraftVersion>, Serializable {
 
+    /**
+     * Version 1.20 - the trails and tails update
+     */
+    public static final MinecraftVersion TRAILS_AND_TAILS = new MinecraftVersion("1.20");
+
     /**
      * Version 1.19.4 - the rest of the feature preview
      */
@@ -126,7 +131,7 @@ public final class MinecraftVersion implements Comparable<MinecraftVersion>, Ser
     /**
      * The latest release version of minecraft.
      */
-    public static final MinecraftVersion LATEST = FEATURE_PREVIEW_2;
+    public static final MinecraftVersion LATEST = TRAILS_AND_TAILS;
 
     // used when serializing
     private static final long serialVersionUID = -8695133558996459770L;

src/main/java/com/comphenix/protocol/wrappers/WrappedLevelChunkData.java

@@ -7,6 +7,7 @@
 import com.comphenix.protocol.reflect.accessors.FieldAccessor;
 import com.comphenix.protocol.reflect.fuzzy.FuzzyFieldContract;
 import com.comphenix.protocol.utility.MinecraftReflection;
+import com.comphenix.protocol.utility.MinecraftVersion;
 import com.comphenix.protocol.utility.ZeroBuffer;
 import com.comphenix.protocol.wrappers.nbt.NbtCompound;
 import com.comphenix.protocol.wrappers.nbt.NbtFactory;
@@ -163,11 +164,18 @@ public static class LightData extends AbstractWrapper {
                     MinecraftReflection.getPacketDataSerializerClass(), int.class, int.class);
             BIT_SET_ACCESSORS = Accessors.getFieldAccessorArray(HANDLE_TYPE, BitSet.class, true);
             BYTE_ARRAY_LIST_ACCESSORS = Accessors.getFieldAccessorArray(HANDLE_TYPE, List.class, true);
-            TRUST_EDGES_ACCESSOR = Accessors.getFieldAccessor(reflection.getField(FuzzyFieldContract.newBuilder()
-                    .typeExact(boolean.class)
-                    .build()));
+
+            if(MinecraftVersion.TRAILS_AND_TAILS.atOrAbove()) {
+                TRUST_EDGES_ACCESSOR = null;
+            } else {
+                TRUST_EDGES_ACCESSOR = Accessors.getFieldAccessor(reflection.getField(FuzzyFieldContract.newBuilder()
+                        .typeExact(boolean.class)
+                        .build()));
+            }
         }
 
+        private boolean dummyTrustEdges = false;
+
         public LightData(Object handle) {
             super(HANDLE_TYPE);
 
@@ -268,27 +276,65 @@ public List<byte[]> getBlockUpdates() {
 
         /**
          * Whether edges can be trusted for light updates or not.
-         *
+         * @deprecated Removed in 1.20
          * @return {@code true} if edges can be trusted, {@code false} otherwise.
          */
+        @Deprecated
         public boolean isTrustEdges() {
+            if(MinecraftVersion.TRAILS_AND_TAILS.atOrAbove()) {
+                return dummyTrustEdges; // ensure backwards compatability and prevent inconsistent states
+            }
             return (boolean) TRUST_EDGES_ACCESSOR.get(handle);
         }
 
         /**
          * Sets whether edges can be trusted for light updates or not.
-         *
+         * @deprecated Removed in 1.20
          * @param trustEdges the new value
          */
+        @Deprecated
         public void setTrustEdges(boolean trustEdges) {
+            if(MinecraftVersion.TRAILS_AND_TAILS.atOrAbove()) {
+                dummyTrustEdges = trustEdges; // ensure backwards compatability and prevent inconsistent states
+                return;
+            }
             TRUST_EDGES_ACCESSOR.set(handle, trustEdges);
         }
 
+        /**
+         * Constructs new LightData from values
+         * @param trustEdges Ignored
+         * @param skyYMask skyYMask
+         * @param blockYMask blockYMask
+         * @param emptySkyYMask emptySkyYMask
+         * @param emptyBlockYMask emptyBlockYMask
+         * @param skyUpdates skyUpdates
+         * @param blockUpdates blockUpdates
+         * @deprecated Parameter trustEdges was removed in 1.20
+         * @return new light data
+         */
+        @Deprecated
         public static LightData fromValues(BitSet skyYMask, BitSet blockYMask, BitSet emptySkyYMask, BitSet emptyBlockYMask,
                                            List<byte[]> skyUpdates, List<byte[]> blockUpdates, boolean trustEdges) {
+            LightData lightData = fromValues(skyYMask, blockYMask, emptySkyYMask, emptyBlockYMask, skyUpdates, blockUpdates);
+            lightData.setTrustEdges(trustEdges);
+            return lightData;
+        }
+
+        /**
+         * Constructs new LightData from values
+         * @param skyYMask skyYMask
+         * @param blockYMask blockYMask
+         * @param emptySkyYMask emptySkyYMask
+         * @param emptyBlockYMask emptyBlockYMask
+         * @param skyUpdates skyUpdates
+         * @param blockUpdates blockUpdates
+         * @return new light data
+         */
+        public static LightData fromValues(BitSet skyYMask, BitSet blockYMask, BitSet emptySkyYMask, BitSet emptyBlockYMask,
+                                           List<byte[]> skyUpdates, List<byte[]> blockUpdates) {
             LightData data = new LightData(LIGHT_UPDATE_PACKET_DATA_CONSTRUCTOR.invoke(MinecraftReflection.getPacketDataSerializer(new ZeroBuffer()), 0, 0));
 
-            data.setTrustEdges(trustEdges);
             data.setSkyYMask(skyYMask);
             data.setBlockYMask(blockYMask);
             data.setEmptySkyYMask(emptySkyYMask);

src/test/java/com/comphenix/protocol/BukkitInitialization.java

@@ -14,10 +14,10 @@
 import org.bukkit.Bukkit;
 import org.bukkit.Server;
 import org.bukkit.World;
-import org.bukkit.craftbukkit.v1_19_R3.CraftServer;
-import org.bukkit.craftbukkit.v1_19_R3.CraftWorld;
-import org.bukkit.craftbukkit.v1_19_R3.inventory.CraftItemFactory;
-import org.bukkit.craftbukkit.v1_19_R3.util.Versioning;
+import org.bukkit.craftbukkit.v1_20_R1.CraftServer;
+import org.bukkit.craftbukkit.v1_20_R1.CraftWorld;
+import org.bukkit.craftbukkit.v1_20_R1.inventory.CraftItemFactory;
+import org.bukkit.craftbukkit.v1_20_R1.util.Versioning;
 import org.spigotmc.SpigotWorldConfig;
 
 import static org.mockito.Mockito.mock;

src/test/java/com/comphenix/protocol/MinecraftVersionTest.java

@@ -18,16 +18,19 @@
 package com.comphenix.protocol;
 
 import com.comphenix.protocol.utility.MinecraftVersion;
+import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertFalse;
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.*;
 
 class MinecraftVersionTest {
+    @BeforeAll
+    public static void beforeAll() {
+        BukkitInitialization.initializeAll();
+    }
 
     @Test
-    void testComparision() {
+    void testComparison() {
         MinecraftVersion within = new MinecraftVersion(1, 2, 5);
         MinecraftVersion outside = new MinecraftVersion(1, 7, 0);
 
@@ -41,12 +44,12 @@ void testComparision() {
         assertFalse(outside.compareTo(within) < 0 && outside.compareTo(highest) < 0);
         assertTrue(atLeast.isAtLeast(MinecraftVersion.BOUNTIFUL_UPDATE));
     }
-    
-    /* @Test
-    public void testSnapshotVersion() {
-        MinecraftVersion version = MinecraftVersion.fromServerVersion("git-Spigot-1119 (MC: 13w39b)");
-        assertEquals(version.getSnapshot(), new SnapshotVersion("13w39b"));
-    } */
+
+
+    @Test
+    void testCurrent() {
+        assertEquals(MinecraftVersion.TRAILS_AND_TAILS, MinecraftVersion.getCurrentVersion());
+    }
 
     @Test
     void testParsing() {

src/test/java/com/comphenix/protocol/PacketTypeTest.java

@@ -14,26 +14,12 @@
  */
 package com.comphenix.protocol;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertFalse;
-import static org.junit.jupiter.api.Assertions.assertTrue;
-
 import com.comphenix.protocol.PacketType.Protocol;
 import com.comphenix.protocol.PacketType.Sender;
 import com.comphenix.protocol.events.PacketContainer;
 import com.comphenix.protocol.injector.packet.PacketRegistry;
 import com.comphenix.protocol.utility.MinecraftReflection;
 import com.comphenix.protocol.utility.MinecraftReflectionTestUtil;
-import java.lang.reflect.Field;
-import java.util.ArrayList;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Map;
-import java.util.Map.Entry;
-import java.util.Set;
-import java.util.TreeMap;
-
-import com.comphenix.protocol.wrappers.BukkitConverters;
 import com.comphenix.protocol.wrappers.WrappedChatComponent;
 import net.minecraft.network.EnumProtocol;
 import net.minecraft.network.protocol.EnumProtocolDirection;
@@ -43,6 +29,12 @@
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
+import java.lang.reflect.Field;
+import java.util.*;
+import java.util.Map.Entry;
+
+import static org.junit.jupiter.api.Assertions.*;
+
 /**
  * @author dmulloy2
  */

src/test/java/com/comphenix/protocol/events/PacketContainerTest.java

@@ -443,7 +443,7 @@ public void testAttributeList() {
         // are inner classes (which is ultimately pointless because AttributeSnapshots don't access any
         // members of the packet itself)
         PacketPlayOutUpdateAttributes packet = (PacketPlayOutUpdateAttributes) attribute.getHandle();
-        AttributeBase base = BuiltInRegistries.u.a(MinecraftKey.a("generic.max_health"));
+        AttributeBase base = BuiltInRegistries.v.a(MinecraftKey.a("generic.max_health"));
         AttributeSnapshot snapshot = new AttributeSnapshot(base, 20.0D, modifiers);
         attribute.getSpecificModifier(List.class).write(0, Lists.newArrayList(snapshot));
 
@@ -838,7 +838,7 @@ public void testCloning() {
                                     0,
                                     Registry.getItemStackSerializer(false),
                                     BukkitConverters.getItemStackConverter().getGeneric(new ItemStack(Material.WOODEN_AXE))),
-                            new WrappedDataValue(0, Registry.get(CatVariant.class), BuiltInRegistries.ai.e(CatVariant.e)),
+                            new WrappedDataValue(0, Registry.get(CatVariant.class), BuiltInRegistries.aj.e(CatVariant.e)),
                             new WrappedDataValue(0, Registry.get(FrogVariant.class), FrogVariant.a)
                     ));
                 } else if (type == PacketType.Play.Server.CHAT) {

src/test/java/com/comphenix/protocol/injector/EntityUtilitiesTest.java

@@ -12,8 +12,8 @@
 import net.minecraft.server.level.PlayerChunkMap.EntityTracker;
 import net.minecraft.server.level.WorldServer;
 import net.minecraft.world.entity.Entity;
-import org.bukkit.craftbukkit.v1_19_R3.CraftWorld;
-import org.bukkit.craftbukkit.v1_19_R3.entity.CraftEntity;
+import org.bukkit.craftbukkit.v1_20_R1.CraftWorld;
+import org.bukkit.craftbukkit.v1_20_R1.entity.CraftEntity;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 

src/test/java/com/comphenix/protocol/utility/MinecraftReflectionTest.java

@@ -13,7 +13,7 @@
 import net.minecraft.world.level.block.state.IBlockData;
 import org.bukkit.Material;
 import org.bukkit.block.Block;
-import org.bukkit.craftbukkit.v1_19_R3.inventory.CraftItemStack;
+import org.bukkit.craftbukkit.v1_20_R1.inventory.CraftItemStack;
 import org.bukkit.entity.Entity;
 import org.bukkit.inventory.ItemStack;
 import org.junit.jupiter.api.AfterAll;

src/test/java/com/comphenix/protocol/utility/MinecraftReflectionTestUtil.java

@@ -2,8 +2,8 @@
 
 public class MinecraftReflectionTestUtil {
 
-    public static final String RELEASE_TARGET = "1.19.3";
-    public static final String PACKAGE_VERSION = "v1_19_R3";
+    public static final String RELEASE_TARGET = "1.20";
+    public static final String PACKAGE_VERSION = "v1_20_R1";
     public static final String NMS = "net.minecraft";
     public static final String OBC = "org.bukkit.craftbukkit." + PACKAGE_VERSION;
 

src/test/java/com/comphenix/protocol/wrappers/WrappedAttributeTest.java

@@ -93,7 +93,7 @@ private AttributeSnapshot getAttributeCopy(WrappedAttribute attribute) {
             modifiers.add((AttributeModifier) wrapper.getHandle());
         }
 
-        AttributeBase base = BuiltInRegistries.u.a(MinecraftKey.a(attribute.getAttributeKey()));
+        AttributeBase base = BuiltInRegistries.v.a(MinecraftKey.a(attribute.getAttributeKey()));
         return new AttributeSnapshot(base, attribute.getBaseValue(), modifiers);
     }
 

src/test/java/com/comphenix/protocol/wrappers/WrappedBlockDataTest.java

@@ -19,9 +19,9 @@
 import org.bukkit.Material;
 import org.bukkit.block.BlockFace;
 import org.bukkit.block.data.type.GlassPane;
-import org.bukkit.craftbukkit.v1_19_R3.block.data.CraftBlockData;
-import org.bukkit.craftbukkit.v1_19_R3.block.impl.CraftStainedGlassPane;
-import org.bukkit.craftbukkit.v1_19_R3.util.CraftMagicNumbers;
+import org.bukkit.craftbukkit.v1_20_R1.block.data.CraftBlockData;
+import org.bukkit.craftbukkit.v1_20_R1.block.impl.CraftStainedGlassPane;
+import org.bukkit.craftbukkit.v1_20_R1.util.CraftMagicNumbers;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
@@ -56,7 +56,7 @@ public void testMaterialCreation() {
 
     @Test
     public void testDataCreation() {
-        IBlockData nmsData = CraftMagicNumbers.getBlock(Material.CYAN_STAINED_GLASS_PANE).o();
+        IBlockData nmsData = CraftMagicNumbers.getBlock(Material.CYAN_STAINED_GLASS_PANE).n();
         GlassPane data = (GlassPane) CraftBlockData.fromData(nmsData);
         data.setFace(BlockFace.EAST, true);
 

src/test/java/com/comphenix/protocol/wrappers/WrappedDataWatcherTest.java

@@ -14,23 +14,19 @@
  */
 package com.comphenix.protocol.wrappers;
 
-import java.util.UUID;
-
 import com.comphenix.protocol.BukkitInitialization;
 import com.comphenix.protocol.wrappers.WrappedDataWatcher.Registry;
 import com.comphenix.protocol.wrappers.WrappedDataWatcher.Serializer;
 import com.comphenix.protocol.wrappers.WrappedDataWatcher.WrappedDataWatcherObject;
 import net.minecraft.world.entity.projectile.EntityEgg;
-import org.bukkit.craftbukkit.v1_19_R3.entity.CraftEgg;
-import org.bukkit.craftbukkit.v1_19_R3.entity.CraftEntity;
+import org.bukkit.craftbukkit.v1_20_R1.entity.CraftEgg;
+import org.bukkit.craftbukkit.v1_20_R1.entity.CraftEntity;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertFalse;
-import static org.junit.jupiter.api.Assertions.assertNotNull;
-import static org.junit.jupiter.api.Assertions.assertNotSame;
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import java.util.UUID;
+
+import static org.junit.jupiter.api.Assertions.*;
 
 /**
  * @author dmulloy2