Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Game crashes when interacting with Fruit #3

Open
HawkSteak opened this issue May 17, 2023 · 15 comments
Open

Game crashes when interacting with Fruit #3

HawkSteak opened this issue May 17, 2023 · 15 comments
Labels
1.16 1.18 bug Something isn't working

Comments

@HawkSteak
Copy link

Currently running into an issue on our server where interacting with fruit from PHC2 will cause that player to crash. This also extends to interactions with WAWLA/HWYLA, so looking at the fruit will also crash the player. Their crash log is attached. I believe this is related to the following errors that get thrown when the server first boots: server errors.txt

crash-2023-05-06_00.57.37-fml.txt

@HawkSteak
Copy link
Author

Updating this to add that it looks like the names for the fruit are wrong within the code. It's currently looking for entities as pamhc2trees:[fruitname]item (ex: pamhc2trees:almonditem), but looking at the entity in singleplayer identifies it as pamhc2trees:pam[fruitname] (ex: pamhc2trees:pamalmond).

Even after attempting to tweak this though, the server still fails to find the registered blocks.

@AzoSBear
Copy link

I also saw that it fails to find the registry name for the harvest craft fruits in the latestlog file when booting up minecraft
bootup log.txt

@supermassimo
Copy link
Member

I am being unable to reproduce these bugs. Are you sure you are using the latest versions of all mods?
The crash seems to be related to some issues with the config files, not with the fruit.

@supermassimo
Copy link
Member

The crash is a dup of #2 but I am leaving this open as the other registration issue is unrelated

@HawkSteak
Copy link
Author

Just did a fresh server install to test and still receiving the same errors. I removed all mods but the following, which are all up-to-date (version number listed as well for good measure). Configs were not touched aside from changing the world name and setting world generator to Biomes O Plenty.

Forge (36.2.39)
Biomes O Plenty (13.1.0.488)
Dynamic Trees (0.10.5)
Dynamic Trees Plus (0.1.0)
Dynamic Trees BOP (2.0.9)
Dynamic Trees PHC2 (1.0.0)
Pam HC2 Food Core (1.0.2)
Pam HC2 Food Extended (1.0.4)
Pam HC2 Crops (1.0.2)
Pam HC2 Trees (1.0.1)

server.errors.txt

@HawkSteak
Copy link
Author

Also able to replicate it with only Dynamic Trees PHC2 and its dependencies (DT, PHC2, PHC2trees) installed. Client side is fine unless joining the server without loading singleplayer beforehand.

@supermassimo
Copy link
Member

Ohh I see, so to reproduce join a server without joining singleplayer first

@HawkSteak
Copy link
Author

Ohh I see, so to reproduce join a server without joining singleplayer first

Yup! The server throws the errors attached to the last post on boot, but it only impacts/crashes the client if they go to multiplayer first.

@Squidvad3r
Copy link

Squidvad3r commented Jul 5, 2023

A friend of mine is getting the same issue. Whenever she looks at fruit, specifically from PHC2:Dynamic trees, she crashes.
The same happens to me if I join her server.

However, if you load a single-player world and look at a fruit tree (idk if the 'looking' part is needed, knock on wood), then join a server, you won't crash from looking at a random pear/apricot/gooseberry.

So, yeah, launch single-player, look at some fruit, then quit and you can play on multiplayer. It's a 'Janky fix' in her words, but idk what she's complaining about.

Hope this helps :)

@supermassimo
Copy link
Member

Yeah this is unfortunately a side effect of how PHC2 registers it's items in a very non-standard way. We are still looking into a fix from our side

@Cryptillian
Copy link

Just wanted to bump this that my friend is having this issue on our LAN server as well when he looks directly at fruit on a tree (specifically he was looking at gooseberries).

@Likeabozz123
Copy link

same issue still

@Foulest
Copy link

Foulest commented May 29, 2024

Fixed in my fork of DynamicTrees for 1.16.5.

@supermassimo
Copy link
Member

@Foulest what was the change needed to fix this? Tried to look for it in your fork's commits but they are too large to efficiently find all the changes

@Foulest
Copy link

Foulest commented May 30, 2024

@Foulest what was the change needed to fix this? Tried to look for it in your fork's commits but they are too large to efficiently find all the changes

https://github.com/Foulest/DynamicTrees/blob/8a16d73c49b4c7a5256623f98d60f2a539c3c3af/src/main/java/com/ferreusveritas/dynamictrees/systems/fruit/Fruit.java#L179

@supermassimo supermassimo added bug Something isn't working 1.16 1.18 1.19 and removed 1.19 labels May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.16 1.18 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants