Scheduler Issue

Solved: Just needed to use .isQueued instead of .isCurrentlyRunning when checking the scheduler.

Show Spoiler Hide Spoiler

Well, I have some weird scheduler issue which I do not understand. Anyway, the issue:
I have two plugins which do something similar to the following: Whenever a player quits, it schedules a task. It puts them into a HashMap too. If the player connects before the task executes, it cancels it and removes them from the HashMap.
The above is not working. Here is the code:

PlayerQuitEvent (open) PlayerQuitEvent (close)

Code:java

@EventHandler
public void onPlayerQuit(final PlayerQuitEvent e)
{
Player player = e.getPlayer();
String playerName = player.getName();
 
int task = Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable()
{
public void run()
{ // CODE }
 
}, 30*20);
clearLag.put(playerName, task);
}

PlayerJoinEvent (open) PlayerJoinEvent (close)

Code:java

if(clearLag.containsKey(playerName))
{
System.out.println("IS RUNNING!"); // <-- Just some debug
int task = clearLag.get(playerName);
if(Bukkit.getServer().getScheduler().isCurrentlyRunning(task))
{
Bukkit.getServer().getScheduler().cancelTask(task);
}
}

From a bunch of debugging, I found out that whenever the player joins, it states that the player is not in the HashMap, but under Quit, as another debug message, I made it get the task ID, which returned fine.

Is this a bukkit bug, or is this something I missed by mistake?

 

clearlag.put should be up one line?

 

Why you say that?

O.K. I put my plugin onto my main server, and discovered that out of the 50th connection, my code worked.

I believe this is a bukkit bug.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

What's it doing where it is?

 

When the player disconnects, it puts them into a hashmap with a scheduler. When the player connects before the scheduler ends, it cancels the scheduler (and removes the player from the hashmap).

 

Still no clue what you want, but I think you want:

Code:

int task = Bukkit.getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable() {
  public void run() {
    clearLag.put(playerName, task);
  }
}, 30*20);

 

No! O.K. read carefully! When a player disconnects, it puts they're name and the task into a hashmap. Key = PlayerName, Value = Task.
If a player connects before the scheduler starts the code (inside run()), I want it to cancel the task!

It can't get any more simpler then that.

 

All I did was fix the code you provided in the OP..

 

Then report it so it can get fixed.

 

You didn't fix it because that is not what I'm aiming for.

I'm not sure if it is an issue. I created a thread because I thought something in my code was wrong. If others are having this issue, then I will report it.

 

Then why did you post it?

 

I think I might of figured out the problem. Instead of using isCurrentlyRunning, I should use isQueued. Ima test it out.

EDIT: Yup. Works now!

EDIT by Moderator: merged posts, please use the edit button instead of double posting.