-
Notifications
You must be signed in to change notification settings - Fork 222
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
Remove /invis smite, add /invis clear #1011
Conversation
Resolves TotalFreedom#959
Again, as stated on the issue I disagree with removing TFM functionality for the sake of it. |
@@ -11,19 +11,19 @@ | |||
import org.bukkit.potion.PotionEffectType; | |||
|
|||
@CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) | |||
@CommandParameters(description = "Shows (optionally smites) invisisible players", usage = "/<command> (smite)") | |||
@CommandParameters(description = "Shows (optionally clears) invisisible players", usage = "/<command> (clear)") |
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.
This should be an optional parameter. Usage is [parameter]
@LegendIsAwesomes One small change and this PR is good to go. |
There we go @JeromSar |
@LegendIsAwesomes Thanks for contributing! This PR is ready to merge and will be soon. :) |
It's been accepted, invis smite just smites innocents |
@DUFCLiam The implementation is fine as-is. For reasoning behind removing /invis smite, see the original issue. |
{ | ||
playerMsg("Smitten " + smites + " players"); | ||
playerMsg("Cleared invisibility effect from " + clears + " players""); |
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 added 2 "'s at the end.
Fixed. @JeromSar |
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.
Should be ready to merge.
Merged in 0eb0c7a. Thanks for contributing! |
Resolves #959