Adding starter/newbie kit to EssentialsSpawn

Optimization to EssentialsSpawn join event.

Essentials/src/com/earth2me/essentials/IEssentials.java

@@ -9,6 +9,7 @@ import org.bukkit.command.CommandSender;
 import org.bukkit.plugin.Plugin;
 import org.bukkit.scheduler.BukkitScheduler;
 
+
 /**
  * @deprecated This will be moved to the api package soon
  */

Essentials/src/com/earth2me/essentials/ISettings.java

@@ -20,6 +20,8 @@ public interface ISettings extends IConf
 
 	boolean getAnnounceNewPlayers();
 
+	String getNewPlayerKit();
+
 	String getBackupCommand();
 
 	long getBackupInterval();

Essentials/src/com/earth2me/essentials/Kit.java

@@ -63,7 +63,8 @@ public class Kit
 
 	public static List<String> getItems(final User user, final Map<String, Object> kit) throws Exception
 	{
-		if (kit == null) {
+		if (kit == null)
+		{
 			throw new Exception(_("kitError2"));
 		}
 

Essentials/src/com/earth2me/essentials/Settings.java

@@ -351,6 +351,12 @@ public class Settings implements ISettings
 		return new SimpleTextInput(Util.replaceColor(config.getString("newbies.announce-format", "&dWelcome {DISPLAYNAME} to the server!")));
 	}
 
+	@Override
+	public String getNewPlayerKit()
+	{
+		return config.getString("newbies.kit", "");
+	}
+
 	@Override
 	public String getNewbieSpawn()
 	{

Essentials/src/config.yml

@@ -548,6 +548,11 @@ newbies:
   # Set to "none" if you want to use the spawn point of the world.
   spawnpoint: newbies
   
+  # Do we want to give users anything on first join? Set to '' to disable
+  # This kit will be given reguardless of cost, and permissions.
+  #kit: ''
+  kit: tools
+
 # Set this to lowest, if you want Multiverse to handle the respawning
 # Set this to high, if you want EssentialsSpawn to handle the respawning
 # Set this to highest, if you want to force EssentialsSpawn to handle the respawning

EssentialsSpawn/src/com/earth2me/essentials/spawn/EssentialsSpawnPlayerListener.java

@@ -1,12 +1,13 @@
 package com.earth2me.essentials.spawn;
 
+import com.earth2me.essentials.*;
 import static com.earth2me.essentials.I18n._;
-import com.earth2me.essentials.IEssentials;
-import com.earth2me.essentials.OfflinePlayer;
-import com.earth2me.essentials.User;
 import com.earth2me.essentials.textreader.IText;
 import com.earth2me.essentials.textreader.KeywordReplacer;
 import com.earth2me.essentials.textreader.SimpleTextPager;
+import java.util.List;
+import java.util.Locale;
+import java.util.Map;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 import org.bukkit.Bukkit;
@@ -67,13 +68,14 @@ public class EssentialsSpawnPlayerListener implements Listener
 
 	public void onPlayerJoin(final PlayerJoinEvent event)
 	{
-		final User user = ess.getUser(event.getPlayer());
+		if (event.getPlayer().hasPlayedBefore())
-
-		if (user.hasPlayedBefore())
 		{
 			LOGGER.log(Level.FINE, "Old player join");
 			return;
 		}
+
+		final User user = ess.getUser(event.getPlayer());
+
 		if (!"none".equalsIgnoreCase(ess.getSettings().getNewbieSpawn()))
 		{
 			ess.scheduleSyncDelayedTask(new NewPlayerTeleport(user), 1L);
@@ -86,6 +88,21 @@ public class EssentialsSpawnPlayerListener implements Listener
 			ess.broadcastMessage(user, pager.getString(0));
 		}
 
+		final String kitName = ess.getSettings().getNewPlayerKit();
+		if (!kitName.isEmpty())
+		{
+			try
+			{
+				final Map<String, Object> kit = ess.getSettings().getKit(kitName.toLowerCase(Locale.ENGLISH));
+				final List<String> items = Kit.getItems(user, kit);
+				Kit.expandItems(ess, user, items);
+			}
+			catch (Exception ex)
+			{
+				LOGGER.log(Level.WARNING, ex.getMessage());
+			}
+		}
+
 		LOGGER.log(Level.FINE, "New player join");
 	}
 
@@ -109,7 +126,7 @@ public class EssentialsSpawnPlayerListener implements Listener
 
 			try
 			{
-				Location spawn = spawns.getSpawn(ess.getSettings().getNewbieSpawn());
+				final Location spawn = spawns.getSpawn(ess.getSettings().getNewbieSpawn());
 				if (spawn != null)
 				{
 					user.getTeleport().now(spawn, false, TeleportCause.PLUGIN);