Fix RecipeManager mixin & add ShulkerBoxScreenHandler Mixin

src/main/java/com/javazilla/bukkitfabric/mixin/recipe/MixinRecipeManager.java

@@ -49,7 +49,7 @@ public class MixinRecipeManager implements IMixinRecipeManager {
      */
     @Overwrite
     @SuppressWarnings({ "unchecked", "rawtypes" })
-    protected void apply(Map<Identifier, JsonElement> map, ResourceManager iresourcemanager, Profiler gameprofilerfiller) {
+    public void apply(Map<Identifier, JsonElement> map, ResourceManager iresourcemanager, Profiler gameprofilerfiller) {
         this.errored = false;
         Map<RecipeType<?>, Map<Identifier, Recipe<?>>> map1 = Maps.newHashMap();
         for (RecipeType<?> recipeType : Registry.RECIPE_TYPE)

src/main/java/com/javazilla/bukkitfabric/mixin/screen/MixinShulkerBoxScreenHandler.java

@@ -0,0 +1,40 @@
+package com.javazilla.bukkitfabric.mixin.screen;
+
+import org.bukkit.craftbukkit.inventory.CraftInventory;
+import org.bukkit.craftbukkit.inventory.CraftInventoryView;
+import org.bukkit.entity.Player;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.Shadow;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.Inject;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
+
+import com.javazilla.bukkitfabric.interfaces.IMixinServerEntityPlayer;
+
+import net.minecraft.entity.player.PlayerInventory;
+import net.minecraft.inventory.Inventory;
+import net.minecraft.screen.ShulkerBoxScreenHandler;
+
+@Mixin(ShulkerBoxScreenHandler.class)
+public class MixinShulkerBoxScreenHandler extends MixinScreenHandler {
+
+    @Shadow
+    public Inventory inventory;
+
+    private CraftInventoryView bukkitEntity;
+    private PlayerInventory player;
+
+    @Inject(method = "<init>*", at = @At("TAIL"))
+    public void setPlayerInv(int i, PlayerInventory playerinventory, Inventory iinventory, CallbackInfo ci) {
+        this.player = (PlayerInventory) playerinventory;
+    }
+
+    @Override
+    public CraftInventoryView getBukkitView() {
+        if (bukkitEntity != null)
+            return bukkitEntity;
+
+        bukkitEntity = new CraftInventoryView((Player)((IMixinServerEntityPlayer)this.player.player).getBukkitEntity(), new CraftInventory(this.inventory), (ShulkerBoxScreenHandler)(Object)this);
+        return bukkitEntity;
+    }
+}

src/main/java/com/javazilla/bukkitfabric/nms/ReflectionMethodVisitor.java

@@ -81,6 +81,11 @@ public void visitMethodInsn(int opcode, String owner, String name, String desc,
             return;
         }
 
+        if (owner.equalsIgnoreCase("java/lang/Class") && name.equalsIgnoreCase("getName") && desc.equalsIgnoreCase("()Ljava/lang/String;")) {
+            super.visitMethodInsn( Opcodes.INVOKESTATIC, "com/javazilla/bukkitfabric/nms/ReflectionRemapper", "getClassName", "(Ljava/lang/Class;)Ljava/lang/String;", false);
+            return;
+        }
+
         if (owner.startsWith("net/minecraft/class_")) {
             if (!name.startsWith("method_"))
                 name = MappingsReader.METHODS2.getOrDefault(name + desc, MappingsReader.getIntermedMethod(owner.replace('/', '.'), name));

src/main/java/com/javazilla/bukkitfabric/nms/ReflectionRemapper.java

@@ -204,6 +204,15 @@ public static String getPackageName(Package pkage) {
         return name;
     }
 
+    /**
+     */
+    public static String getClassName(Class<?> clazz) {
+        String name = clazz.getName();
+        if (name.startsWith("org.bukkit.craftbukkit"))
+            name = name.replace("org.bukkit.craftbukkit", "org.bukkit.craftbukkit." + NMS_VERSION);
+        return name;
+    }
+
     /**
      */
     public static String getMinecraftServerVersion() {

src/main/resources/bukkitfabric.mixins.json

@@ -99,6 +99,7 @@
     "screen.MixinHorseScreenHandler",
     "screen.MixinLecternScreenHandler",
     "screen.MixinMerchantScreenHandler",
+    "screen.ShulkerBoxScreenHandler",
 
     "enchantment.MixinFrostWalkerEnchantment"
   ],