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

Exception when you delete Town #3398

Closed
HotdogVanDriver opened this issue Aug 25, 2019 · 14 comments
Closed

Exception when you delete Town #3398

HotdogVanDriver opened this issue Aug 25, 2019 · 14 comments

Comments

@HotdogVanDriver
Copy link

This ticket template is for bug/error reports. If you are suggesting a feature and not reporting a bug please delete all the text here and begin your ticket Title with Suggestion:

What steps will reproduce the problem?

  1. Use deletion command
  2. Confirm town deletion
  3. Exception thrown over and over again in console

What is the expected output?

  1. Town deletes normally

What is your Towny version?

0.94.0.5

What is your spigot/craftbukkit version?

Paper version git-Paper-135

Please use Pastebin.com to link the following files:

Skipping these files is not an option. It is the easiest way to diagnose an issue ticket properly. If you do skip this step then your ticket will most likely be deleted and you will be asked to resubmit.

  1. Your full server startup from the logs\latest.log :
    https://pastebin.com/YHyGfRsv

  2. Your Towny config.yml (if using MySQL please remove password,) :
    https://pastebin.com/YuryB2GZ

  3. Your townyperms.yml :
    https://pastebin.com/180Sgtzt

  4. Your log's error :
    https://pastebin.com/XmiiBk6K

(optional) If this is to do with permissions,
5. Your permissions file :
N/A

(optional) If this is to do with chat,
6. Your chatconfig.yml : N/A
7. Your channels.yml : N/A

@LlmDl
Copy link
Member

LlmDl commented Aug 25, 2019

The town that was deleted is having a hard time reverting the blocks back to their pre-claimed state.

Maybe the town was claimed on an older version of minecraft and towny?

@HotdogVanDriver
Copy link
Author

I had only made the claim about 30 minutes prior. Didn't change the version minecraft or Towny in between. I only installed Towny today.

Just checked my console and it's throwing a new exception literally every single second:
https://pastebin.com/ym7yhYPa

@LlmDl
Copy link
Member

LlmDl commented Aug 25, 2019

Stop the server and delete the towny\data\regen.txt file.

@winnipeg21
Copy link

Having the same issue, world regen is turned off. Config syntax is valid. Paperspigot 172

@LlmDl
Copy link
Member

LlmDl commented Aug 26, 2019

@winnipeg21 turned off in teh config or turned off in the world (use /tw toggle revertunclaim) the config is only for default settings.

@Momshroom
Copy link
Contributor

Same issue: Paper 176 (1.14.4), Towny 0.94.0.6.
The town in question is created, then I immediately do /ta newday... town correctly gets deleted but console starts spamming errors.

@Momshroom
Copy link
Contributor

I can confirm that toggling revertunclaim off before the town is deleted prevents this from happening.

@LlmDl
Copy link
Member

LlmDl commented Sep 3, 2019

@Momshroom anything that would make this town's plot snapshots strange? Which version of Towny were they claimed with, was it pre 0.93.1.0?

@Momshroom
Copy link
Contributor

No. I claimed it immediately before the /ta newday command in 0.94.0.6. I was testing your iConomy fix. (Ty for that!)

@LlmDl
Copy link
Member

LlmDl commented Sep 3, 2019

Can you tell from the log if the snapshot had finished being made?

@Momshroom
Copy link
Contributor

Momshroom commented Sep 3, 2019 via email

@Momshroom
Copy link
Contributor

Momshroom commented Sep 3, 2019

Okay, after updating to 0.94.0.7, I toggled revertunclaim back on, made a new town, waited until the log said the plot snapshots were completed, and then caused the town to fall with /ta newday. It still spams console with the errors:

[15:13:06 WARN]: [Towny] Task #27 for Towny v0.94.0.7 generated an exception
java.lang.NullPointerException: null
        at com.palmergames.bukkit.towny.object.TownBlock.getWorldCoord(TownBlock.java:357) ~[?:?]
        at com.palmergames.bukkit.towny.regen.PlotBlockData.restoreNextBlock(PlotBlockData.java:99) ~[?:?]
        at com.palmergames.bukkit.towny.tasks.RepeatingTimerTask.run(RepeatingTimerTask.java:33) ~[?:?]
        at org.bukkit.craftbukkit.v1_14_R1.scheduler.CraftTask.run(CraftTask.java:84) ~[patched_1.14.4.jar:git-Paper-176]
        at org.bukkit.craftbukkit.v1_14_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:452) ~[patched_1.14.4.jar:git-Paper-176]
        at net.minecraft.server.v1_14_R1.MinecraftServer.b(MinecraftServer.java:1148) ~[patched_1.14.4.jar:git-Paper-176]
        at net.minecraft.server.v1_14_R1.DedicatedServer.b(DedicatedServer.java:417) ~[patched_1.14.4.jar:git-Paper-176]
        at net.minecraft.server.v1_14_R1.MinecraftServer.a(MinecraftServer.java:1075) ~[patched_1.14.4.jar:git-Paper-176]
        at net.minecraft.server.v1_14_R1.MinecraftServer.run(MinecraftServer.java:919) ~[patched_1.14.4.jar:git-Paper-176]
        at java.lang.Thread.run(Thread.java:748) [?:1.8.0_222]
[15:13:07 WARN]: [Towny] Task #27 for Towny v0.94.0.7 generated an exception
java.lang.NullPointerException: null
        at com.palmergames.bukkit.towny.object.TownBlock.getWorldCoord(TownBlock.java:357) ~[?:?]
        at com.palmergames.bukkit.towny.regen.PlotBlockData.restoreNextBlock(PlotBlockData.java:99) ~[?:?]
        at com.palmergames.bukkit.towny.tasks.RepeatingTimerTask.run(RepeatingTimerTask.java:33) ~[?:?]
        at org.bukkit.craftbukkit.v1_14_R1.scheduler.CraftTask.run(CraftTask.java:84) ~[patched_1.14.4.jar:git-Paper-176]
        at org.bukkit.craftbukkit.v1_14_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:452) ~[patched_1.14.4.jar:git-Paper-176]
        at net.minecraft.server.v1_14_R1.MinecraftServer.b(MinecraftServer.java:1148) ~[patched_1.14.4.jar:git-Paper-176]
        at net.minecraft.server.v1_14_R1.DedicatedServer.b(DedicatedServer.java:417) ~[patched_1.14.4.jar:git-Paper-176]
        at net.minecraft.server.v1_14_R1.MinecraftServer.a(MinecraftServer.java:1075) ~[patched_1.14.4.jar:git-Paper-176]
        at net.minecraft.server.v1_14_R1.MinecraftServer.run(MinecraftServer.java:919) ~[patched_1.14.4.jar:git-Paper-176]
        at java.lang.Thread.run(Thread.java:748) [?:1.8.0_222]

@LlmDl
Copy link
Member

LlmDl commented Sep 3, 2019

Ok, tracked this down to a change I made to removing TownBlocks. I'd made the change to greatly reduce spam Exception messages when a town has many townblocks removed. Looks like the fix I made has the side effect of deleting the townblocks too early. Probably have to revert the whole mess and see if there's another better way to solve the exception spam.

LlmDl added a commit that referenced this issue Sep 3, 2019
on Townblocks (Ticket #3398)
      - Had to revert a previous fix for Exception spam that involves
the now-unused RemoveManyTownblocks.
@LlmDl
Copy link
Member

LlmDl commented Sep 3, 2019

This is not longer an issue as of 0.94.0.8, we are reverting the above fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants