Add length to database primary keys.

Tweak some of the itembans code to be less inefficient.
Removed a chunk of code from the buff code that prevented users from being buffed with a buff that comes from an item.  This code seemed to cause max cpu usage when being disabled.  I don't think this code is useful, but I leave that to people who own servers.  IMO all buff code should be removed.

TShockAPI/TShock.cs

@@ -804,6 +804,7 @@ namespace TShockAPI
 					}
 					player.IgnoreActionsForCheating = check;
 					check = "none";
+					//todo: pretty sure we check every place a players inventory can change, so do we really need to do this?
 					foreach (Item item in player.TPlayer.armor)
 					{
 						if (Itembans.ItemIsBanned(item.name, player))