TorchHelper.placeTorch() uses item in main hand

[vhd]

This can be easily reproduced with TR omni-tool. If you right click on block it will be voided. That's because ItemUsageContext constructor set's stack to player.getStackInHand(hand). Current workaround is to make custom class like this

public class ItemUsageContextCustomStack extends ItemUsageContext {

    public ItemUsageContextCustomStack(World world, @Nullable PlayerEntity player, Hand hand, ItemStack stack, BlockHitResult hit) {
        super(world, player, hand, stack, hit);
    }

}

and use it like that (inside TorchHelper.placeTorch())

ItemUsageContext context = new ItemUsageContextCustomStack(itemUsageContext.getWorld(),player,itemUsageContext.getHand(),torchStack,new BlockHitResult(itemUsageContext.getHitPos(), itemUsageContext.getSide(), itemUsageContext.getBlockPos(), true));

My current dirty fix for TR omnitool Omni-tool_fix.patch.txt