chore: merge follow up change

pom.xml

@@ -281,7 +281,7 @@
         <dependency>
             <groupId>com.github.starwishsama.dough</groupId>
             <artifactId>dough-api</artifactId>
-            <version>20dcab228c</version>
+            <version>f266e39d48</version>
             <scope>compile</scope>
         </dependency>
         <dependency>
@@ -445,8 +445,15 @@
         <dependency>
             <groupId>com.mojang</groupId>
             <artifactId>authlib</artifactId>
-            <version>1.5.25</version>
+            <version>6.0.52</version>
             <scope>provided</scope>
+            <exclusions>
+                <exclusion>
+                    <!-- We don't need any of the dependencies -->
+                    <groupId>*</groupId>
+                    <artifactId>*</artifactId>
+                </exclusion>
+            </exclusions>
         </dependency>
 
         <!-- TODO: Remove this dependency -->

src/main/java/io/github/thebusybiscuit/slimefun4/implementation/listeners/SlimefunItemInteractListener.java

@@ -54,13 +54,6 @@ public void onRightClick(PlayerInteractEvent e) {
                 return;
             }
 
-            // Fixes #4087 - Prevents players from interacting with a block that is about to be deleted
-            // We especially don't want to open inventories as that can cause duplication
-            if (e.getClickedBlock() != null && Slimefun.getTickerTask().isDeletedSoon(e.getClickedBlock().getLocation())) {
-                e.setCancelled(true);
-                return;
-            }
-
             // Fire our custom Event
             PlayerRightClickEvent event = new PlayerRightClickEvent(e);
             Bukkit.getPluginManager().callEvent(event);
@@ -122,7 +115,7 @@ private boolean rightClickBlock(PlayerRightClickEvent event) {
             }
 
             boolean interactable =
-                    sfItem.callItemHandler(BlockUseHandler.class, handler -> handler.onRightClick(event));
+                sfItem.callItemHandler(BlockUseHandler.class, handler -> handler.onRightClick(event));
 
             if (!interactable) {
                 Player p = event.getPlayer();
@@ -152,22 +145,22 @@ private void openInventory(Player p, SlimefunItem item, Block clickedBlock, Play
                     openMenu(blockData.getBlockMenu(), clickedBlock, p);
                 } else {
                     Slimefun.getDatabaseManager()
-                            .getBlockDataController()
-                            .loadBlockDataAsync(blockData, new IAsyncReadCallback<>() {
-                                @Override
-                                public boolean runOnMainThread() {
-                                    return true;
+                        .getBlockDataController()
+                        .loadBlockDataAsync(blockData, new IAsyncReadCallback<>() {
+                            @Override
+                            public boolean runOnMainThread() {
+                                return true;
+                            }
+
+                            @Override
+                            public void onResult(SlimefunBlockData result) {
+                                if (!p.isOnline()) {
+                                    return;
                                 }
 
-                                @Override
-                                public void onResult(SlimefunBlockData result) {
-                                    if (!p.isOnline()) {
-                                        return;
-                                    }
-
-                                    openMenu(result.getBlockMenu(), clickedBlock, p);
-                                }
-                            });
+                                openMenu(result.getBlockMenu(), clickedBlock, p);
+                            }
+                        });
                 }
             }
         } catch (Exception | LinkageError x) {
@@ -184,4 +177,4 @@ private void openMenu(BlockMenu menu, Block b, Player p) {
             }
         }
     }
-}
+}

src/main/java/io/github/thebusybiscuit/slimefun4/implementation/setup/PostSetup.java

@@ -1,5 +1,6 @@
 package io.github.thebusybiscuit.slimefun4.implementation.setup;
 
+import io.github.thebusybiscuit.slimefun4.api.events.SlimefunItemRegistryFinalizedEvent;
 import io.github.thebusybiscuit.slimefun4.api.items.SlimefunItem;
 import io.github.thebusybiscuit.slimefun4.implementation.Slimefun;
 import io.github.thebusybiscuit.slimefun4.implementation.SlimefunItems;

src/main/java/io/github/thebusybiscuit/slimefun4/storage/backend/legacy/LegacyStorage.java

@@ -51,9 +51,9 @@ public PlayerData loadPlayerData(@Nonnull UUID uuid) {
                     items.put(i, playerFile.getItem("backpacks." + key + ".contents." + i));
                 }
 
-                PlayerBackpack backpack = PlayerBackpack.load(uuid, id, size, items);
+                //PlayerBackpack backpack = PlayerBackpack.load(uuid, id, size, items);
 
-                backpacks.put(id, backpack);
+                // backpacks.put(id, backpack);
             } catch (Exception x) {
                 Slimefun.logger().log(Level.WARNING, x, () -> "Could not load Backpack \"" + key + "\" for Player \"" + uuid + '"');
             }