CB #2267 Bukkit #1505

Updates to gamemode command, mode is now mandatory [creative|survival|adventure]
Updates to gamemode sign, now requires another line with gamemode

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

@@ -67,7 +67,7 @@ import org.yaml.snakeyaml.error.YAMLException;
 
 public class Essentials extends JavaPlugin implements IEssentials
 {
-	public static final int BUKKIT_VERSION = 2149;
+	public static final int BUKKIT_VERSION = 2267;
 	private static final Logger LOGGER = Logger.getLogger("Minecraft");
 	private transient ISettings settings;
 	private final transient TNTExplodeListener tntListener = new TNTExplodeListener(this);

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

@@ -1051,4 +1051,22 @@ public class OfflinePlayer implements Player
 	{
 		throw new UnsupportedOperationException("Not supported yet.");
 	}
+	
+	@Override
+	public int getExpToLevel()
+	{
+		throw new UnsupportedOperationException("Not supported yet.");
+	}
+
+	@Override
+	public boolean hasLineOfSight(Entity entity)
+	{
+		throw new UnsupportedOperationException("Not supported yet.");
+	}
+
+	@Override
+	public boolean isValid()
+	{
+		throw new UnsupportedOperationException("Not supported yet.");
+	}	
 }

Essentials/src/com/earth2me/essentials/commands/Commandgamemode.java

@@ -19,24 +19,27 @@ public class Commandgamemode extends EssentialsCommand
 	@Override
 	protected void run(final Server server, final CommandSender sender, final String commandLabel, final String[] args) throws Exception
 	{
-		if (args.length < 1)
+		if (args.length < 2)
 		{
 			throw new NotEnoughArgumentsException();
 		}
-
 		gamemodeOtherPlayers(server, sender, args);
 	}
 
 	@Override
 	protected void run(final Server server, final User user, final String commandLabel, final String[] args) throws Exception
 	{
-		if (args.length > 0 && args[0].trim().length() > 2 && user.isAuthorized("essentials.gamemode.others"))
+		if(args.length < 1)
+		{
+			throw new NotEnoughArgumentsException();
+		}
+		
+		if (args.length > 1 && args[0].trim().length() > 2 && user.isAuthorized("essentials.gamemode.others"))
 		{
 			gamemodeOtherPlayers(server, user, args);
 			return;
 		}	
-
+		performSetMode(args[0], user);
-		user.setGameMode(user.getGameMode() == GameMode.SURVIVAL ? GameMode.CREATIVE : GameMode.SURVIVAL);
 		user.sendMessage(_("gameMode", _(user.getGameMode().toString().toLowerCase(Locale.ENGLISH)), user.getDisplayName()));
 	}
 
@@ -49,23 +52,24 @@ public class Commandgamemode extends EssentialsCommand
 			{
 				continue;
 			}		
-
+			performSetMode(args[1], player);
-			if (args.length > 1)
-			{
-				if (args[1].contains("creat") || args[1].equalsIgnoreCase("1"))
-				{
-					player.setGameMode(GameMode.CREATIVE);
-				}
-				else
-				{
-					player.setGameMode(GameMode.SURVIVAL);
-				}
-			}
-			else
-			{
-				player.setGameMode(player.getGameMode() == GameMode.SURVIVAL ? GameMode.CREATIVE : GameMode.SURVIVAL);
-			}
 			sender.sendMessage(_("gameMode", _(player.getGameMode().toString().toLowerCase(Locale.ENGLISH)), player.getDisplayName()));
 		}
 	}
+	
+	private void performSetMode(String mode, Player player)
+	{
+			if (mode.contains("survi") || mode.equalsIgnoreCase("0"))
+			{
+				player.setGameMode(GameMode.SURVIVAL);
+			}
+			else if (mode.contains("creat") || mode.equalsIgnoreCase("1"))
+			{
+				player.setGameMode(GameMode.CREATIVE);
+			}
+			else if (mode.contains("advent") || mode.equalsIgnoreCase("2"))
+			{
+				player.setGameMode(GameMode.ADVENTURE);
+			}
+	}
 }

Essentials/src/com/earth2me/essentials/craftbukkit/FakeWorld.java

@@ -669,4 +669,10 @@ public class FakeWorld implements World
 	{
 		throw new UnsupportedOperationException("Not supported yet.");
 	}
+
+	@Override
+	public Entity spawnEntity(Location lctn, EntityType et)
+	{
+		throw new UnsupportedOperationException("Not supported yet.");
+	}
 }

Essentials/src/com/earth2me/essentials/signs/SignGameMode.java

@@ -7,6 +7,7 @@ import com.earth2me.essentials.Trade;
 import com.earth2me.essentials.User;
 import java.util.Locale;
 import org.bukkit.GameMode;
+import org.bukkit.entity.Player;
 
 
 public class SignGameMode extends EssentialsSign
@@ -27,11 +28,35 @@ public class SignGameMode extends EssentialsSign
 	protected boolean onSignInteract(final ISign sign, final User player, final String username, final IEssentials ess) throws SignException, ChargeException
 	{
 		final Trade charge = getTrade(sign, 1, ess);
+		final String mode = sign.getLine(2).trim();
+		
+		if(mode.isEmpty())
+		{
+			throw new SignException(_("invalidSignLine", 3));
+		}
+		
 		charge.isAffordableFor(player);
 
-		player.setGameMode(player.getGameMode() == GameMode.SURVIVAL ? GameMode.CREATIVE : GameMode.SURVIVAL);
+		performSetMode(mode, player);
 		player.sendMessage(_("gameMode", _(player.getGameMode().toString().toLowerCase(Locale.ENGLISH)), player.getDisplayName()));
 		charge.charge(player);
 		return true;
 	}
+	
+	private void performSetMode(String mode, Player player)
+	{
+			if (mode.contains("survi") || mode.equalsIgnoreCase("0"))
+			{
+				player.setGameMode(GameMode.SURVIVAL);
+			}
+			else if (mode.contains("creat") || mode.equalsIgnoreCase("1"))
+			{
+				player.setGameMode(GameMode.CREATIVE);
+			}
+			else if (mode.contains("advent") || mode.equalsIgnoreCase("2"))
+			{
+				player.setGameMode(GameMode.ADVENTURE);
+			}
+	}
+	
 }

Essentials/src/plugin.yml

@@ -117,7 +117,7 @@ commands:
     aliases: [efireball]
   gamemode:
     description: Change player gamemode.
-    usage: /<command> [player]
+    usage: /<command> [player] <creative|survival|adventure>
     aliases: [gm,creative,creativemode,egamemode,ecreative,ecreativemode,egm]
   getpos:
     description: Get your current coordinates or those of a player.

Essentials/test/com/earth2me/essentials/FakeServer.java

@@ -741,4 +741,16 @@ public class FakeServer implements Server
 	{
 		throw new UnsupportedOperationException("Not supported yet.");
 	}
+
+	@Override
+	public boolean isPrimaryThread()
+	{
+		throw new UnsupportedOperationException("Not supported yet.");
+	}
+
+	@Override
+	public String getMotd()
+	{
+		throw new UnsupportedOperationException("Not supported yet.");
+	}
 }