Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jail attempts to jail players that have never joined the server #82

Closed
hitechwizard opened this issue Jun 8, 2015 · 3 comments
Closed

Comments

@hitechwizard
Copy link

Using the /jail command, I seem to jail people like sign, etc. Before attempting to jail a player, jail should verify that the player has been on the server before.

The easiest way to check this is to use the Bukkit.OfflinePlayer.getLastPlayed(), which will return -1 if the player hasn't been on the server.

@graywolf336
Copy link
Owner

Couple ideas came to me from this.

First one is doing this but making it controllable via the config that way
if someone wanted that feature enabled it could be. Then if it is enabled
they have to type the confirm command to really jail someone who has never
been on before.

Second is command matching/suggestions. If you only provide one argument
then it suggests commands if it didn't find any.
On Jun 7, 2015 9:55 PM, "hitechwizard" notifications@github.com wrote:

Using the /jail command, I seem to jail people like sign, etc. Before
attempting to jail a player, jail should verify that the player has been on
the server before.

The easiest way to check this is to use the
Bukkit.OfflinePlayer.getLastPlayed(), which will return -1 if the player
hasn't been on the server.


Reply to this email directly or view it on GitHub
#82.

@graywolf336
Copy link
Owner

Marking it as an enhancement since the old version of jail allowed for jailing players who had never visited the server before.

@graywolf336
Copy link
Owner

You can overwrite this with -f in the /jail command now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants