Terrain Control

Terrain Control

235k Downloads

Cannot load multiple TerrainControl worlds w/Sponge, 1.10

Dockter opened this issue · 10 comments

commented

Hello,

So I got one world loaded into TerrainControl but when I go to create the second world I get the following error:

http://pastebin.com/raw/evkPeKAW

Let me know if you need additional information.

commented

This error is also easy to re-produce.

commented

Yeah, this is a missing feature. The Forge version is not able to load multiple worlds at the same time yet. It would be nice if it could.

commented

Rutgerkok,

What exactly is missing to support this? I kinda need this in order to move my server from Cauldron 1.7.10 to SpongeForge 1.10.

Thanks,

commented

Rutgerkok,

Do you hide out on any IRC, I'd like to PM you regarding this issue.

Thanks,

commented

@rutgerkok , I found what I think is a fix that will work for us, basically I remove the following if check:

https://github.com/MCTCP/TerrainControl/blob/master/platforms/forge/src/main/java/com/khorn/terraincontrol/forge/WorldLoader.java#L144

Once I removed that line I was able to load and create multiple TC worlds within Forge without issue. On our server we don't ever unload a world once its loaded so I won't run into a double load scenario since we only load the world once per server session.

commented

@mcsnetworks Nice to see someone fixed that issue. I tried your solution, but this doesn't seem to work for me. Can you please explain the following to me:

  • How do you create new dimensons / worlds?
    • What mod / sponge pluging
    • What commands
  • Can you please give me your patched and compiled binary

Thank you

Michael (aka BakermanLP)

commented

My fix did not work. TerrainControl for Forge doesn't load unique instances of each world. Therefore each additional world overwrites the ForgeWorld configs that are loaded, thus screwing everything up.

I've been trying to get @rutgerkok to finish this and even volunteered to pay for his time but have received no response.

commented

We are testing a few changes internally to add Multi-World support for TC Forge. I'll PR it once we finalize the code.

commented

@rutgerkok I believe this was solved with #469 and ab469d5. Can be closed.

commented

This is no longer an issue in latest 1.12 snapshot.