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

[Request] Disabling pet riding from CMI #1145

Closed
mercurialmusic opened this issue Aug 5, 2018 · 3 comments
Closed

[Request] Disabling pet riding from CMI #1145

mercurialmusic opened this issue Aug 5, 2018 · 3 comments
Labels

Comments

@mercurialmusic
Copy link

Please provide the following info.

Bukkit/Spigot/Bungee Build/Version:
Spigot-642f6d2-57ab4cf (MC: 1.12.2) (Implementing API version 1.12.2-R0.1-SNAPSHOT
MyPet Build/Version:
MyPet-2.4.0-SNAPSHOT-b#574
Do you use the premium version? (this wont change anything for you)
Yes
What is the expected result?
We have riding pets disabled since we have only wolf, ocelot, and parrot available to all members and the rest of the pet types are "premium". Riding certain pets like bats enables players to fly or hover slightly and endermites allow people to fit through very small spaces (the gaps between stacked chests). We don't want to mess with it so we disable it completely.
What happens instead of that?
A third plugin, CMI, has a /ride command that still allows players to ride their pets. I'm not sure if this should be reported/requested of you or CMI, but it would be useful to have some sort of hook between the two to disallow riding pets.

Strangely, the CMI /ride command does not typically allow the riders to control their mounts except for mobs that are meant to be ridden in vanilla (horses), but when using the CMI /ride command on a MyPet they are able to control the pet.

I know both of these plugins are busy updating to 1.13 right now, so I'm not in a hurry to have this corrected. In the meantime we have limited the CMI /ride command to only a few entity types until we have a solution, but it would be nice to have a full resolution eventually. Thanks for your consideration!

Error:
No console errors.

@scomans
Copy link
Member

scomans commented Aug 6, 2018

Could you provice me with a link to the CMI plugin?

@scomans scomans added the Bug label Aug 6, 2018
@scomans
Copy link
Member

scomans commented Aug 6, 2018

@scomans scomans closed this as completed in eb208e8 Aug 6, 2018
@mercurialmusic
Copy link
Author

That's the one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants