Charging energy items doesn't update mutated stack

[raoulvdberge]

Issue description

Charging Refined Storage items in a Portable Energy Storage does nothing.

The issue is that the item is never charging
So I've been stepping through the code, the item does end up charged but the energy handler at RS side always retrieves a new ItemStack
So I checked the code here: https://github.com/Rearth/Oritech/blob/1.21/src/main/java/rearth/oritech/block/base/entity/ExpandableEnergyStorageBlockEntity.java#L145
And I think the issue is that you never "update" the slot with the mutated stack
The flow should be Create container item context -> Call energy handler to charge -> Update the inventory slot with the mutated stack
The container item context is immutable IIRC so changes never get propagated to the backing inventory

TechReborn is suffering from the same problem: TechReborn/TechReborn#3302

Steps to reproduce

1. Place a Portable Energy Storage
2. Place an empty Portable Grid from Refined Storage into the Portable Energy Storage
3. Notice that it isn't charging

Minecraft version

1.21.1 (Latest)

Fabric loader version

0.16.0

Fabric API version

0.102.0+1.21.1

Oritech version

0.9.1

Other relevant versions

• Refined Storage 2: https://github.com/refinedmods/refinedstorage2
• See original issue here Portable Storage not charging properly as item refinedmods/refinedstorage2#696

If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)

No response

[raoulvdberge]

This was a Refined Storage issue, we were not using the container item context properly.
Fixed in refinedmods/refinedstorage2#741