PyroMining and Slimefun Incompatibility
ItsSynchro opened this issue ยท 5 comments
๐ Description (REQUIRED)
Whilst using the PyroMining plugin along side Slimefun's Explosive Pickaxe, it appears that players are able to mass produce the items the plugin drops from mining the ore. I've gone ahead and talked to Pyro (developer of the PyroMining plugin), and he's mentioned that he has already set the events to the highest priority so that its more than likely Slimefun overriding something.
๐ Steps to reproduce the Issue (REQUIRED)
๐ก Expected behavior (REQUIRED)
Prevent the Explosive Pickaxe from being able to override the events from other plugins.
๐งญ Environment (REQUIRED)
@LinoxGH Nope, we should change our event priority as suggested.
I just looked at the code and it seems to operate under the MONITOR
priority which is absolutely not right at all.
It should probably run under something like HIGH
instead.
@LinoxGH Nope, we should change our event priority as suggested.
I just looked at the code and it seems to operate under theMONITOR
priority which is absolutely not right at all.
It should probably run under something likeHIGH
instead.
Well even then checking if event is cancelled will probably fix at least the dupe errors in sf addons like luckyblocks, PrivateStorage, Barrels.
It's because of the priority.
MONITOR is only for monitoring and not for manipulating the event outcome, however our Listener does manipulate the outcome, this did not use to be the case in the past but now it is and the priority remained unchanged even though it should have been changed. Hence why we should change it by now. That would fix this issue.