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

Small userfriendly enchantments #16

Closed
wants to merge 1 commit into from

Conversation

Arkobat
Copy link

@Arkobat Arkobat commented Oct 20, 2019

Fixed ArrayIndexOutOfBoundsException when using tfly without args in console
Fixed return false in command handler, causing the plugin to write the command to the player.
Added support for writing invalid args when using /tfly. (It just handle the command as no args where used)

Fixed ArrayIndexOutOfBoundsException when using tfly without args in console
Fixed `return false` in command handler, causing the plugin to write the command to the player.
Added support for writing invalid args when using /tfly. (It just handle the command as no args where used)
@LlmDl
Copy link
Member

LlmDl commented Oct 20, 2019

Looks good, but the whitespace rules in your ide are different than mine. If you don't want to adjust it I can do the changes on my end and give you credit in the changelog.

@Arkobat
Copy link
Author

Arkobat commented Oct 20, 2019

I think it's better taht you do it, so it have the correct spacing.
I've done it with IntelliJ auto format.
No credit is needed :D

@LlmDl
Copy link
Member

LlmDl commented Oct 23, 2019

Taken care of with 1.5.2.

@LlmDl LlmDl closed this Oct 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants