Open Andre601 opened 1 year ago
After further testing is this also happening with normal sculk blocks it seems.
Is the arena protected with WorldGuard? If not, do so. If it still happens, WorldGuard might not cover sculk veins and that's a bug report for them.
WorldGuard is installed. They have a flag which wasn't set and setting it did prevent ghe spread.
The issue still remains that MobArena's own protection should cover this as it does with all the other kinds of block break and block change events.
Perhaps it would be a good idea to consolidate "arena structure protection" issues into one issue. We also have #518, #580, #447, #445. It might actually be possible to rework some of the restoration and protection logic now that 1.13 is the minimum supported version. The major API changes in 1.13 made it overwhelmingly difficult to maintain in the current code base, but now that the API is stable (right?) in that regard, we could maybe give it a bit of an overhaul without too much effort?
Maybe it's worth splitting the problem into protection and restoration. Protection should generally just be about cancelling events, whereas restoration is more about recording region information prior to allowing changes to happen, and then restoring the original layout at arena end.
Is this something you'd be up for taking a stab at @Andre601?
Is this something you'd be up for taking a stab at @Andre601?
No, because of two things:
So you have to find someone else to do this with or try and make it yourself. I can't and won't help on this one unfortunately.
Alright, if you muster up the confidence to take a stab at it at some point, I'll obviously help you out :) I don't expect anyone to do it alone. Good architecture is hard enough as it is and definitely requires a lot of back and forth. Refactoring an old architecture in a foreign code base even more so. It would be an opportunity to get involved and participate in some of the rework efforts, but it's not for everyone :)
But no worries, I'll get around to it at some point, but if anyone wants to join in, hit me up on Discord.
Bug report
Short description When you kill a mob near a sculk catalyst will the blocks not turn into sculk, which is good. But sculk veins still spread without any issue.
Reproduction steps
Details
Additional info