Solved ItemStacks Durability not changing, it only changes once.

Hello bukkit!
I have tried many ways, to make an item go from almost 1 durability then pretty much go up to full. Even though it only changes once!
I know the Runnable works, I've tested with debugs.

Code:

ItemStack mgm = e.getItem();
                        Bukkit.getScheduler().scheduleSyncDelayedTask(this.plugin, new Runnable(){
                            @Override
                            public void run() {
                               
                                Bukkit.getScheduler().scheduleSyncDelayedTask(InteractEvent.plugin, new Runnable(){

                                    @Override
                                    public void run() {
                                        mgm.setDurability( (short) 29);
                                        e.getPlayer().updateInventory();
                                        Bukkit.getScheduler().scheduleSyncDelayedTask(InteractEvent.plugin, new Runnable(){

                                            @Override
                                            public void run() {
                                                mgm.setDurability( (short) 24);
                                                e.getPlayer().updateInventory();
                                                Bukkit.getScheduler().scheduleSyncDelayedTask(InteractEvent.plugin, new Runnable(){

                                                    @Override
                                                    public void run() {
                                                        mgm.setDurability((short) 17);
                                                        e.getPlayer().updateInventory();
                                                        Bukkit.getScheduler().scheduleSyncDelayedTask(InteractEvent.plugin, new Runnable(){

                                                            @Override
                                                            public void run() {
                                                                mgm.setDurability((short) 12);
                                                                e.getPlayer().updateInventory();
                                                                Bukkit.getScheduler().scheduleSyncDelayedTask(InteractEvent.plugin, new Runnable(){

                                                                    @Override
                                                                    public void run() {
                                                                        mgm.setDurability((short) 4);
                                                                        e.getPlayer().updateInventory();
                                                                        Bukkit.getScheduler().scheduleSyncDelayedTask(InteractEvent.plugin, new Runnable(){

                                                                            @Override
                                                                            public void run() {
                                                                                mgm.setDurability((short) 0);
                                                                                e.getPlayer().updateInventory();
                                                                            }
                                                                           
                                                                        }, 4);
                                                                    }
                                                                   
                                                                }, 4);
                                                            }
                                                           
                                                        }, 4);
                                                    }
                                                   
                                                }, 4);
                                            }
                                           
                                        }, 4);
                                    }
                                   
                                }, 4);
                            }
                           
                        }, 4);

Thanks,
me.

 

why so many shedulers!? instead use one repeating, and every so decrease the durability by X when it reaches 0 stop the task

 

Your code has a huuuuge indent in it.

 

Yeah.. good idea
@teej107 does that affect the code?

 

@ChipDev It affects readability.

 

Thats why you can scroll.

 

That's not the problem, It will just hurt our eyes if we try to read it.
Edit: You're executing the code inside the run() methods after 4 ticks, 20 ticks = 1 second

 

Ok, Yes.. My problem is that my item seems to only be able to be changed once or outside of the scheduler?

 

@ChipDev
Explain? What do you mean by "changed once"

I didn't test your code or anything, I just kindof assumed that was the problem since that would be executing the run methods with very little delay.

Edit: Also, you should definitely check out what @mine-care said.

 

Did that!

Code:

ItemStack mgm = e.getItem();
                        e.getItem().setDurability((short) 31);
                        Bukkit.getScheduler().scheduleSyncRepeatingTask(this.plugin, new Runnable() {
                            @Override
                            public void run() {
                                    e.getItem().setDurability((short) 5);
                            }
                           
                        }, 1, 3);

It only sets the items durability to 31 (Lowest before it breaks , for my golden hoe) and it doesn't regenerate health. It just stays there.

 

I guess I'll check it out on a testing server, But if that is the code you're using it'll just stay at 5.

@ChipDev
Right, I edited the code a little bit and it seemed to work for me?

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

 

Can you show me your code? Usually this works for me, I must be making a moony mistake!

 

@ChipDev
Well, people are going to hate me for this but why not:

Code:php

        final ItemStack mgm = e.getPlayer().getItemInHand();
        mgm.setDurability((short) mgm.getType().getMaxDurability());
        new BukkitRunnable() {
            short next = mgm.getDurability();
 
            @Override
            public void run() {
                next -= 5;
                mgm.setDurability(next);
                if(e.getPlayer() != null) e.getPlayer().updateInventory();
                if(mgm.getDurability() <= 0) this.cancel();
 
            }
        }.runTaskTimer(main, 0L, 3L);

 

That still keeps the item's durability at 0, nothing changes :/
EDIT: It gives me 5 debug messages, because it is increasing by 5 .. And the max durability is 32. (It shows it as 1 durability, because I changed your code to show item.getMaxDurability() - 1)
Does this mean my client is not rendering this correctly?

 

@ChipDev
Post your current code? that worked for me

 

I actually got it working by giving the player a new itemstack. Thanks