Disable remap on accessor and fix #130

MattiDragon · Sep 7, 2024 · 2dcdf55 · 2dcdf55
1 parent 1ddb750
commit 2dcdf55
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/changelog/3.0.4+1.21.1.md b/changelog/3.0.4+1.21.1.md
@@ -1,2 +1,3 @@
 * Work around sodium breaking drawer icons by using vanilla rendering
-  * The real issue is still a bug in sodium, but this should prevent it from causing issues
+  * The real issue is still a bug in sodium, but this should prevent it from causing issues
+* Fix crash when interacting with some slots of compacting drawers caused by previous bug (#130)
diff --git a/src/client/java/io/github/mattidragon/extendeddrawers/client/mixin/RenderSystemAccess.java b/src/client/java/io/github/mattidragon/extendeddrawers/client/mixin/RenderSystemAccess.java
@@ -7,7 +7,7 @@
 
 @Mixin(RenderSystem.class)
 public interface RenderSystemAccess {
-    @Accessor
+    @Accessor(remap = false)
     static Vector3f[] getShaderLightDirections() {
         throw new AssertionError();
     }

diff --git a/src/main/java/io/github/mattidragon/extendeddrawers/storage/CompactingDrawerStorage.java b/src/main/java/io/github/mattidragon/extendeddrawers/storage/CompactingDrawerStorage.java
@@ -171,7 +171,7 @@ public int getSlotCount() {
 
     @Override
     public Slot getSlot(int index) {
-        return getActiveSlotArray()[index];
+        return getSlotArray()[index];
     }
 
     @Override