Crash when hovering over (disabled?) Geolosys ingot in Engineer's Manual #135

Closed
opened 2018-06-21 10:09:29 +00:00 by import_user_oitsjustjose_95mQ · 6 comments
import_user_oitsjustjose_95mQ commented 2018-06-21 10:09:29 +00:00 (Migrated from git.oitsjustjose.com)

Versions:

  • Minecraft Forge: 14.23.4.2705
  • Geolosys: 1.9.4
  • ImmersiveEngineering: 1.1.9

What happens:

When looking at the Engineer's Manual, it displays disabled Geolosys ingots as one of the possible ingredients. Hovering over them immediately crashed the game.

Screenshot

What should happen:

Ideally, the ingots shouldn't even appear in the manual in the first place.

Logs:

Crash log: https://paste.ee/p/5D6SC

Additional comments:

Ingots don't appear in JEI. Have they perhaps been added to the ore dictionary even though they were not registered? Maybe it's actually Immersive Engineering's fault.

## Versions: - **Minecraft Forge:** 14.23.4.2705 - **Geolosys:** 1.9.4 - **ImmersiveEngineering:** 1.1.9 ## What happens: When looking at the *Engineer's Manual*, it displays disabled *Geolosys* ingots as one of the possible ingredients. Hovering over them immediately crashed the game. ![Screenshot](/uploads/e8777291274006305fe3c2dcaefbbe2c/41711037-6a39581c-7547-11e8-9271-062eddb8f1c0.png) ## What should happen: Ideally, the ingots shouldn't even appear in the manual in the first place. ## Logs: **Crash log:** https://paste.ee/p/5D6SC ## Additional comments: Ingots don't appear in *JEI*. Have they perhaps been added to the ore dictionary even though they were not registered? Maybe it's actually *Immersive Engineering*'s fault.
import_user_oitsjustjose_95mQ commented 2018-06-21 12:10:41 +00:00 (Migrated from git.oitsjustjose.com)

Hmmmm. Do you have another mod that does oreDictionary unification by chance?

Otherwise, it's definitely 100% IE's fault, but I have no idea how.... I don't even initialize the item unless the config is enabled. If it's enabled in the config, then it's initialized. And the oreDictionary registration is done in the constructor of the Ingot object, so it's literally impossible for it to be added unless it was initialized - OR, unless you didn't always have them disabled and IE does some weird persistent caching.

Hmmmm. Do you have another mod that does oreDictionary unification by chance? Otherwise, it's definitely 100% IE's fault, but I have no idea how.... I don't even initialize the item unless the config is enabled. If it's enabled in the config, then it's initialized. And the oreDictionary registration is done in the constructor of the Ingot object, so it's literally impossible for it to be added unless it was initialized - OR, unless you didn't *always* have them disabled and IE does some weird persistent caching.
import_user_oitsjustjose_95mQ commented 2018-06-21 12:36:32 +00:00 (Migrated from git.oitsjustjose.com)

I had UniDict in the pack before, and did have them enabled as well, but I strongly believe I completely cleaned my instance afterwards and I can't find any place where this could be cached.

I had UniDict in the pack before, and did have them enabled as well, but I strongly believe I completely cleaned my instance afterwards and I can't find any place where this could be cached.
import_user_oitsjustjose_95mQ commented 2018-06-21 12:37:55 +00:00 (Migrated from git.oitsjustjose.com)

This has just become super mysterious. I'm gonna look into it.

This has just become super mysterious. I'm gonna look into it.
import_user_oitsjustjose_95mQ commented 2018-06-21 12:48:18 +00:00 (Migrated from git.oitsjustjose.com)

So in my dev environment (which has an admittedly slightly-outdated IE install) this doesn't occur; I started with Geolosys ingots, saw them in the engineer's manual, and then removed them. Once removed, they no longer appear in engineer's manual.

I'm curious what other mod is breaking things here... but it might not be that. I'm updating my IE install now to see if it's a newly implemented bug.

EDIT: Wait, no i'm not, I forgot that my dev environment is set to get the latest version when I setup in Gradle, so I actually am up-to-date.

So in my dev environment (which has an admittedly slightly-outdated IE install) this doesn't occur; I started *with* Geolosys ingots, saw them in the engineer's manual, and then removed them. Once removed, they no longer appear in engineer's manual. I'm curious what other mod is breaking things here... but it might not be that. I'm updating my IE install now to see if it's a *newly implemented* bug. EDIT: Wait, no i'm not, I forgot that my dev environment is set to get the latest version when I setup in Gradle, so I actually am up-to-date.
import_user_oitsjustjose_95mQ commented 2018-06-21 20:27:27 +00:00 (Migrated from git.oitsjustjose.com)

I just found a possible cause for the bug: The server and client don't have the same config. So on the server, the ingots were disabled, and on the client, they weren't. Not sure if this is something you want to "fix" or not so I'll leave the issue open.

I just found a possible cause for the bug: The server and client don't have the same config. So on the server, the ingots were disabled, and on the client, they weren't. Not sure if this is something you want to "fix" or not so I'll leave the issue open.
import_user_oitsjustjose_95mQ commented 2018-06-25 14:42:26 +00:00 (Migrated from git.oitsjustjose.com)

Closing because I don't know that I'm going to struggle with config-syncing. That's a headache that I'm not interested in pursuing, honestly.

Closing because I don't know that I'm going to struggle with config-syncing. That's a headache that I'm not interested in pursuing, honestly.
import_user_oitsjustjose_95mQ (Migrated from git.oitsjustjose.com) closed this issue 2018-06-25 14:42:26 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
oitsjustjose/Geolosys#135
No description provided.