search

ChanceSD / PvPManager

A Spigot/Paper plugin to toggle PvP, stop combat log, protect new players and much more
https://www.spigotmc.org/resources/pvpmanager.845/
Other
130 stars 47 forks source link

/announce command?!? #452

Closed MarvinKlar closed 7 months ago

MarvinKlar commented 7 months ago

Describe the bug Users can use "/announce", but that command is not mentioned anywhere?

To Reproduce Steps to reproduce the behavior:

  1. Install PvPManager, start the server, run "/announce test"

Expected behavior I expect this command not to be there.

Plugin and Server version Any

Screenshots grafik

ChanceSD commented 7 months ago

That announce is not from PvPManager.

MarvinKlar commented 7 months ago

It is, when disabling the plugin, the command is no longer there

MarvinKlar commented 7 months ago

@ChanceSD

ChanceSD commented 7 months ago

It is not, there is no message saying "You have to use /announce message" here, and the command from here requires a permission https://github.com/ChanceSD/PvPManager/blob/master/pvpmanager/src/main/java/me/NoChance/PvPManager/Commands/Announce.java Edit: Also since suddenly 2 people noticed this command (and appear to have an issue with it for some reason) and for any future readers, I added a description of the command to the spigot page. And the command won't be removed as there isn't any reason to do so. The command has existed for years, I found it useful on my own server back then to be able to broadcast messages without a predefined prefix, it's also useful in the config to broadcast messages for any features using commands.