Attempt to restore compatibility with MCPC/Cauldron

Addresses aadnk#90, aadnk#83, and aadnk#71

ProtocolLib/src/main/java/com/comphenix/protocol/CommandProtocol.java

@@ -42,12 +42,10 @@ class CommandProtocol extends CommandBase {
 	public static final String NAME = "protocol";
 
 	private Plugin plugin;
-	private ProtocolConfig config;
 
-	public CommandProtocol(ErrorReporter reporter, Plugin plugin, ProtocolConfig config) {
+	public CommandProtocol(ErrorReporter reporter, Plugin plugin) {
 		super(reporter, CommandBase.PERMISSION_ADMIN, NAME, 1);
 		this.plugin = plugin;
-		this.config = config;
 	}
 
 	@Override
@@ -140,16 +138,6 @@ class CommandProtocol extends CommandBase {
 		sender.sendMessage(ChatColor.WHITE + "Issues: " + ChatColor.GREEN + "https://github.com/dmulloy2/ProtocolLib/issues");
 	}
 
-	/**
-	 * Prevent further automatic updates until the next delay.
-	 */
-	public void updateFinished() {
-		long currentTime = System.currentTimeMillis() / ProtocolLibrary.MILLI_PER_SECOND;
-
-		config.setAutoLastTime(currentTime);
-		config.saveAll();
-	}
-
 	public void reloadConfiguration(CommandSender sender) {
 		plugin.reloadConfig();
 		sender.sendMessage(ChatColor.YELLOW + "Reloaded configuration!");

ProtocolLib/src/main/java/com/comphenix/protocol/ProtocolConfig.java

@@ -1,4 +1,4 @@
-/*
+/**
  *  ProtocolLib - Bukkit server library that allows access to the Minecraft protocol.
  *  Copyright (C) 2012 Kristian S. Stangeland
  *
@@ -14,11 +14,10 @@
  *  if not, write to the Free Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
  *  02111-1307 USA
  */
-
 package com.comphenix.protocol;
 
 import java.io.File;
-import java.io.IOException;
+import java.util.ArrayList;
 import java.util.List;
 
 import org.bukkit.configuration.Configuration;
@@ -26,10 +25,8 @@ import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.plugin.Plugin;
 
 import com.comphenix.protocol.injector.PacketFilterManager.PlayerInjectHooks;
-import com.google.common.base.Charsets;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Lists;
-import com.google.common.io.Files;
 
 /**
  * Represents the configuration of ProtocolLib.
@@ -37,10 +34,7 @@ import com.google.common.io.Files;
  * @author Kristian
  */
 public class ProtocolConfig {
-	private static final String LAST_UPDATE_FILE = "lastupdate";
-	
 	private static final String SECTION_GLOBAL = "global";
-	private static final String SECTION_AUTOUPDATER = "auto updater";
 
 	private static final String METRICS_ENABLED = "metrics";
 
@@ -54,24 +48,11 @@ public class ProtocolConfig {
 	private static final String SCRIPT_ENGINE_NAME = "script engine";
 	private static final String SUPPRESSED_REPORTS = "suppressed reports";
 
-	private static final String UPDATER_NOTIFY = "notify";
-	private static final String UPDATER_DOWNLAD = "download";
-	private static final String UPDATER_DELAY = "delay";
-	
-	// Defaults
-	private static final long DEFAULT_UPDATER_DELAY = 43200;
-	
 	private Plugin plugin;
 	private Configuration config;
-	private boolean loadingSections;
-	
 	private ConfigurationSection global;
-	private ConfigurationSection updater;
 
-	// Last update time
-	private long lastUpdateTime;
 	private boolean configChanged;
-	private boolean valuesChanged;
 
 	// Modifications
 	private int modCount;
@@ -87,102 +68,32 @@ public class ProtocolConfig {
 	public void reloadConfig() {
 		// Reset
 		configChanged = false;
-		valuesChanged = false;
 		modCount++;
 
 		this.config = plugin.getConfig();
-		this.lastUpdateTime = loadLastUpdate();
-		loadSections(!loadingSections);
-	}
-	
-	/**
-	 * Load the last update time stamp from the file system.
-	 * @return Last update time stamp.
-	 */
-	private long loadLastUpdate() {
-		File dataFile = getLastUpdateFile();
-		
-		if (dataFile.exists()) {
-			try {
-				return Long.parseLong(Files.toString(dataFile, Charsets.UTF_8));
-			} catch (NumberFormatException e) {
-				plugin.getLogger().warning("Cannot parse " + dataFile + " as a number.");
-			} catch (IOException e) {
-				plugin.getLogger().warning("Cannot read " + dataFile);
-			}
-		}
-		// Default last update
-		return 0;
-	}
-	
-	/**
-	 * Store the given time stamp.
-	 * @param value - time stamp to store.
-	 */
-	private void saveLastUpdate(long value) {
-		File dataFile = getLastUpdateFile();
-	
-		// The data folder must exist
-		dataFile.getParentFile().mkdirs();
-		
-		if (dataFile.exists())
-			dataFile.delete();
-		
-		try {
-			Files.write(Long.toString(value), dataFile, Charsets.UTF_8);
-		} catch (IOException e) {
-			throw new RuntimeException("Cannot write " + dataFile, e);
-		}
-	}
-	
-	/**
-	 * Retrieve the file that is used to store the update time stamp.
-	 * @return File storing the update time stamp.
-	 */
-	private File getLastUpdateFile() {
-		return new File(plugin.getDataFolder(), LAST_UPDATE_FILE);
-	}
-	
-	/**
-	 * Load data sections.
-	 * @param copyDefaults - whether or not to copy configuration defaults.
-	 */
-	private void loadSections(boolean copyDefaults) {
 		if (config != null) {
+			config.options().copyDefaults(true);
 			global = config.getConfigurationSection(SECTION_GLOBAL);
 		}
-		if (global != null) {
-			updater = global.getConfigurationSection(SECTION_AUTOUPDATER);
-		}
-
-		// Automatically copy defaults
-		if (copyDefaults && (!getFile().exists() || global == null || updater == null)) {
-			loadingSections = true;
-			
-			if (config != null)
-				config.options().copyDefaults(true);
-			plugin.saveDefaultConfig();
-			plugin.reloadConfig();
-			loadingSections = false;
-			
-			// Inform the user
-			ProtocolLibrary.log("Created default configuration.");
-		}
 	}
 
-	/**
-	 * Set a particular configuration key value pair.
-	 * @param section - the configuration root.
-	 * @param path - the path to the key.
-	 * @param value - the value to set.
-	 */
 	private void setConfig(ConfigurationSection section, String path, Object value) {
 		configChanged = true;
 		section.set(path, value);
 	}
 
+	@SuppressWarnings("unchecked")
+	private <T> T getGlobalValue(String name, T def) {
+		try {
+			return (T) global.get(name, def);
+		} catch (Throwable ex) {
+			return def;
+		}
+	}
+
 	/**
 	 * Retrieve a reference to the configuration file.
+	 * 
 	 * @return Configuration file on disk.
 	 */
 	public File getFile() {
@@ -191,66 +102,36 @@ public class ProtocolConfig {
 
 	/**
 	 * Determine if detailed error reporting is enabled. Default FALSE.
+	 * 
 	 * @return TRUE if it is enabled, FALSE otherwise.
 	 */
 	public boolean isDetailedErrorReporting() {
-		return global.getBoolean(DETAILED_ERROR, false);
+		return getGlobalValue(DETAILED_ERROR, false);
 	}
 
 	/**
 	 * Set whether or not detailed error reporting is enabled.
+	 * 
 	 * @param value - TRUE if it is enabled, FALSE otherwise.
 	 */
 	public void setDetailedErrorReporting(boolean value) {
 		global.set(DETAILED_ERROR, value);
 	}
 
-	/**
-	 * Retrieve whether or not ProtocolLib should determine if a new version has been released.
-	 * @return TRUE if it should do this automatically, FALSE otherwise.
-	 */
-	public boolean isAutoNotify() {
-		return updater.getBoolean(UPDATER_NOTIFY, true);
-	}
-	
-	/**
-	 * Set whether or not ProtocolLib should determine if a new version has been released.
-	 * @param value - TRUE to do this automatically, FALSE otherwise.
-	 */
-	public void setAutoNotify(boolean value) {
-		setConfig(updater, UPDATER_NOTIFY, value);
-		modCount++;
-	}
-	
-	/**
-	 * Retrieve whether or not ProtocolLib should automatically download the new version.
-	 * @return TRUE if it should, FALSE otherwise.
-	 */
-	public boolean isAutoDownload() {
-		return updater != null && updater.getBoolean(UPDATER_DOWNLAD, true);
-	}
-	
-	/**
-	 * Set whether or not ProtocolLib should automatically download the new version.
-	 * @param value - TRUE if it should. FALSE otherwise.
-	 */
-	public void setAutoDownload(boolean value) {
-		setConfig(updater, UPDATER_DOWNLAD, value);
-		modCount++;
-	}
-	
 	/**
 	 * Determine whether or not debug mode is enabled.
 	 * <p>
 	 * This grants access to the filter command.
+	 * 
 	 * @return TRUE if it is, FALSE otherwise.
 	 */
 	public boolean isDebug() {
-		return global.getBoolean(DEBUG_MODE_ENABLED, false);
+		return getGlobalValue(DEBUG_MODE_ENABLED, false);
 	}
 
 	/**
 	 * Set whether or not debug mode is enabled.
+	 * 
 	 * @param value - TRUE if it is enabled, FALSE otherwise.
 	 */
 	public void setDebug(boolean value) {
@@ -260,14 +141,16 @@ public class ProtocolConfig {
 
 	/**
 	 * Retrieve an immutable list of every suppressed report type.
+	 * 
 	 * @return Every suppressed report type.
 	 */
 	public ImmutableList<String> getSuppressedReports() {
-		return ImmutableList.copyOf(global.getStringList(SUPPRESSED_REPORTS));
+		return ImmutableList.copyOf(getGlobalValue(SUPPRESSED_REPORTS, new ArrayList<String>()));
 	}
 
 	/**
 	 * Set the list of suppressed report types,
+	 * 
 	 * @param reports - suppressed report types.
 	 */
 	public void setSuppressedReports(List<String> reports) {
@@ -275,42 +158,19 @@ public class ProtocolConfig {
 		modCount++;
 	}
 
-	/**
-	 * Retrieve the amount of time to wait until checking for a new update.
-	 * @return The amount of time to wait.
-	 */
-	public long getAutoDelay() {
-		// Note that the delay must be greater than 59 seconds
-		return Math.max(updater.getInt(UPDATER_DELAY, 0), DEFAULT_UPDATER_DELAY);
-	}
-	
-	/**
-	 * Set the amount of time to wait until checking for a new update.
-	 * <p>
-	 * This time must be greater than 59 seconds.
-	 * @param delaySeconds - the amount of time to wait.
-	 */
-	public void setAutoDelay(long delaySeconds) {
-		// Silently fix the delay
-		if (delaySeconds < DEFAULT_UPDATER_DELAY)
-			delaySeconds = DEFAULT_UPDATER_DELAY;
-		setConfig(updater, UPDATER_DELAY, delaySeconds);
-		modCount++;
-	}
-	
 	/**
 	 * The version of Minecraft to ignore the built-in safety feature.
+	 * 
 	 * @return The version to ignore ProtocolLib's satefy.
 	 */
 	public String getIgnoreVersionCheck() {
-		return global.getString(IGNORE_VERSION_CHECK, "");
+		return getGlobalValue(IGNORE_VERSION_CHECK, "");
 	}
 
 	/**
 	 * Sets under which version of Minecraft the version safety feature will be ignored.
 	 * <p>
-	 * This is useful if a server operator has tested and verified that a version of ProtocolLib works,
+	 * This is useful if a server operator has tested and verified that a version of ProtocolLib works, but doesn't want or can't upgrade to a newer version.
-	 * but doesn't want or can't upgrade to a newer version.
 	 * 
 	 * @param ignoreVersion - the version of Minecraft where the satefy will be disabled.
 	 */
@@ -321,10 +181,11 @@ public class ProtocolConfig {
 
 	/**
 	 * Retrieve whether or not metrics is enabled.
+	 * 
 	 * @return TRUE if metrics is enabled, FALSE otherwise.
 	 */
 	public boolean isMetricsEnabled() {
-		return global.getBoolean(METRICS_ENABLED, true);
+		return getGlobalValue(METRICS_ENABLED, true);
 	}
 
 	/**
@@ -341,16 +202,18 @@ public class ProtocolConfig {
 
 	/**
 	 * Retrieve whether or not the background compiler for structure modifiers is enabled or not.
+	 * 
 	 * @return TRUE if it is enabled, FALSE otherwise.
 	 */
 	public boolean isBackgroundCompilerEnabled() {
-		return global.getBoolean(BACKGROUND_COMPILER_ENABLED, true);
+		return getGlobalValue(BACKGROUND_COMPILER_ENABLED, true);
 	}
 
 	/**
 	 * Set whether or not the background compiler for structure modifiers is enabled or not.
 	 * <p>
 	 * This setting will take effect next time ProtocolLib is started.
+	 * 
 	 * @param enabled - TRUE if is enabled/running, FALSE otherwise.
 	 */
 	public void setBackgroundCompilerEnabled(boolean enabled) {
@@ -358,28 +221,9 @@ public class ProtocolConfig {
 		modCount++;
 	}
 
-	/**
-	 * Retrieve the last time we updated, in seconds since 1970.01.01 00:00.
-	 * @return Last update time.
-	 */
-	public long getAutoLastTime() {
-		return lastUpdateTime;
-	}
-
-	/**
-	 * Set the last time we updated, in seconds since 1970.01.01 00:00.
-	 * <p>
-	 * Note that this is not considered to modify the configuration, so the modification count
-	 * will not be incremented.
-	 * @param lastTimeSeconds - new last update time.
-	 */
-	public void setAutoLastTime(long lastTimeSeconds) {
-		this.valuesChanged = true;
-		this.lastUpdateTime = lastTimeSeconds;
-	}
-	
 	/**
 	 * Retrieve the unique name of the script engine to use for filtering.
+	 * 
 	 * @return Script engine to use.
 	 */
 	public String getScriptEngineName() {
@@ -390,6 +234,7 @@ public class ProtocolConfig {
 	 * Set the unique name of the script engine to use for filtering.
 	 * <p>
 	 * This setting will take effect next time ProtocolLib is started.
+	 * 
 	 * @param name - name of the script engine to use.
 	 */
 	public void setScriptEngineName(String name) {
@@ -399,6 +244,7 @@ public class ProtocolConfig {
 
 	/**
 	 * Retrieve the default injection method.
+	 * 
 	 * @return Default method.
 	 */
 	public PlayerInjectHooks getDefaultMethod() {
@@ -407,6 +253,7 @@ public class ProtocolConfig {
 
 	/**
 	 * Retrieve the injection method that has been set in the configuration, or use a default value.
+	 * 
 	 * @return Injection method to use.
 	 * @throws IllegalArgumentException If the configuration option is malformed.
 	 */
@@ -423,6 +270,7 @@ public class ProtocolConfig {
 
 	/**
 	 * Set the starting injection method to use.
+	 * 
 	 * @param hook Injection method
 	 */
 	public void setInjectionMethod(PlayerInjectHooks hook) {
@@ -432,6 +280,7 @@ public class ProtocolConfig {
 
 	/**
 	 * Retrieve the number of modifications made to this configuration.
+	 * 
 	 * @return The number of modifications.
 	 */
 	public int getModificationCount() {
@@ -442,13 +291,10 @@ public class ProtocolConfig {
 	 * Save the current configuration file.
 	 */
 	public void saveAll() {
-		if (valuesChanged)
-			saveLastUpdate(lastUpdateTime);
 		if (configChanged)
 			plugin.saveConfig();
 
 		// And we're done
-		valuesChanged = false;
 		configChanged = false;
 	}
 }

ProtocolLib/src/main/java/com/comphenix/protocol/ProtocolLibrary.java

@@ -169,6 +169,10 @@ public class ProtocolLibrary extends JavaPlugin {
 		DetailedErrorReporter detailedReporter = new DetailedErrorReporter(this);
 		reporter = getFilteredReporter(detailedReporter);
 
+		// Configuration
+		saveDefaultConfig();
+		reloadConfig();
+
 		try {
 			config = new ProtocolConfig(this);
 		} catch (Exception e) {
@@ -248,7 +252,7 @@ public class ProtocolLibrary extends JavaPlugin {
 			try {
 				switch (command) {
 				case PROTOCOL:
-					commandProtocol = new CommandProtocol(reporter, this, config);
+					commandProtocol = new CommandProtocol(reporter, this);
 					break;
 				case FILTER:
 					commandFilter = new CommandFilter(reporter, this, config);

ProtocolLib/src/main/java/com/comphenix/protocol/injector/LoginPackets.java

@@ -38,18 +38,19 @@ class LoginPackets {
 		}
 		serverSide.add(Packets.Server.KICK_DISCONNECT);
 		
-		// MCPC++ contains Forge, which uses packet 250 during login
+		// MCPC+/Cauldron contains Forge, which uses packet 250 during login
-		if (isMCPC()) {
+		if (isCauldronOrMCPC()) {
 			clientSide.add(Packets.Client.CUSTOM_PAYLOAD);
 		}
 	}
 	
 	/**
-	 * Determine if we are runnign MCPC.
+	 * Determine if we are running MCPC or Cauldron.
 	 * @return TRUE if we are, FALSE otherwise.
 	 */
-	private static boolean isMCPC() {
+	private static boolean isCauldronOrMCPC() {
-		return Bukkit.getServer().getVersion().contains("MCPC-Plus");
+		String version = Bukkit.getServer().getVersion();
+		return version.contains("MCPC") || version.contains("Cauldron");
 	}
 	
 	/**

ProtocolLib/src/main/java/com/comphenix/protocol/injector/PacketFilterManager.java

@@ -1070,7 +1070,7 @@ public final class PacketFilterManager implements ProtocolManager, ListenerInvok
 			playerInjection.uninjectPlayer(event.getPlayer().getAddress());
 			playerInjection.injectPlayer(event.getPlayer(), ConflictStrategy.OVERRIDE);
 		} catch (ServerHandlerNull e) {
-			// Caused by logged out players, or fake login events in MCPC++. Ignore it.
+			// Caused by logged out players, or fake login events in MCPC+/Cauldron. Ignore it.
 		} catch (Exception e) {
 			reporter.reportDetailed(PacketFilterManager.this,
 					Report.newBuilder(REPORT_CANNOT_INJECT_PLAYER).callerParam(event).error(e)

ProtocolLib/src/main/java/com/comphenix/protocol/injector/packet/ProxyPacketInjector.java

@@ -334,7 +334,7 @@ class ProxyPacketInjector implements PacketInjector {
 				return packetRecieved(packet, client, buffered);
 			} else {
 				// The timeout elapsed!
-				reporter.reportWarning(this, Report.newBuilder(REPORT_UNKNOWN_ORIGIN_FOR_PACKET).messageParam(input, packet.getType()));
+				reporter.reportDetailed(this, Report.newBuilder(REPORT_UNKNOWN_ORIGIN_FOR_PACKET).messageParam(input, packet.getType()));
 				return null;
 			}
 			

ProtocolLib/src/main/java/com/comphenix/protocol/injector/player/PlayerInjector.java

@@ -743,7 +743,7 @@ public abstract class PlayerInjector implements SocketInjector {
 	/**
 	 * Indicates that a player's NetServerHandler or PlayerConnection was NULL.
 	 * <p>
-	 * This is usually because the player has just logged out, or due to it being a "fake" player in MCPC++.
+	 * This is usually because the player has just logged out, or due to it being a "fake" player in MCPC+/Cauldron.
 	 * @author Kristian
 	 */
 	public static class ServerHandlerNull extends IllegalAccessError {

ProtocolLib/src/main/java/com/comphenix/protocol/utility/MinecraftReflection.java

@@ -74,9 +74,9 @@ import com.google.common.collect.Maps;
  * @author Kristian
  */
 public class MinecraftReflection {
-	public static final ReportType REPORT_CANNOT_FIND_MCPC_REMAPPER = new ReportType("Cannot find MCPC remapper.");
+	public static final ReportType REPORT_CANNOT_FIND_MCPC_REMAPPER = new ReportType("Cannot find MCPC/Cauldron remapper.");
-	public static final ReportType REPORT_CANNOT_LOAD_CPC_REMAPPER = new ReportType("Unable to load MCPC remapper.");
+	public static final ReportType REPORT_CANNOT_LOAD_CPC_REMAPPER = new ReportType("Unable to load MCPC/Cauldron remapper.");
-	public static final ReportType REPORT_NON_CRAFTBUKKIT_LIBRARY_PACKAGE = new ReportType("Cannot find standard Minecraft library location. Assuming MCPC.");
+	public static final ReportType REPORT_NON_CRAFTBUKKIT_LIBRARY_PACKAGE = new ReportType("Cannot find standard Minecraft library location. Assuming MCPC/Cauldron.");
 
 	/**
 	 * Regular expression that matches a canonical Java class.

ProtocolLib/src/main/java/com/comphenix/protocol/utility/RemappedClassSource.java

@@ -24,6 +24,7 @@ package com.comphenix.protocol.utility;
 import java.lang.reflect.Method;
 
 import org.bukkit.Bukkit;
+import org.bukkit.Server;
 
 import com.comphenix.protocol.reflect.FieldUtils;
 import com.comphenix.protocol.reflect.MethodUtils;
@@ -56,11 +57,17 @@ class RemappedClassSource extends ClassSource {
 	 */
 	public RemappedClassSource initialize() {
 		try {
-			if (Bukkit.getServer() == null || !Bukkit.getServer().getVersion().contains("MCPC-Plus")) {
+			Server server = Bukkit.getServer();
+			if (server == null) {
+				throw new IllegalStateException("Bukkit not initialized.");
+			}
+
+			String version = server.getVersion();
+			if (!server.getVersion().contains("MCPC-Plus") && !version.contains("Cauldron")) {
 				throw new RemapperUnavaibleException(Reason.MCPC_NOT_PRESENT);
 			}
 
-			// Obtain the Class remapper used by MCPC+
+			// Obtain the Class remapper used by MCPC+/Cauldron
 			this.classRemapper = FieldUtils.readField(getClass().getClassLoader(), "remapper", true);
 			
 			if (this.classRemapper == null) {
@@ -112,8 +119,8 @@ class RemappedClassSource extends ClassSource {
 		private static final long serialVersionUID = 1L;
 
 		public enum Reason {
-			MCPC_NOT_PRESENT("The server is not running MCPC+"),
+			MCPC_NOT_PRESENT("The server is not running MCPC+/Cauldron"),
-			REMAPPER_DISABLED("Running an MCPC+ server but the remapper is unavailable. Please turn it on!");
+			REMAPPER_DISABLED("Running an MCPC+/Cauldron server but the remapper is unavailable. Please turn it on!");
 			
 			private final String message;
 			

ProtocolLib/src/main/resources/config.yml

@@ -1,12 +1,4 @@
 global:
-  # Settings for the automatic version updater
-  auto updater:
-    notify: true
-    download: false
-    
-    # Number of seconds to wait until a new update is downloaded
-    delay: 43200 # 12 hours
- 
   metrics: true
   
   # Automatically compile structure modifiers