Make auto-spliting on item damages optional (tools/weapons/etc.)

[NukuHack]

it's just when u want to equip more than 1 armor it don't let to do , it allways equip only one (but u can equip shields fine)

[NukuHack]

lol tested if u somehow manage to equip more than 1 armor in one slot it unequip them if it got damaged (one will stay "in you" and the others got send back to ur inv) sadly it also work with shield

[NukuHack]

lol if u have something in ur hand (more than 1) and it got dmg (lost durability) it got unstacked so u will have 1 in ur hand and the rest in ur inv

[NukuHack]

ofc if ur inv is full , u will have only one of that thing and the rest will be dropped on the ground

[KrisCris]

lol if u have something in ur hand (more than 1) and it got dmg (lost durability) it got unstacked so u will have 1 in ur hand and the rest in ur inv

ofc if ur inv is full , u will have only one of that thing and the rest will be dropped on the ground

I think that's the intended behavior, I made it do that.

Whereas without my fixes, all damageable items just got the same dmg lol

[KrisCris]

AllStackable/src/main/java/online/connlost/allstackable/mixin/MixinItemStack.java

Lines 33 to 46 in 5c85bee

	@Redirect(method = "damage(ILnet/minecraft/util/math/random/Random;Lnet/minecraft/server/network/ServerPlayerEntity;)Z", at = @At(value = "INVOKE", target = "Lnet/minecraft/item/ItemStack;setDamage(I)V"))
	private void splitStackedTools(ItemStack stack, int damage, int amount, Random random, @Nullable ServerPlayerEntity player){
	ItemStack rest = null;
	if (stack.getCount()>1 && ItemsHelper.isModified(stack) && player!=null){
	rest = stack.copy();
	rest.decrement(1);
	stack.setCount(1);
	}
	stack.setDamage(damage);
	if (rest != null){
	ItemsHelper.insertNewItem(player,rest);
	}
	}
	}

I honestly can't come up with a better solution. So this probably won't be fixed as this is not a bug.

[NukuHack]

in vanilla mc if u obtain stacked armors/tools it's cool bc if u break it u will get all the rest with full durability i mean the durability resets so this is useless :(

[NukuHack]

AllStackable/src/main/java/online/connlost/allstackable/mixin/MixinItemStack.java

Lines 33 to 46 in 5c85bee

	@Redirect(method = "damage(ILnet/minecraft/util/math/random/Random;Lnet/minecraft/server/network/ServerPlayerEntity;)Z", at = @At(value = "INVOKE", target = "Lnet/minecraft/item/ItemStack;setDamage(I)V"))
	private void splitStackedTools(ItemStack stack, int damage, int amount, Random random, @Nullable ServerPlayerEntity player){
	ItemStack rest = null;
	if (stack.getCount()>1 && ItemsHelper.isModified(stack) && player!=null){
	rest = stack.copy();
	rest.decrement(1);
	stack.setCount(1);
	}
	stack.setDamage(damage);
	if (rest != null){
	ItemsHelper.insertNewItem(player,rest);
	}
	}
	}

I honestly can't come up with a better solution. So this probably won't be fixed as this is not a bug.

it wouls be cool if u would make it to togle-able or something like that

[KrisCris]

hmm what do you mean? when you are using a stack of tools/shields/weapons, only one of them will get damaged.

[KrisCris]

by disable that code, not just one, ALL stack of them lose durability

[NukuHack]

by disable that code, not just one, ALL stack of them lose durability

yes ... but if u manage to break that item one will be loosen