Hello,

I am looking to create an event handler for whenever the crafting matrix of the vanilla Inventory or Crafting Table menu has a valid craft.  The way that I had thought about doing that would be by checking if the menu's resultSlot is occupied by something.

My idea was to do so through events: call getScreen() from a MouseClickedEvent and check if the screen was InventoryScreen or CraftingScreen.  From there I would try to access the menu, but the menu attribute in AbstractContainerScreen is protected, so that is a no-go for me. 

In short, my questions are:

1. Is there a way of accessing the Menu from an event?
2. Is there a way of telling whether or not there is a valid craft in the crafting matrix?

Your proposed solution won't work - for interest you would implement it like this:

    @Mod.EventBusSubscriber(modid = MODID, bus = Mod.EventBusSubscriber.Bus.FORGE, value = Dist.CLIENT)
    public class DONT_TRY_TO_USE_THIS
    {
        @SubscribeEvent
        public static void DOES_NOT_WORK(ScreenEvent.MouseButtonPressed.Post event)
        {
            if (event.getScreen() instanceof CraftingScreen craftingScreen) {
                CraftingMenu menu = craftingScreen.getMenu();
                LOGGER.info("result: " + menu.getSlot(0).getItem()); // Prints the wrong thing!
            }
        }
    }

I posted this, so you can see how to get the menu.

 

The problem is the result has not been determined when you handle the mouse click.

The game has to send a network packet to the server to ask if there is a valid recipe.

The result of that query does not get updated on the client until *after* the mouse event.

 

There is an ItemCraftedEvent, but that handles items removed from the result slot, which doesn't sound like what you want?

Edited July 21, 20223 yr by warjort

Quote

There is an ItemCraftedEvent, but that handles items removed from the result slot, which doesn't sound like what you want?

Thank you for the example, I really should have seen the obvious getMenu() function.  Yes, ItemCraftedEvent is not what I want.  Ideally this event would fire after the player either clicks one of the recipes in the recipe book (with all of the needed materials for that recipe), or after the player slots in an item into the crafting matrix that results in a valid craft.

Quote

What do you want to do as a result of this?

My intention is to detect when the crafting matrix has been filled with a valid craft, and submit to the Minecraft instance a TutorialToast.  This is part of a student project of mine where I augment the vanilla Tutorial with more instructions on how to play the game.