Canceling a Repeating Task Not Working

Here's my code:

Code:java

//Running the smelt timer
                            int taskid = Bukkit.getServer().getScheduler().scheduleSyncRepeatingTask(plugin, new Runnable(){
 
                                public void run() {
 
                                    if(player.getInventory().contains(15)){
                                        int randomNum = 0 + (int)(Math.random()*((1 - 0) + 1));
                                        if(randomNum == 1){
                                            int taken = 0; //The amount that has been taken from the player's inventory
                                            Material mat = Material.getMaterial(15); //id is the item id gotten from the config
                                            int needed = 1;//needed was also gotten from the config and it is the amount that I need to take from the player
                                            itemRemover(player, taken, mat, needed);
 
                                            playersSmithing.remove(players);
 
                                            int empty = playerinv.firstEmpty();
                                            try {
                                                smithExp.put(265, (int) 12.5); //iron
 
                                                Statement s = c.createStatement();
                                                s.executeUpdate("UPDATE MMOSkills_smithing "
                                                        + "SET EXP=EXP+"+smithExp.get(clickeditem)+" WHERE Player='"+players+"';");
 
 
                                                levelChecker(players, player);
                                            } catch (SQLException e) {
                                                // TODO Auto-generated catch block
                                                e.printStackTrace();
                                            }
 
                                            player.sendMessage(ChatColor.GOLD+"You successfully smelt a bar.");
                                            ItemCreator(empty, smelteditem, player, clickeditem, clickeditemdata);
                                        }else if(randomNum == 0){
                                            int taken = 0; //The amount that has been taken from the player's inventory
                                            Material mat = Material.getMaterial(15); //id is the item id gotten from the config
                                            int needed = 1;//needed was also gotten from the config and it is the amount that I need to take from the player
                                            itemRemover(player, taken, mat, needed);
 
                                            playersSmithing.remove(players);
                                            player.sendMessage(ChatColor.GOLD+"You fail to smelt a bar.");
                                        }
                                    }else{
                                        player.sendMessage(ChatColor.RED+"You run out of ore to smelt and stop smelting.");
                                        Bukkit.getScheduler().cancelTask(playersSmithing.get(players));
                                        playersSmithing.remove(players);
                                    }
 
                                }
 
                            }, 1L, 40L);
                            playersSmithing.put(players, taskid);
 

Line 44 is where I attempt to cancel the task.

It's funny because I use this same exact method for delayed tasks to cancel and it works (in a different context).

What exactly am I doing wrong for this not to work for a repeating task?

 

Still having problems with it and still can't figure it out.

 

Shad

PHP:

Bukkit.getServer().getScheduler().cancelTask(yourbukkittask.getTaskId());

 

How am I getting the Bukkit task name?

Also why isn't just getting the taskid via the int taskid = blahblah not working?

 

Shad
Set your task to a variable of BukkitTask, then cancel it using the code I posted.

 

Henzz
Even I didn't get that.

 

Shad
You're trying to cancel the playersSmithing.get(players), change it to taskId

 

BukkitTask task = Bukkit,getServer()...ect?

I tried that it wasn't working. I guess I'm just really stupid at this.

The hashmap should be able to cancel the task because the key players should bring the taskid, I do it with a delayed task and it works.

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