fixed older version support

Author: hyperdefined

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>lol.hyper</groupId>
     <artifactId>lecterncrashfix</artifactId>
-    <version>1.0</version>
+    <version>1.1</version>
     <packaging>jar</packaging>
 
     <name>LecternCrashFix</name>

src/main/java/lol/hyper/lecterncrashfix/LecternCrashFix.java

@@ -6,6 +6,7 @@
 import com.comphenix.protocol.events.PacketAdapter;
 import com.comphenix.protocol.events.PacketEvent;
 import lol.hyper.lecterncrashfix.wrapper.WrapperPlayClientWindowClick;
+import org.bukkit.Bukkit;
 import org.bukkit.entity.Player;
 import org.bukkit.event.inventory.InventoryType;
 import org.bukkit.inventory.InventoryView;
@@ -19,14 +20,18 @@ public final class LecternCrashFix extends JavaPlugin {
 
     @Override
     public void onEnable() {
+        String bukkitPackageName = Bukkit.getServer().getClass().getPackage().getName();
+        String bukkitVersion = bukkitPackageName.substring(bukkitPackageName.lastIndexOf(".") + 1);
+        int ver = Integer.parseInt(bukkitVersion.split("_")[1]);
+
         ProtocolLibrary.getProtocolManager().addPacketListener(new PacketAdapter(this, ListenerPriority.HIGHEST, PacketType.Play.Client.WINDOW_CLICK) {
             @Override
             public void onPacketReceiving(PacketEvent event) {
                 if (event.getPlayer() == null) {
                     return;
                 }
 
-                WrapperPlayClientWindowClick packet = new WrapperPlayClientWindowClick(event.getPacket());
+                WrapperPlayClientWindowClick packet = new WrapperPlayClientWindowClick(event.getPacket(), ver);
                 Player player = event.getPlayer();
                 InventoryView inv = player.getOpenInventory();
                 if (inv.getType() == InventoryType.LECTERN) {

src/main/java/lol/hyper/lecterncrashfix/wrapper/WrapperPlayClientWindowClick.java

@@ -25,14 +25,17 @@
 
 public class WrapperPlayClientWindowClick extends AbstractPacket {
     public static final PacketType TYPE = PacketType.Play.Client.WINDOW_CLICK;
+    private final int VERSION;
 
-    public WrapperPlayClientWindowClick() {
+    public WrapperPlayClientWindowClick(int version) {
         super(new PacketContainer(TYPE), TYPE);
+        VERSION = version;
         handle.getModifier().writeDefaults();
     }
 
-    public WrapperPlayClientWindowClick(PacketContainer packet) {
+    public WrapperPlayClientWindowClick(PacketContainer packet, int version) {
         super(packet, TYPE);
+        VERSION = version;
     }
 
     /**
@@ -135,7 +138,15 @@ public void setClickedItem(ItemStack value) {
     }
 
     public InventoryClickType getShift() {
-        return handle.getEnumModifier(InventoryClickType.class, 4).read(0);
+        // index changes here because versions use different ones idk
+        int index = 0;
+        if (VERSION == 16) {
+            index = 5;
+        }
+        if (VERSION >= 17) {
+            index = 4;
+        }
+        return handle.getEnumModifier(InventoryClickType.class, index).read(0);
     }
 
     public void setShift(InventoryClickType value) {

src/main/resources/plugin.yml

@@ -1,5 +1,5 @@
 name: LecternCrashFix
 version: '${project.version}'
 main: lol.hyper.lecterncrashfix.LecternCrashFix
-api-version: 1.18
+api-version: 1.16
 depend: [ProtocolLib]