Bee and Zombified_Piglin's swap attributes based on their order in VanillaMobs.yml

Summary

When editting a Bee and a Zombified_Piglin in the VanillaMobs.yml They take all attributes you set from the other based on the order they are in inside the VanillaMobs.yml file.

Steps to reproduce

  1. put these mobs in your VanillaMobs.yml https://pastebin.com/mMf0KCQL
  2. If you have Bee above Zombified piglin, and do /summon minecraft:bee youll see it will have 22 hp and the name of a zombified_piglin, while its hp is set to 10 and would scale to 11, and the name is set to bee.
  3. If you then move zombified_piglin above the bee inside the VanillaMobs.yml file and reload you will see it swap. The Bee with have its correct name and hp, but doing /summon minecraft:zombified_piglin will spawn a zombified_piglin with the name and hp of the bee.
  4. Try to summon either of them with /mm m spawn bee or /mm m spawn zombified_piglin throws an error in the console.

Intended correct behavior

The bee should keep its own attributes and the zombified_piglin should keep its own attributes.

Server log file

Latest Log: https://pastebin.com/gCTmyBm5

Error Snippet: https://pastebin.com/z2pLaz04 (when trying to do /mm m spawn bee)

Proposed fixes

Figure out what makes them pull from each other. I think it would always take from the bottom on because they are considered the same entity/mob in the code. Just like if you have 2 sets of Skills: in a mob file, it always takes from the bottom set of Skills:.

Edited Dec 20, 2020 by Akim91
Assignee Loading
Time tracking Loading