Potential compatibility issues with horse/horse-like mods

noobanidus / dwmh

Dude! Where's my Horse?

MIT License

2 stars 1 forks source link

Potential compatibility issues with horse/horse-like mods #14

Closed noobanidus closed 5 years ago

noobanidus commented 5 years ago

Mods to check compatibility:

~~Wings, Horns, Hooves~~ (functions by default as they derive from AbstractHorse)
~~Astikoor (Horse Carts)~~ Moved to its own issue (#17)
~~Horsebones~~ not actually a mod

noobanidus commented 5 years ago

~~'Mo Creatures~~

noobanidus commented 5 years ago

Zawa: ~~Issues would require iterating over EntityTameable and then consulting the data manager (potentially) to determine if there's a saddle on the entity.~~

~~public boolean isOwner(final EntityPlayer player) { return player.getUniqueID().equals(this.getOwnerId()); }~~

~~I believe the ownership code is the same.~~

All of these issues have been resolved by the new compatibility system. It is simply a matter of confirming that it functions properly.

noobanidus commented 5 years ago

~~- Mutated Mobs: The only eligible mod is the spider pig which you can ride by default. I'm loath to add Animania-esque compatibility for something that doesn't even have the potential to be tamed.~~

noobanidus commented 5 years ago

~~- Rideable John Cena: Despite the glorious name, this is an MCreator mod which I will not support.~~

noobanidus commented 5 years ago

All of these have been resolved. All others are being tracked on discord.