I have not modified it because it would be a breaking change, but ideally
```java
(checkMetadata && metadata > 0 ? ":" + metadata : "");
```
should be:
```
(checkMetadata ? ":" + metadata : "");
```
As this allows creating wildcards like "modifyworld.blocks.interact.stone:\*" when check-metadata is enabled, which right now has to be done either creating two different nodes ("modifyworld.blocks.interact.stone" and "modifyworld.blocks.interact.stone:\*"), or creating a single node with an asterisk at the end (like "modifyworld.blocks.interact.stone\*"), which is a bad idea, since for example it would also match "stonebutton".