Solved Potion to ItemStack - Short durability incorrect?

I have made an instance of the Potion class with my custom item, but it is giving the ItemStack the wrong durability value:

Current Code (open) Current Code (close)

Code:java

    @SuppressWarnings("deprecation")
    private ItemStack stringToPotion(String string) {
 
        Potion potion = null;
        string = string.toLowerCase();
 
        if (string.startsWith("swp")) {
            potion = new Potion(PotionType.SPEED);
        } else {
            return null;
        }
 
        if ((!(string.endsWith("s"))) && (!(string.endsWith("1"))) && (!(string.endsWith("1e"))) && (!(string.endsWith("2")))) {
            return null;
        }
 
        if (string.endsWith("s")) {
            string = string.replace("s", "");
            potion.setSplash(true);
        }
 
        if (string.endsWith("1")) {
            potion.setLevel(1);
        } else if (string.endsWith("2")) {
            potion.setLevel(2);
        } else if (string.endsWith("1e")) {
            potion.setHasExtendedDuration(true);
        }
 
        System.out.println(potion.toDamageValue() + "<DAMAGEVALUE");
        return potion.toItemStack(1);
 
    }

So for this case if I use the input string of 'swp2' it should return an ItemStack of Speed, with the level modifier of 2, which it does, but with the incorrect durability. Instead of a durability of '8226' as it should be here, it is instead a durability of 34:


If there is an alternative way of doing this I'd like to know, because I think it may be a problem with the Potion class as it works fine for potions that are splashable.

Thanks.

 

elementalgodz11 Why does it matter? You get your potion which is essentially the same - maybe it's just not including the base potion in the DV

 

AdamQpzm

When you obtain a potion the correct way such as brewing it, the data value will be correct to such as a wiki would show,

therefore if I tried to get the ItemStack using the stringToPotion method, it will not recognise it as a potion because the string is not equal to the actual potions data value

 

elementalgodz11
I believe 34 is actually correct, seeing as the first potion that isn't a bottle starts with data value 8192, then increments from there. So, if you can just add 8192 to the data value and you should be fine, although it won't matter at all.

Edit: Yep: Tested in-game: /give AoH_Ruthless 373:34 gives me a swiftness II potion.

 

AoH_Ruthless

Thank you!

Would this work for splashable potions too as I noticed they generate correctly.

If so, what pattern could I attempt to correct this? Check if the durability is less than 1000 or it returns true for the isSplashable boolean before adding 8192?

 

elementalgodz11
Splash potions I don't think work. If you notice, their durability is 16,000+ so I don't know the algorithm. Check if the durability is less than, say, 500 and then add 8192.

 

AoH_Ruthless Sure checking isSplashable or if the value is less than 8192 makes more sense, right?

 

AdamQpzm
Yeah you are actually right. elementalgodz11 (tahged to make sure you see above post)

 

Thank you guys.