Wireless Crafting Terminal does not properly consume action on use

[shartte]

Tested on Forgecraft, Neoforge 1.20.4

[Mari023]

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

[Mari023]

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