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

Crash when using the W&S workstation #1133

Closed
BrazenDynamo opened this issue Jun 11, 2014 · 4 comments
Closed

Crash when using the W&S workstation #1133

BrazenDynamo opened this issue Jun 11, 2014 · 4 comments
Labels
Status: Needs Discussion Requires help discussing a reported issue or provided PR Type: Bug Issues reporting and PRs fixing problems

Comments

@BrazenDynamo
Copy link

Logfile here: http://pastebin.com/jnj3j20j

I don't know why, but when I try opening the subfolder for "Planks" under "Buildings", the game crashes. D:

@Cervator
Copy link
Member

Confirmed! Able to reproduce with the latest build. It isn't just you :-)

Central log snippet:

01:30:12.208 [main] WARN  org.terasology.asset.AssetManager - Unable to resolve asset: shape:Engine:StructuralResources:PillarTop
01:30:12.209 [main] WARN  o.t.w.b.internal.BlockManagerImpl - Failed to resolve block family 'Core:Plank:Engine:StructuralResources:PillarTop'
01:30:12.213 [main] ERROR o.terasology.engine.TerasologyEngine - Uncaught exception
java.lang.NullPointerException: null
    at org.terasology.was.system.PlankBlockRecipe$PlankBlockRecipeResultFactory.getBlock(PlankBlockRecipe.java:67) ~[na:na]
    at org.terasology.crafting.system.recipe.render.result.BlockRecipeResultFactory.setupDisplay(BlockRecipeResultFactory.java:67) ~[na:na]

The issue is from when we moved some shapes to the StructuralResources module from the engine. The addPlankBlockShapeRecipe helper method in RegisterWoodAndStoneRecipes (in the WoodAndStone module) is hard coded to use shapes from "Engine" and we end up with something like shape:Engine:StructuralResources:PillarTop

This isn't a hard fix, could probably sort something out, but I wonder how @MarcinSc would prefer it fixed - you out there Marcin?

Helper method in question:

    private void addPlankBlockShapeRecipe(String shape, int ingredientMultiplier, int durabilityMultiplier, int resultMultiplier) {
        addPlankBlockShapeRecipe(shape, "Engine", ingredientMultiplier, durabilityMultiplier, resultMultiplier);
    }

There is another spot in the same class that may also need to be more dynamic (line 197)

@Cervator Cervator added the Bug label Jun 12, 2014
@Cervator
Copy link
Member

@BrazenDynamo - try this now? I triggered a rebuild of the main game distribution (unstable version) after @MarcinSc applied a couple fixes to the WoodAndStone module

http://jenkins.movingblocks.net/job/Terasology/lastSuccessfulBuild/artifact/build/distributions/Terasology.zip

@BrazenDynamo
Copy link
Author

@Cervator Just tried it- it's working fine now! :D
I'll play around with it more and report if there are any other issues.

@Cervator
Copy link
Member

Good to hear :-)

If you find any related issues check the WoodAndStone tracker + Jenkins jobs in case they're known or fixed. I rebuilt the main game distribution just to re-bundle the latest version, but Marcin might make more fixes you can grab directly even without a new game distribution

https://github.com/Terasology/WoodAndStone
http://jenkins.movingblocks.net/view/Modules/job/WoodAndStone/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Discussion Requires help discussing a reported issue or provided PR Type: Bug Issues reporting and PRs fixing problems
Projects
None yet
Development

No branches or pull requests

2 participants