-
Notifications
You must be signed in to change notification settings - Fork 20
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
Crash w/ Tolkien Mobs Ore #139
Comments
Thanks for the report! Ok, so here's what's happening. Pyrotech is hooking Tolkien Mobs' (TM) override for @Override
public ItemStack getItem(World world, BlockPos pos, IBlockState state) {
int oreTypeDrop = world.getBlockState(pos).getValue(ORE_TYPE).getMeta();
return new ItemStack(this, 1, oreTypeDrop);
} At this point, the
The root of the problem is the TM mod not using the given state, however, Pyrotech should not assume that other mods won't assume that the block is still in the world when this method is called. The best solution is to fix the root problem. The second best solution is for Pyrotech to catch any exceptions, log them, and move on. |
I'll go ahead and report this to Tolkien Mobs as well. |
This problem should no longer hard crash in |
Thanks!-----------------------Sincerely, David NewvilleOn Jul 23, 2019 1:56 PM, Taylor <notifications@github.com> wrote:This problem should no longer hard crash in 1.2.9 -- it will still have an error, but it should just log the error and continue.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.
|
Sorry to comment on closed issue, but codetaylor, do you possibly have any ideas on how to fix this? I am currently looking at it but the challenge is because I need to check for meta due to how I manage my different ores |
Issue Description
I have a crash that occurs after mining any ore from the Tolkien Mobs mod.
What Happens
I found Mithril Ore and Morgul Iron Ore, both gave me a crash after mining it. I assume that the Ammolite Ore from the mod would also crash.
What You Expect to Happen
The block usually drops itself.
Script
No script used.
Crash Log
Link
Affected Versions
The text was updated successfully, but these errors were encountered: