QuiltServerTools / HeyThatsMine

MIT License
12 stars 13 forks source link

Hopper flag is not respected when Lithium is installed #15

Closed MojiRS closed 1 year ago

MojiRS commented 3 years ago

Lithium optimizes hopper behavior, and whatever changes it makes causes hoppers to still be able to insert/remove items from containers flagged with '/htm flag HOPPERS false'.

The issue happens consistently for all containers with only Hey That's Mine and Lithium installed.

Let me know if I should open an issue with Lithium instead - I figured it would be easier to add compatibility from your side, but I could be completely wrong.

Versions: Hey That's Mine 1.1.3 Lithium 0.7.3

PotatoPresident commented 3 years ago

Talked to Caffine, we're working on it

Carlosjrlu commented 2 years ago

May I ask, any progress with this issue?

cpm9 commented 2 years ago

Yeah this is game breaking. Had some players come through and rob our entire server this way, was a nightmare to sort through the consolidated items afterwards...

PotatoPresident commented 2 years ago

Current workaround is to add mixin.block.hopper=false to lithium.properties config file. Will add this to mod description.

youstube-1 commented 2 years ago

There's no config for mixin.block.hopper=false? or at least this didn't work for me.

arnokeesman commented 2 years ago

just add a line with mixin.block.hopper=false to /config/lithium.properties

youstube-1 commented 2 years ago

Looking at the lithium config page (https://github.com/CaffeineMC/lithium-fabric/wiki/Configuration-File#block), I don't see a config for that. Is it just not in there for some reason?

ps. It didn't work for me.

tylrae commented 2 years ago

This mod is useless if chests aren't actually protected.

tylrae commented 2 years ago

Removal of Lithium does not fix the issue. I've tried it on both the hopper flag when set to either true or false.