Integrated Terminals

Integrated Terminals

20M Downloads

Issue closed

GuivinALT opened this issue · 8 comments

commented

Issue closed

commented

Thanks for reporting!

commented

If this problem only occurs in combination with other mods, attempt to find the smallest set of mods that causes the problem.
--> No :D

What does that mean?
That it doesn't occur with other mods?
Or that you didn't test it?

commented

I have reproduced this issue with the following mods/versions:

  • Minecraft: 1.18.2
  • Forge: 40.1.74
  • Common Capabilities version 1.18.2-2.8.2
  • Cyclops Core version 1.18.2-1.16.0
  • Integrated Terminals version 1.18.2-1.3.1
  • Integrated Crafting version 1.18.2-1.0.25
  • Integrated Dynamics version 1.18.2-1.11.12
  • Integrated Tunnels version 1.18.2-1.8.15

This issue is not specific to NTC2, nor is it specific to 1.16.5.

commented

Thanks for checking!

commented

I have also separately confirmed that this is specific to the Portable Storage Terminal; the static one that is placed on a cable is not afflicted by this duplication bug. Perhaps the Portable Storage Terminal has an internal inventory that isn't being cleared?

commented

Well, this about confirms that theory:
image
Making use of an Entity Reader and watching the NBT of the Portable Storage Terminal (namely integratedterminals:namedInventories/minecraft:itemstack_crafting/Items) in the first slot of my inventory, I spotted that the internal inventory is not being cleared after shift-clicking that button.

Without the ability to step through with a debugger, I can only offer that the issue lies somewhere within TerminalStorageIngredientItemStackCraftingGridClear.clearGrid. Though I can just now theorize that perhaps the crafting inventory does not have a flushing update at the end of the clearing session... clearing to storage still writes back an empty slot on a successful slot emptying but no such "dirty-marking write" occurs in the event of a clear to the player's inventory. 🤔

commented

Is this issue closed? It appears to still exist as of Integrated Terminals 1.18.2-1.3.2.

( for reference, this issue was a duplication bug stemming from shift-clicking the "clear crafting grid" button in the Portable Storage Terminal; it does not actually empty out the grid's inventory on the server when clicking the button - it clears client-side, but closing and re-opening the terminal shows a tangible copy of what was previously there )

commented

@sctjkc01 Could you create a new issue for it?