
Here we can see that you register one permission but check a different one without a "noammoconsumption." prefix. I would suggest removing prefix from Init and adding it to config instead.

There isn't really a need for it to be configurable, I'd keep it in the plugin itself.
Oh thank you for pointing it out misticos
Wulf
There isn't really a need for it to be configurable, I'd keep it in the plugin itself.
I think it's important for a plugin like this to leave room for such configuration options. however I could add a usePermissions option to the config if thats what you meant?
Ah, I didn't look to see how it used. I'd say keep the prefix in the plugin though at least, keeping the config like it is to avoid issues if the plugin is ever renamed.KibbeWater
I think it's important for a plugin like this to leave room for such configuration options. however I could add a usePermissions option to the config if thats what you meant?
oh thats what you mean, yeah for sure.
Wasn't what I meant originally, but having the prefix in the config wouldn't really be necessary.KibbeWater
oh thats what you mean, yeah for sure.