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
Implement Saddle & Horse Armor #6307
base: minor-next
Are you sure you want to change the base?
Conversation
Is it enough if it stays as it is now (as I said, there are no mobs) or should I add the Horse Armor to Living::attack() anyway? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to regenerate the static functions
for whatever reason i can't run the file on my laptop. can do that when i am back at the pc 👌 |
As the PR name and objective is to add these items and not the horse logics, its enough. |
Cauldron behaviour is also missing. In vanilla, leather armour can be dyed in cauldron. |
Jup but this is currently not implemented in pocketmine and I don´t feel like it is something for this pr.. |
What do you mean? Cauldron is implemented |
Introduction
Implement Saddle & Horse Armor.
(Currently this is just for cosmetic because there are no mobs in pocketmine but I need this for my mob plugin :P)
Changes
API changes
VanillaItems::SADDLE()
VanillaItems::LEATHER_HORSE_ARMOR()
VanillaItems::IRON_HORSE_ARMOR()
VanillaItems::DIAMOND_HORSE_ARMOR()
VanillaItems::GOLDEN_HORSE_ARMOR()
Tested LEATHER_HORSE_ARMOR custom colour with following code:
Tests
I tested this PR by doing the following (tick all that apply):