Skip to content

Conditionally registers Agricraft harvest handler.#2138

Merged
wrincewind merged 1 commit intoWayofTime:1.20.1from
Saereth:agricrash
Dec 18, 2025
Merged

Conditionally registers Agricraft harvest handler.#2138
wrincewind merged 1 commit intoWayofTime:1.20.1from
Saereth:agricrash

Conversation

@Saereth
Copy link
Copy Markdown
Contributor

@Saereth Saereth commented Dec 11, 2025

Registers the Agricraft harvest handler only if the Agricraft mod is loaded.

Prevents a crash when Agricraft is not present.

Registers the Agricraft harvest handler only if the Agricraft mod is loaded.

Prevents a crash when Agricraft is not present.
@wrincewind
Copy link
Copy Markdown
Collaborator

This does fix the problem in worlds where the crash hasn't yet occured, but when i to fix a world that was already bricked by the issue by updating to this version, it continues to crash with the same error (noClassDefFoundError: com/agricraft/agricraft/common/block/entity/CropBlockEntity').

Do you think you can include a fix to un-brick already broken worlds? if not we can update the changelog to include a note about how to solve this, like 'use the config setting removeErroringBlockEntities in serverconfigs/forge-server.toml', but i'd prefer a more elegant solution if possible.

@Saereth
Copy link
Copy Markdown
Contributor Author

Saereth commented Dec 15, 2025

I can include a datafixer to sort this yeah, I'll get it in tomorrow since its getting late here today.

@wrincewind
Copy link
Copy Markdown
Collaborator

as per discussion on discord, it'll be safer to keep this commit as-is and include this note in the changelogs:
"in your world save directory, go to /serverconfig/forge-server.toml' and change 'removeErroringBlockEntities' to 'true', launch your world once, then change it back to false."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants