Open ItsBecauseUHasNoGoodSpawn ignores islands in world save and always overwrites them

[zeth_russ]

Summary of the problem ItsBecauseUHasNoGoodSpawn ignores islands in world save and always overwrites them

Pack Version 3.0.0

What is the bug? ItsBecauseUHasNoGoodSpawn now puts island information into the world folder at world/island.json (As posted by Darkosto). This information is not used by the mod though. The mod still uses the data from config/island.json to fill its island list. Furthermore it overwrites the values in world/island.json with the values from config/island.json on every save and server start.

This makes the storage of the information in world/island.json completely useless as we still can't discard config/ contents on server upgrades. While an initial migration of islands is necessary the mod should delete config/island.json afterwards and never write to it again. It should furthermore use the values from world/island.json as its source of islands.

Mod & Version Skyfactory 3.0.4
ItsBecauseUHasNoGoodSpawn-1.0.5.jar

Link to log file

Is it repeatable? * start server
* /createIslandFor @p
* stop server

there are now 2 island.json files, one in world/island.json and one in config/island.json. Both contain the island information.

* stop server
* remove config/island.json
* start server

world/island.json is now empty too.

Known Fix none

 

[Darkosto]

Wait, is this mod seriously not deleting the file in the config folder when it's done?

so much facepalming right now. I'll speak to the mod dev and have this fixed for the next pack update.

 

[zeth_russ]

Thanks, the curse page of ItsBecauseUHasNoGoodSpawn does not link to a bugtracker I could post this to.

 

[Darkosto]

What else would you like as a server admin?

 

[zeth_russ]

I'm only a small time admin hosting a private server. As I don't have much experience I can't realy provide you much useful input, sorry.