diff --git a/src/main/java/appeng/parts/automation/PartImportBus.java b/src/main/java/appeng/parts/automation/PartImportBus.java index 47e42e3ac..a56b5851f 100644 --- a/src/main/java/appeng/parts/automation/PartImportBus.java +++ b/src/main/java/appeng/parts/automation/PartImportBus.java @@ -218,6 +218,11 @@ public class PartImportBus extends PartSharedItemBus implements IInventoryDestin private boolean importStuff( final InventoryAdaptor myAdaptor, final IAEItemStack whatToImport, final IMEMonitor inv, final IEnergySource energy, final FuzzyMode fzMode ) { final int toSend = this.calculateMaximumAmountToImport( myAdaptor, whatToImport, inv, fzMode ); + + if( toSend == 0 ){ + return false; + } + final ItemStack newItems; if( this.getInstalledUpgrades( Upgrades.FUZZY ) > 0 ) @@ -288,6 +293,10 @@ public class PartImportBus extends PartSharedItemBus implements IInventoryDestin if( itemAmountNotStorable != null ) { + if (simResult.getCount() == itemAmountNotStorable.getStackSize()) + { + return 0; + } return (int) Math.min( simResult.getCount() - itemAmountNotStorable.getStackSize(), toSend ); } diff --git a/src/main/java/appeng/parts/misc/ItemHandlerAdapter.java b/src/main/java/appeng/parts/misc/ItemHandlerAdapter.java index 52236c957..1eec21d7f 100644 --- a/src/main/java/appeng/parts/misc/ItemHandlerAdapter.java +++ b/src/main/java/appeng/parts/misc/ItemHandlerAdapter.java @@ -19,12 +19,7 @@ package appeng.parts.misc; -import java.util.ArrayList; -import java.util.Arrays; -import java.util.HashMap; -import java.util.Iterator; -import java.util.List; -import java.util.Map; +import java.util.*; import appeng.api.config.Settings; import appeng.api.config.StorageFilter; @@ -42,11 +37,11 @@ import appeng.api.storage.channels.IItemStorageChannel; import appeng.api.storage.data.IAEItemStack; import appeng.api.storage.data.IItemList; import appeng.core.AELog; -import appeng.me.GridAccessException; import appeng.me.helpers.IGridProxyable; import appeng.me.storage.ITickingMonitor; import appeng.util.Platform; import appeng.util.item.AEItemStack; +import org.apache.commons.lang3.tuple.Pair; /** @@ -81,11 +76,19 @@ class ItemHandlerAdapter implements IMEInventory, IBaseMonitor> injectedList = new ArrayList<>(); // This uses a brute force approach and tries to jam it in every slot the inventory exposes. for( int i = 0; i < slotCount && !remaining.isEmpty(); i++ ) { + int countPre = remaining.getCount(); remaining = this.itemHandler.insertItem( i, remaining, simulate ); + int countPos = remaining.getCount(); + if (remaining.isEmpty()){ + injectedList.add( Pair.of( i,countPre ) ); + } + else if (countPos > countPre) { + injectedList.add( Pair.of( i,countPos - countPre ) ); + } } // At this point, we still have some items left... @@ -97,13 +100,17 @@ class ItemHandlerAdapter implements IMEInventory, IBaseMonitor pair : injectedList ) { - this.proxyable.getProxy().getTick().alertDevice( this.proxyable.getProxy().getNode() ); - } - catch( GridAccessException ex ) - { - // meh + if( this.cache.cachedAeStacks[pair.getKey()] == null ) + { + this.cache.cachedAeStacks[pair.getKey()] = iox.copy().setStackSize( pair.getValue() ); + } + else + { + this.cache.cachedAeStacks[pair.getKey()].incStackSize( pair.getValue() ); + } } } @@ -121,8 +128,10 @@ class ItemHandlerAdapter implements IMEInventory, IBaseMonitor> extractedList = new ArrayList<>(); - for( int i = 0; i < this.itemHandler.getSlots(); i++ ) + + for ( int i = 0; i < this.itemHandler.getSlots(); i++ ) { ItemStack stackInInventorySlot = this.itemHandler.getStackInSlot( i ); @@ -140,6 +149,7 @@ class ItemHandlerAdapter implements IMEInventory, IBaseMonitor remainingCurrentSlot ) @@ -151,6 +161,8 @@ class ItemHandlerAdapter implements IMEInventory, IBaseMonitor, IBaseMonitor 0 ); + while ( !extracted.isEmpty() && remainingCurrentSlot > 0 ); remainingSize -= stackSizeCurrentSlot - remainingCurrentSlot; @@ -180,13 +192,14 @@ class ItemHandlerAdapter implements IMEInventory, IBaseMonitor pair : extractedList ) { - this.proxyable.getProxy().getTick().alertDevice( this.proxyable.getProxy().getNode() ); - } - catch( GridAccessException ex ) - { - // meh + this.cache.cachedAeStacks[pair.getKey()].decStackSize( pair.getValue() ); + if( this.cache.cachedAeStacks[pair.getKey()].getStackSize() == 0 ) + { + this.cache.cachedAeStacks[pair.getKey()] = null; + } } }