Thanks I'll take a look.
Was the zone actually created when you deleted the crate, or was a creation delay in effect?
Merged post
Also, please try with debug logging enabled.
I don't see how this can happen, as there's only one place in code where zone delete happens, and it's not possible for the hook call to be skipped.
In fact, you should often get TWO delete hook calls, because DPVP calls it first before scheduling a delayed delete in case you want to cancel the delete, then calls it again when the actual deletion occurs. I'm not sure how that was ever useful to anyone, because it's not possible for another plugin to determine which of those two cases it's being called for.
Merged post
First, I was wrong about the hooks - there are two of them:
- OnDeleteDynamicPVP gets called before deleting the zone, and plugins can return non-null to prevent deletion (which is problematic IMO because then it just gets stuck forever)
- OnDeletedDynamicPVP gets called after deleting the zone, to notify plugins of zone deletion
Second,
I can't replicate this. I tried adding debug logging and deleting a hackable crate with DPVP 4.10.0 on a non-staging test server, both with and without event stop delays, and both times it fired both hooks at the correct time.
Here is my version 4.10.0 with debug logging added if you want to try it:
https://pastebin.com/CanTRSCpMerged postFYI, running a non-current version of Zone Manager seems to be causing issues. Make sure you're up to date.