[SPIGOT-4705] Zombie implement Ageable Created: 18/Apr/19 Updated: 13/Aug/20 Resolved: 13/Aug/20 |
|
| Status: | Resolved |
| Project: | Spigot |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | New Feature | Priority: | Minor |
| Reporter: | Jacob Abernathy | Assignee: | Unassigned |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | entity, spigot | ||
| Version: | LATEST |
| Guidelines Read: | Yes |
| Description |
|
Let Zombie implement Ageable. It doesn't make sense why it would have separate setBaby isBaby methods when just having it extend Ageable would suffice. It has 2 states just like a Villager, and Villager extends Ageable. Why wouldn't Zombie? |
| Comments |
| Comment by Shane Bee [ 05/Aug/20 ] |
|
I submitted a PR for this, hopefully it makes everyones dreams come true. |
| Comment by blablubbabc [ 26/Jun/20 ] |
|
+1 for adding a common Babyable interface for everything that provides set/isBaby methods. With 1.16 there seem have been a few more mobs added to which this would apply as well. |
| Comment by Jacob Abernathy [ 25/Apr/19 ] |
|
I understand it's a preference at this point, but for the sake of a consistent API implementation, is there any way we can rework this and provide a separate interface for entities that are 'ageable' and for those that are 'breedable' |
| Comment by Cynthia Yantis [ 25/Apr/19 ] |
|
Not sure how I feel about this: Ageable indicates the ability for the entity to "grow up" per se. I do like the idea of a new interface for it to a degree though. |
| Comment by md_5 [ 20/Apr/19 ] |
|
Babyable
Bit icky (better name maybe...), but it is an option |
| Comment by Jacob Abernathy [ 18/Apr/19 ] |
|
Maybe the interface names should be changed to something more fitting. Technically, Zombies are'ageable' |
| Comment by md_5 [ 18/Apr/19 ] |
|
https://hub.spigotmc.org/javadocs/spigot/org/bukkit/entity/Ageable.html
Because Ageable has far more methods than that. For example Zombies don't breed, but Villagers do. |