[0.3] Fix StackOverflowError exception.

Use the Bukkit scheduler API.

Author: jamierocks

pom.xml

@@ -9,7 +9,7 @@
 	<name>PlayTimeLimiter</name>
 	<description>PlayTimeLimiter is a Bukkit plugin to allow server owners to limit the amount of time their players spend on a server.</description>
 	<url>https://github.com/UnoModding/Bukkit-PlayTimeLimiter</url>
-	<version>0.3-dev</version>
+	<version>0.3</version>
 	<inceptionYear>2014</inceptionYear>
 	<packaging>jar</packaging>
 

src/main/java/unomodding/bukkit/playtimelimiter/PlayTimeLimiter.java

@@ -1,5 +1,5 @@
 /**
- * Copyright 2014 by UnoModding, RyanTheAllmighty and Contributors
+ * Copyright 2014-2015 by UnoModding, RyanTheAllmighty and Contributors
  *
  * This work is licensed under the Creative Commons Attribution-ShareAlike 3.0 Unported License.
  * To view a copy of this license, visit http://creativecommons.org/licenses/by-sa/3.0/.
@@ -14,7 +14,6 @@
 import java.io.IOException;
 import java.util.HashMap;
 import java.util.Map;
-import java.util.Timer;
 import java.util.UUID;
 
 import org.bukkit.ChatColor;
@@ -44,8 +43,6 @@ public class PlayTimeLimiter extends JavaPlugin {
 	private Map<String, Boolean> seenWarningMessages = new HashMap<String, Boolean>();
 
 	private boolean shutdownHookAdded = false;
-	private Timer savePlayTimeTimer = null;
-	private Timer checkPlayTimeTimer = null;
 	private boolean started = false;
 	private final Gson GSON = new Gson();
 
@@ -94,14 +91,14 @@ public void onEnable() {
 			getConfig().set("timeTravels", true);
 			saveConfig();
 		}
-		if (!getConfig().isSet("timeCap")) {
+		/*if (!getConfig().isSet("timeCap")) {
 			getConfig().set("timeCap", true);
 			saveConfig();
 		}
 		if (!getConfig().isSet("timeCapValue")) {
 			getConfig().set("timeCapValue", 18000);
 			saveConfig();
-		}
+		}*/
 
 		getLogger().info(
 				String.format("Server started at %s which was %s seconds ago!",
@@ -118,18 +115,13 @@ public void onEnable() {
 		// Load the playtime from file
 		this.loadPlayTime();
 
-		if (savePlayTimeTimer == null) {
-			this.savePlayTimeTimer = new Timer();
-			this.savePlayTimeTimer.scheduleAtFixedRate(new PlayTimeSaverTask(
-					this), 30000,
-					getConfig().getInt("secondsBetweenPlayTimeSaving") * 1000);
-		}
-		if (checkPlayTimeTimer == null) {
-			this.checkPlayTimeTimer = new Timer();
-			this.checkPlayTimeTimer.scheduleAtFixedRate(
-					new PlayTimeCheckerTask(this), 30000,
-					getConfig().getInt("secondsBetweenPlayTimeChecks") * 1000);
-		}
+		this.getServer().getScheduler().scheduleSyncRepeatingTask(this,
+			    new PlayTimeSaverTask(this), 30000,
+			        getConfig().getInt("secondsBetweenPlayTimeSaving") * 1000);
+		this.getServer().getScheduler().scheduleSyncRepeatingTask(this,
+				new PlayTimeCheckerTask(this), 30000,
+				getConfig().getInt("secondsBetweenPlayTimeChecks") * 1000);
+
 
 		try {
 			Metrics metrics = new Metrics(this);
@@ -232,6 +224,10 @@ public void setPlayerLoggedIn(UUID uuid) {
 				(int) (System.currentTimeMillis() / 1000));
 	}
 
+	public void setPlayerLoggedOut(UUID uuid) {
+		setPlayerLoggedOut(uuid.toString());
+	}
+
 	private void setPlayerLoggedOut(String uuid) {
 		if (this.timeLoggedIn.containsKey(uuid)) {
 			int timePlayed = (int) ((System.currentTimeMillis() / 1000) - this.timeLoggedIn
@@ -260,10 +256,6 @@ private void setPlayerLoggedOut(String uuid) {
 		}
 	}
 
-	public void setPlayerLoggedOut(UUID uuid) {
-		setPlayerLoggedOut(uuid);
-	}
-
 	public boolean hasPlayerSeenMessage(UUID uuid, int time) {
 		if (this.seenWarningMessages.containsKey(uuid + ":" + time)) {
 			return this.seenWarningMessages.get(uuid + ":" + time);

src/main/java/unomodding/bukkit/playtimelimiter/threads/PlayTimeCheckerTask.java

@@ -1,13 +1,12 @@
 /**
- * Copyright 2013-2014 by UnoModding, ATLauncher and Contributors
+ * Copyright 2013-2015 by UnoModding, ATLauncher and Contributors
  *
  * This work is licensed under the Creative Commons Attribution-ShareAlike 3.0 Unported License.
  * To view a copy of this license, visit http://creativecommons.org/licenses/by-sa/3.0/.
  */
 package unomodding.bukkit.playtimelimiter.threads;
 
 import java.io.File;
-import java.util.TimerTask;
 
 import org.bukkit.ChatColor;
 import org.bukkit.entity.Player;
@@ -16,14 +15,13 @@
 import unomodding.bukkit.playtimelimiter.utils.FileUtils;
 import unomodding.bukkit.playtimelimiter.utils.Timestamper;
 
-public class PlayTimeCheckerTask extends TimerTask {
+public class PlayTimeCheckerTask implements Runnable {
 	private final PlayTimeLimiter plugin;
 
 	public PlayTimeCheckerTask(PlayTimeLimiter instance) {
 		this.plugin = instance;
 	}
 
-	@Override
 	public void run() {
 		for (Player player : this.plugin.getServer().getOnlinePlayers()) {
 			int timeLeft = this.plugin.getTimeAllowedInSeconds(player

src/main/java/unomodding/bukkit/playtimelimiter/threads/PlayTimeSaverTask.java

@@ -1,23 +1,20 @@
 /**
- * Copyright 2013-2014 by ATLauncher and Contributors
+ * Copyright 2013-2015 by ATLauncher and Contributors
  *
  * This work is licensed under the Creative Commons Attribution-ShareAlike 3.0 Unported License.
  * To view a copy of this license, visit http://creativecommons.org/licenses/by-sa/3.0/.
  */
 package unomodding.bukkit.playtimelimiter.threads;
 
-import java.util.TimerTask;
-
 import unomodding.bukkit.playtimelimiter.PlayTimeLimiter;
 
-public class PlayTimeSaverTask extends TimerTask {
+public class PlayTimeSaverTask implements Runnable {
 	private final PlayTimeLimiter plugin;
 
 	public PlayTimeSaverTask(PlayTimeLimiter instance) {
 		this.plugin = instance;
 	}
 
-	@Override
 	public void run() {
 		this.plugin.savePlayTime(); // Save playtime every 10 minutes
 	}