Domestication Innovation

Domestication Innovation

2M Downloads

[1.19.2] Collar enchantment duplication glitch

B4CKU opened this issue ยท 4 comments

commented

Mod version: 1.6.1
Minecraft version: 1.19.2
Forge version: 43.2.3

Some collared pets, will keep their collars and their enchantments, when swapped out, while still dropping said collars on the ground.

I'm not sure, but this could be related to the issue #104

Video:

2023-05-28.09-20-53.mp4

List of all used mods:

image

commented

It's because of the enchant called "Healing Aura" and "Blazing Protection"!
the issue#104 hasn't been fixed. :(

commented

Yeah it is still a problem. The only workaround is adding those two enchantments at the end of a maxed collar on a NEW pet.

commented

The fix in #104 only fixes race conditions which are server-side

It always allows updates from the client-side (see canTickCollar) and since onLivingUpdate runs for both sides the client-side will call the sync methods for those (and other enchants) while bypassing the block logic

Not syncing those two from the client-side within onLivingUpdate fixes the issue
It also doesn't seem to break the rendering parts of those enchants (at least for the blazing enchant, I don't know what the other one does)

commented

This is still an issue in the 1.20 version (domesticationinnovation-1.7.0-1.20). The herding enchantment cannot be removed and is causing chaos. Adding a blank collar tag pops the enchanted tag off of the mob, but the enchantments remain. Adding another blank to the mob pops off a fully enchanted tag. The only way to end this is to kill the mob.

Update: It appears I also had the healing aura enchantment on the collar along with herding, so that is probably what caused it to get stuck.