BentoBoxWorld / Level

Add on for BentoBox to calculate island levels for BSkyBlock and AcidIsland. Please read the Wiki!
Eclipse Public License 2.0
25 stars 32 forks source link

Level calculation build 295 #101

Closed x1p closed 5 years ago

x1p commented 5 years ago

Description Build 295 introduces weird level calculation causing all existing islands to lose much levels. Placing this build on a production server makes every island to lose up to 1000 levels. This makes players unhappy. Reverting to build 294 (or lower) resolves it.

Steps to reproduce the behavior:

  1. Start default test server with empty world
  2. Start default island (the one with the moomoo cow)
  3. Calculate is level (I used my own custom level values)
  4. Check level file
    {
    "uniqueId": "my-uuid",
    "levels": {
    "bskyblock_world": -2
    },
    "initialLevel": {
    "bskyblock_world": 186
    }
    }
  5. Placing 1000 blocks with value 4 = 4000, = 40 is levels
  6. Calculate is level
  7. Check level file
    {
    "uniqueId": "my-uuid",
    "levels": {
    "bskyblock_world": 38
    },
    "initialLevel": {
    "bskyblock_world": 186
    }
    }
  8. Stop server, replace add on Level build 294 with build 295, start server again
  9. Login and calculate is level
  10. Check level file that has changed
    {
    "uniqueId": "my-uuid",
    "levels": {
    "bskyblock_world": -145
    },
    "initialLevel": {
    "bskyblock_world": 186
    }
    }

    Placing another 1000 blocks makes the level increase with another 40 levels, however it still counts from the new negative island level.

Expected behavior Level calculation like it used to do before

Screenshots If applicable, add screenshots to help explain your problem.

Server Information:

[Please complete the following information:]

Additional context Starting the same island from scratch again with build 295 gets completely different level initial information:


{
  "uniqueId": "my-uuid",
  "levels": {
    "bskyblock_world": 0
  },
  "initialLevel": {
    "bskyblock_world": 1
  }
}
tastybento commented 5 years ago

In the console you can get a full report on the level. Can you do bsb level (your name) in console with the old build and new build and then paste them so I can see them? By comparing the reports I’ll be able to see exactly where the differences are.

tastybento commented 5 years ago

Also, please try the latest build from the CI.

x1p commented 5 years ago

https://hastebin.com/poxejafome.md Latest 306 build: https://hastebin.com/vodeleripi.md

tastybento commented 5 years ago

Looks like Nether and End island calculations are just being missed. I'll fix.

tastybento commented 5 years ago

It should be fixed now. Please try it.