hawks008/Bukkit-JavaDoc — pull request #1 by hawks008 (2013-10-04 08:24:12+00:00)
Title: [B+C+JD] Add Ban Reason to "Vanilla" Bukkit - Fixes BUKKIT-4285

**The Issue:**
Currently, there is no way to set the ban reason and banning player without using an external plugin.

**Justification:**
By having 1 less plugin to install this will increase (not by much) the server speed of many servers as they will no longer have to write to the databases that they use to store their own bans (which if they are using external servers, can become very laggy for muiltiple write requests)

**Breakdown:**
[C]This just added the required methods to keep current system working as well as new bans.
[B]This added the required changes to the ban command to parse reason and banning player.
[JD]This addresses changes to official javadoc for OfflinePlayer.setBanned

**Testing:**
Compiled server and started banning users with random reasons. Checked banned-players.txt for verification.
Also banned some with no reason to test if both options work.

**Relevant PR(s):**
https://github.com/Bukkit/CraftBukkit/pull/1250
https://github.com/Bukkit/Bukkit/pull/935

**JIRA Ticket:**
https://bukkit.atlassian.net/browse/BUKKIT-4813 - Closed
https://bukkit.atlassian.net/browse/BUKKIT-4285 - Not mine but suits