Skip to content

A **Mostly** working port for 1.21.5 neoforge#832

Open
StoffiPoff wants to merge 41 commits intoVazkiiMods:1.21.xfrom
StoffiPoff:patchouli-1.21.5-neoforge-refactor
Open

A **Mostly** working port for 1.21.5 neoforge#832
StoffiPoff wants to merge 41 commits intoVazkiiMods:1.21.xfrom
StoffiPoff:patchouli-1.21.5-neoforge-refactor

Conversation

@StoffiPoff
Copy link
Copy Markdown

Needs further review and work and testing but mostly works on 1.21.5 now.

StoffiPoff and others added 26 commits October 28, 2025 18:50
- Updates the Just Enough Items (JEI) dependency to the correct version (`21.4.0.27`) for Minecraft 1.21.5 and NeoForge, resolving the previous build failure.
- Removes the deprecated `registerButton` method from the `IComponentRenderContext` interface and its implementation in `GuiBookEntry`. This addresses the deprecation warnings and aligns the code with the current API.
…precation' into refactor-patchouli-1.21.5-jei-deprecation
This commit refactors the book item model to use a custom `ItemModel` implementation. This allows for dynamic model selection based on the book's data in the `ItemStack`.

- A new `BookModel` class has been created to handle the custom rendering logic.
- The `NeoForgeClientInitializer` has been updated to register the new `BookModel` as a custom model type and to register the "completion" item property.
- The book's item model JSON has been updated to use the new custom model loader.

This change fixes the issue with the `modelRegistry` in the NeoForge client initialization and provides a more robust and flexible way to handle book item models.
…as used.

The client-side resource reload listener was being incorrectly registered, causing the book data to not be loaded. This was due to using an outdated API for the project's version of NeoForge.

After extensive research and multiple failed build attempts, it was discovered that the manual registration of the reload listener is no longer necessary. The code has been updated to remove the unnecessary listener, and the project now builds successfully.
…ted, no texture is loaded however.. continuing troubleshooting
…custom-item-model

# Conflicts:
#	NeoForge/src/main/java/vazkii/patchouli/neoforge/client/NeoForgeClientInitializer.java
#	Xplat/src/main/java/vazkii/patchouli/common/item/ItemModBook.java
…custom-item-model

# Conflicts:
#	NeoForge/src/main/java/vazkii/patchouli/neoforge/client/NeoForgeClientInitializer.java
#	Xplat/src/main/java/vazkii/patchouli/common/item/ItemModBook.java
…custom-item-model

# Conflicts:
#	NeoForge/src/main/java/vazkii/patchouli/neoforge/client/NeoForgeClientInitializer.java
…custom-item-model

# Conflicts:
#	NeoForge/src/main/java/vazkii/patchouli/neoforge/client/NeoForgeClientInitializer.java
#	Xplat/src/main/java/vazkii/patchouli/common/item/ItemModBook.java
Corrects an issue where ingredients for shaped crafting recipes were not being displayed in the crafting page. This was caused by an incorrect `instanceof` check that prevented the ingredient processing logic from running. The check has been updated to use the correct `ShapedCraftingRecipeDisplay` type, and the method for retrieving ingredients has been corrected to `ingredients()`.
…cond result item is shown beneath the original
Recipes render fine in the gui, missing some parts for different crafting stations though.
Trying to implement pick with middle mouse button for the rendered visual but it's tricky to get right, not fully implemented yet.
Almost all parts of the original 1.21.1 mod works now for neoforge 1.21.5 but I am still struggling with implementing the book model generation.
Recipes render fine in the gui, missing some parts for different crafting stations though.
Trying to implement pick with middle mouse button for the rendered visual but it's tricky to get right, not fully implemented yet.
Almost all parts of the original 1.21.1 mod works now for neoforge 1.21.5 but I am still struggling with implementing the book model generation.
@StoffiPoff StoffiPoff marked this pull request as draft November 7, 2025 05:32
@StoffiPoff StoffiPoff marked this pull request as ready for review November 7, 2025 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant