TownyAdvanced / Towny

Towny Advanced Minecraft plugin for Bukkit/Spigot.
https://townyadvanced.github.io
Other
528 stars 355 forks source link

[Bug] Frames and paintings are not removed on unclaim. #2951

Closed AIphaDragon closed 6 years ago

AIphaDragon commented 6 years ago

Hello. Faced a small problem. Frames and paintings are not regenerated, after the destruction of the town. Thanks in advance for your help.

Video: https://youtu.be/nC_ceHJg7js

What is your Towny version?

0.92.12

What is your spigot/craftbukkit version?

Spigot 1.11.2 - 1.12.2

Please use Pastebin.com to link the following files:

  1. Full server startup from the logs\latest.log : https://pastebin.com/dPzEH1Zq
  2. Towny config.yml : https://pastebin.com/KbgkNiRg
LlmDl commented 6 years ago

I have tested and confirmed this is happening. I'll have to get it to handle those entities. Probably need to test/add EnderCrystals, ArmorStands to that list and any other blocks which are actually entities.

MartenM commented 6 years ago

Would a 3x3 check round the block (or only the faces, and then remove the frame) be enough to solve this problem? Otherwise I could possibly create a PR for this.

LlmDl commented 6 years ago

There's two really inefficient ways to do this that i know of:

I'm going to see if there's a way to get all entities in a chunk/space. Hopefully that's the case.

LlmDl commented 6 years ago

So I figured out that this whole nonsense is actually caused by Towny, in a fix to a pretty obscure but semi-serious bug. I can undo the previous bugfix but there's no way to then re-fix that bug until Spigot fixes/updates their code. I'd elaborate more but don't want the previous bug to become common knowledge.

LlmDl commented 6 years ago

Fixed as of 0.92.0.13