CloudburstMC / Cloudburst

Cloudburst Server is a Bedrock first server software. Originally forked from Cloudburst Nukkit.
182 stars 71 forks source link

[2.0] Unimplemented Daylight Detector block entity #11

Open squeeglii opened 4 years ago

squeeglii commented 4 years ago

Expected Behavior

The Java 1.12.2 world is expected to convert and then run on the server. If there was a problem with the world file, surely it could have some meaningful error to an end-user or (if it was a specific block) the offender could be removed during conversion.

Actual Behavior

I've only run into this crash on this specific world so it's a minor issue. Forge has been used with this world, however, it has only ever had worldedit on it, thus it shouldn't be an issue with modded blocks or entities.

The server logs the chunks being converted and such and then it just exits without any information. Checking the latest log, it reveals an IllegalStateException caused by a NullPointerException (Thrown by a checknotnull).

Starting the server again, it begins to try to convert the level again, failing every time.

Steps to Reproduce

Load the attached world below onto a server and start the server.

Crashdump, Backtrace or Other Files

Checklist:

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/92150791-2-0-unimplemented-daylight-detector-block-entity?utm_campaign=plugin&utm_content=tracker%2F182923979&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F182923979&utm_medium=issues&utm_source=github).
Extollite commented 4 years ago

It's not working because NukkitX 2.0 has no implemented DaylightDetectors, if you really want to use this map try to use mcctoolchest and delete them.

SupremeMortal commented 4 years ago

We should at least implement the base daylight detector block entity so that the world can be loaded without an error.

squeeglii commented 4 years ago

So we decided to worldedit all the daylight sensors away for the time being, however, we're still running into the exact same error. I did an MCEdit analysis and we now only have these tile entities: analysis

world log