We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GTNewHorizons/GT-New-Horizons-Modpack#3848 version of Thaumic Energistics (thaumicenergistics-1.1.3.0)
The text was updated successfully, but these errors were encountered:
What I suspect is happening, is the check isItemEqual is not a robust enough check
isItemEqual
Call chain: public void onClientRequestExtract https://github.com/Nividica/ThaumicEnergistics/blob/AE2-RV3/src/main/java/thaumicenergistics/common/container/ContainerPartArcaneCraftingTerminal.java#L1226
public void onClientRequestExtract
switch ( mouseButton ) case ThEGuiHelper.MOUSE_BUTTON_LEFT: if( isShiftHeld ) targetSlot = this.locateMergeSlot( requestStack );
switch ( mouseButton )
case ThEGuiHelper.MOUSE_BUTTON_LEFT:
if( isShiftHeld )
targetSlot = this.locateMergeSlot( requestStack );
public final Slot locateMergeSlot https://github.com/Nividica/ThaumicEnergistics/blob/AE2-RV3/src/main/java/thaumicenergistics/common/container/ContainerWithPlayerInventory.java#L304
public final Slot locateMergeSlot
for( Slot slot : this.playerSlots ) if( slot.getHasStack() ) ItemStack slotStack = slot.getStack(); if( ( slotStack.stackSize < slotStack.getMaxStackSize() ) && slotStack.isItemEqual( stack ) ) return slot;
for( Slot slot : this.playerSlots )
if( slot.getHasStack() )
ItemStack slotStack = slot.getStack();
if( ( slotStack.stackSize < slotStack.getMaxStackSize() ) && slotStack.isItemEqual( stack ) )
return slot;
Sorry, something went wrong.
An additional check should be something along the lines of
if( // Slot has room for more ( slotStack.stackSize < slotStack.getMaxStackSize() ) // Same item && slotStack.isItemEqual( stack ) // Neither has NBT, or NBTs are equal && ( (!stack.hasTagCompound() && !slotStack.hasTagCompound() ) || ItemStack.areItemStackTagsEqual(stack, slotStack) ) ){
There may be additional considerations for items with NBT
No branches or pull requests
GTNewHorizons/GT-New-Horizons-Modpack#3848
version of Thaumic Energistics (thaumicenergistics-1.1.3.0)
The text was updated successfully, but these errors were encountered: