Fix locking behavior again (this time thru controller)

jaquadro · May 31, 2015 · bdd3759 · bdd3759
1 parent a6d2973
commit bdd3759
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/com/jaquadro/minecraft/storagedrawers/api/storage/IFractionalDrawer.java b/src/com/jaquadro/minecraft/storagedrawers/api/storage/IFractionalDrawer.java
@@ -16,7 +16,7 @@ public interface IFractionalDrawer extends IDrawer
 
     /**
      * Gets the number of items left in the drawer if the maximum number of equivalent compressed items had been removed.
-     * The equivalency is determined by the conversion rate.
+     * The equivalency is determined by the next compression tier, and not necessarily the conversion rate.
      */
     int getStoredItemRemainder ();
 }
diff --git a/src/com/jaquadro/minecraft/storagedrawers/inventory/StorageInventory.java b/src/com/jaquadro/minecraft/storagedrawers/inventory/StorageInventory.java
@@ -6,6 +6,8 @@
 import com.jaquadro.minecraft.storagedrawers.api.storage.IDrawer;
 import com.jaquadro.minecraft.storagedrawers.api.storage.IDrawerGroup;
 import com.jaquadro.minecraft.storagedrawers.api.inventory.IDrawerInventory;
+import com.jaquadro.minecraft.storagedrawers.api.storage.attribute.ILockable;
+import com.jaquadro.minecraft.storagedrawers.api.storage.attribute.LockAttribute;
 import com.jaquadro.minecraft.storagedrawers.storage.IUpgradeProvider;
 import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.item.ItemStack;
@@ -63,6 +65,9 @@ public boolean canInsertItem (int slot, ItemStack stack) {
         if (drawer == null)
             return false;
 
+        if (drawer.isEmpty() && drawer instanceof ILockable && ((ILockable) drawer).isLocked(LockAttribute.LOCK_EMPTY))
+            return false;
+
         return drawer.canItemBeStored(stack);
     }