Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Villagers could duplicate when turning into Zombie Villagers #992

Closed
atilapan opened this issue Dec 24, 2024 · 4 comments
Closed

Villagers could duplicate when turning into Zombie Villagers #992

atilapan opened this issue Dec 24, 2024 · 4 comments
Assignees
Labels
✔️ Confirmed Confirmed bugs that can be replicated 🔧 Potential fix A potential fix was already submitted

Comments

@atilapan
Copy link

Minecraft's Version

Current Purpur Version: 1.21.1-2329-803bf62 (MC: 1.21.1)*

  • You are running the latest version
    Previous: 1.21.3-2358-16ce24a (MC: 1.21.3)

Plugin's Version

Current Purpur Version: 1.21.1-2329-803bf62 (MC: 1.21.1)\n You are running the latest version\nPrevious: 1.21.3-2358-16ce24a (MC: 1.21.3)
WildStacker version 2024.4
Stacking drops, entities, blocks and spawners into one object!
Website: https://bg-software.com/
Author: Ome_R

Describe the bug

Villages when attacked by zombies and infected sometimes duplicate. Instead of one Villager turning into one Zombie Villager, there are sometimes two Zombie Villagers so for particular reason. And they both seem to keep everything about the original Villager.

To Reproduce

Install plugin
Switch difficulty to Hard (not sure if Normal is affected)
Spawn a Villager and a Zombie
Wait for the Zombie to kill the Villager
And watch... there is some percent chance instead of one you will get two zombie villagers

Additional Information

No response

@atilapan atilapan added the Pending Pending reports to be investigated label Dec 24, 2024
@OmerBenGera OmerBenGera added ✔️ Confirmed Confirmed bugs that can be replicated and removed Pending Pending reports to be investigated labels Dec 27, 2024
@OmerBenGera OmerBenGera added the 🔧 Potential fix A potential fix was already submitted label Dec 27, 2024
@OmerBenGera
Copy link
Member

Check out latest dev build, this should be fixed

@atilapan
Copy link
Author

What a coincidence I was also checking out the bug and I think I actually found the issue. The zombie sometimes could hit the Villagers a second time which would cause eventually checkForZombieVillager to be called twice. Just a question as this seems to only affect Zombie Villages, couldn't we just despawn the Villager before the Zombie Villager is spawned?

@atilapan
Copy link
Author

But yeah your dev build also seems to resolve the issue with the limited testing I did.

@OmerBenGera
Copy link
Member

What a coincidence I was also checking out the bug and I think I actually found the issue. The zombie sometimes could hit the Villagers a second time which would cause eventually checkForZombieVillager to be called twice. Just a question as this seems to only affect Zombie Villages, couldn't we just despawn the Villager before the Zombie Villager is spawned?

Right, that was the issue and it has been fixed :)
Closing this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✔️ Confirmed Confirmed bugs that can be replicated 🔧 Potential fix A potential fix was already submitted
Projects
None yet
Development

No branches or pull requests

2 participants