Merge pull request #54 from CodeMonkeysMods/feat/add-farmers-workbench

Add Most Farmers Workbench Recipies

Author: The-Code-Monkey

src/client/java/com/tcm/MineTale/MineTaleClient.java

@@ -8,7 +8,9 @@
 
 import com.tcm.MineTale.block.workbenches.menu.AbstractWorkbenchContainerMenu;
 import com.tcm.MineTale.block.workbenches.screen.ArmorersWorkbenchScreen;
+import com.tcm.MineTale.block.workbenches.screen.BuildersWorkbenchScreen;
 import com.tcm.MineTale.block.workbenches.screen.CampfireWorkbenchScreen;
+import com.tcm.MineTale.block.workbenches.screen.FarmersWorkbenchScreen;
 import com.tcm.MineTale.registry.ModBlocks;
 import com.tcm.MineTale.registry.ModMenuTypes;
 
@@ -23,19 +25,24 @@
 
 public class MineTaleClient implements ClientModInitializer {
 	/**
-	 * Register client-side screen factories for custom workbench menu types.
+	 * Initialises client-side handlers for the MineTale mod.
 	 *
-	 * Binds ModMenuTypes.FURNACE_WORKBENCH_MENU to FurnaceWorkbenchScreen,
-	 * ModMenuTypes.CAMPFIRE_WORKBENCH_MENU to CampfireWorkbenchScreen, and
-	 * ModMenuTypes.WORKBENCH_WORKBENCH_MENU to WorkbenchWorkbenchScreen so the client
-	 * can create the appropriate GUI when those menus open.
+	 * Registers screen factories for custom workbench menu types, configures render
+	 * layers for furnace workbench blocks, and registers a global network receiver
+	 * that applies nearby inventory items to an open workbench menu.
+	 *
+	 * The network receiver schedules work on the client thread and retries application
+	 * for up to 10 client ticks if the expected workbench menu is not yet open; if
+	 * synchronization still fails it logs a failure message.
 	 */
 	@Override
 	public void onInitializeClient() {
 		MenuScreens.register(ModMenuTypes.FURNACE_WORKBENCH_MENU, FurnaceWorkbenchScreen::new);
 		MenuScreens.register(ModMenuTypes.CAMPFIRE_WORKBENCH_MENU, CampfireWorkbenchScreen::new);
 		MenuScreens.register(ModMenuTypes.WORKBENCH_WORKBENCH_MENU, WorkbenchWorkbenchScreen::new);
 		MenuScreens.register(ModMenuTypes.ARMORERS_WORKBENCH_MENU, ArmorersWorkbenchScreen::new);
+		MenuScreens.register(ModMenuTypes.FARMERS_WORKBENCH_MENU, FarmersWorkbenchScreen::new);
+		MenuScreens.register(ModMenuTypes.BUILDERS_WORKBENCH_MENU, BuildersWorkbenchScreen::new);
 
 		BlockRenderLayerMap.putBlock(ModBlocks.FURNACE_WORKBENCH_BLOCK_T1, ChunkSectionLayer.CUTOUT);
 		BlockRenderLayerMap.putBlock(ModBlocks.FURNACE_WORKBENCH_BLOCK_T2, ChunkSectionLayer.CUTOUT);

src/client/java/com/tcm/MineTale/block/workbenches/screen/ArmorersWorkbenchScreen.java

@@ -88,12 +88,12 @@ private static MineTaleRecipeBookComponent createRecipeBookComponent(ArmorersWor
     }
 
     /**
-         * Configure the screen's GUI dimensions and initialize widgets.
-         *
-         * Sets the layout size (imageWidth = 176, imageHeight = 166), delegates remaining
-         * layout initialization to the superclass, and creates the three craft buttons
-         * ("1", "10", "All") wired to their respective handlers.
-         */
+     * Initialises the screen size and adds the three crafting buttons.
+     *
+     * Sets the GUI image dimensions, delegates further initialisation to the superclass,
+     * and creates/registers three buttons wired to craft one, ten or all items
+     * (they invoke handleCraftRequest with 1, 10 and -1 respectively; -1 signifies "All").
+     */
     @Override
     protected void init() {
         // Important: Set your GUI size before super.init()
@@ -119,45 +119,14 @@ protected void init() {
     }
 
     /**
-     * Sends a crafting request for the currently selected recipe in the integrated recipe book.
+     * Sends a craft request for the recipe remembered by this screen's last known selection.
      *
-     * Locates the last recipe collection and last selected recipe ID from the recipe book component,
-     * resolves the recipe's result item, and sends a CraftRequestPayload to the server containing that
-     * item and the requested amount.
+     * Resolves the remembered recipe to its resulting item(s) and, if available, sends a network
+     * CraftRequestPayload containing the first result and the requested amount. If no remembered
+     * selection or no results are available, no payload is sent.
      *
-     * @param amount the quantity to craft; use -1 to request crafting of the full available stack ("All")
+     * @param amount the quantity to craft; use -1 to request crafting all available units
      */
-
-    // private void handleCraftRequest(int amount) {
-    //     // 1. Cast the book component to the Accessor to get the selected data
-    //     RecipeBookComponentAccessor accessor = (RecipeBookComponentAccessor) this.mineTaleRecipeBook;
-        
-    //     RecipeCollection collection = accessor.getLastRecipeCollection();
-    //     RecipeDisplayId displayId = accessor.getLastRecipe();
-
-    //     if (collection != null && displayId != null) {
-    //         // 2. Find the visual entry
-    //         for (RecipeDisplayEntry entry : collection.getSelectedRecipes(RecipeCollection.CraftableStatus.ANY)) {
-    //             if (entry.id().equals(displayId)) {
-    //                 // 3. Resolve result for the packet
-    //                 List<ItemStack> results = entry.resultItems(SlotDisplayContext.fromLevel(this.minecraft.level));
-                    
-    //                 if (!results.isEmpty()) {
-    //                     ItemStack resultStack = results.get(0);
-                        
-    //                     // 4. LOG FOR DEBUGGING
-    //                     System.out.println("Sending craft request for: " + resultStack + " amount: " + amount);
-                        
-    //                     ClientPlayNetworking.send(new CraftRequestPayload(resultStack, amount));
-    //                 }
-    //                 break;
-    //             }
-    //         }
-    //     } else {
-    //         System.out.println("Request failed: Collection or DisplayID is null!");
-    //     }
-    // }
-
     private void handleCraftRequest(int amount) {
         // Look at our "Memory" instead of the component
         if (this.lastKnownSelectedId != null) {