IntellectualSites / FastAsyncWorldEdit

Blazingly fast world manipulation for artists, builders and everyone else: https://www.spigotmc.org/resources/13932/
Other
601 stars 213 forks source link

Pasting over top of spawners and some other tile entities corrupts/resets them #700

Closed Combustible closed 3 years ago

Combustible commented 3 years ago

Bug report for FastAsyncWorldEdit 1.16.3 / 1.15.2 (reproducible on both)

FastAsyncWorldEdit Configuration Files: https://athion.net/ISPaster/paste/view/de62a1c082364332bf2d1ce33986fa21

Required Information

Description of the problem: Pasting a spawner on top of itself will wipe the data from the spawner. This is easy to reproduce and happens every time.

This happens in 1.15 and 1.16.3, with latest paper and latest FAWE. It also happens with brewstands and furnaces, but it's less obvious since they need to have something in them. Interestingly, this is not a problem with chests - it might be related to tile entities that tick themselves?

How to replicate:

This is an important workflow on my game server where we "respawn" areas of the map using FAWE. We have been working around by basically doing //replace spawner dirt first but this is far from ideal and causes us other problems.

Plugins being used on the server:

Plugins (24): Arceon, BKCommonLib, BungeeTabListPlus, CommandAPI, EpicStructureManagement, (WorldEdit)FastAsyncWorldEdit, goBrush, goPaint, LibraryOfSouls, LightCleaner, LuckPerms, Monumenta, MonumentaRedisSync, MonumentaWarps, NBTAPI, NBTEditor, PlaceholderAPI, PrometheusExporter, ProtocolLib, ScriptedQuests, SpeedChanger*, Vault, VentureChat, VoxelSniper

This is reproducible with just FAWE alone but this is the full list to match the server I gathered the debug paste link from.

Checklist:

Combustible commented 3 years ago

Anything I can do to help with this? I'm actually really struggling right now as somehow the latest FAWE has made my hack replace-spawners-with-dirt-first fix not work as reliably - so getting this solved is of even more value to me.

I'm really handy with plugin code, and have built and contributed to FAWE in the past - but I haven't figured out where to look on this one. I'd be happy to do some digging to get more info / try stuff if I had some idea how FAWE was updating tile entities like spawners to begin with...

Combustible commented 3 years ago

Figured out how to build a better hack to work around this (still setting spawners / brewstands / furnaces to dirt before pasting). So at least my server can continue with newest FAWE :)

Still interested in seeing this fixed and willing to help if I can.

Xaver106 commented 3 years ago

Tested with:

The issue with the tile entitys looks like it is fixed. I tested commen inventory blocks (chest, furnace, brewing stand, etc.), the inventory stayed the same. With the spawners it looks like it is working (I used the spawner from above). The spawner is properly overwritten. The Spawners seem to be fixed as well. It can sometimes happen, theat the spawners, that get pasted over reset themselves. The resetted spawners stay in on chunk, the ones outside are usually not affected. The Spawners can just get pasted over to fix the problem.