Jake Potrebic

Rebased for 1.18.1

This is still valid, but will be implemented with the Registry Modification API found in #8920

Revamped this to use the new registry modification API.

The project is open source for a reason. Go ahead. Oh BTW, I did confirm this issue exists. The event was cancelled insofar as the bolt did not fire, but...

Shouldn't this then mimic the behavior of https://github.com/PaperMC/Paper/pull/11555 where it's not called for non-positive damage. Then smth else needs to be added for "durability change" or something.

I think its fine to fix all the cases here.

I don't like making more functionality dependant on ops. The concept of "op" is not a good thing to really encourage more. People should be using proper permissions. I thought...

Ok, I think we want a permission, but *just* for command suggestions, `paper.bypass-visibility.tab-completion`

I'm not sure I understand what you are saying for the "global" stat vs "material-unique" stat. I know that the API's `Statistic#MINE_BLOCK` should be checked with a material. So I...

Yeah, that code will throw an exception on runtime. You cannot get the stat for MINE_BLOCK without providing a block type. It throws an exception otherwise. It seems to work...