Wireless Crafting Terminal does not properly consume action on use
shartte opened this issue · 2 comments
it isn't quite as easy as I thought since ae2 does also check the return value of MenuOpener#open(), which is server only...
tho I don't think there is a case where that would return false when checkPreconditions() is true
I did some testing for this in ae2wtlib,
I think this happens because WirelessTerminalItem#use() always returns new InteractionResultHolder<>(InteractionResult.FAIL, is), when it should probably return new InteractionResultHolder<>(InteractionResult.sidedSuccess(level.isClientSide()), is) (which it does on the server)
this should be an easy fix, since checkPreconditions() is only an item check now, so it can be performed on the client side too