Send messages the right way. Fixes #378

This commit is contained in:
Trent Hensler
2018-12-29 15:45:06 -08:00
parent f463dedee6
commit 077745733b
@@ -18,7 +18,9 @@
package com.drtshock.playervaults.commands; package com.drtshock.playervaults.commands;
import com.drtshock.playervaults.PlayerVaults;
import com.drtshock.playervaults.translations.Lang; import com.drtshock.playervaults.translations.Lang;
import com.drtshock.playervaults.vaultmanagement.VaultManager;
import com.drtshock.playervaults.vaultmanagement.VaultOperations; import com.drtshock.playervaults.vaultmanagement.VaultOperations;
import org.bukkit.Bukkit; import org.bukkit.Bukkit;
import org.bukkit.ChatColor; import org.bukkit.ChatColor;
@@ -53,9 +55,14 @@ public class DeleteCommand implements CommandExecutor {
// TODO: fix the stupid message inconsistencies where sometimes this class sends, sometimes vaultops does. // TODO: fix the stupid message inconsistencies where sometimes this class sends, sometimes vaultops does.
if (args[1].equalsIgnoreCase("all")) { if (args[1].equalsIgnoreCase("all")) {
VaultOperations.deleteOtherAllVaults(sender, target); if (sender.hasPermission("playervaults.delete.all")) {
VaultManager.getInstance().deleteAllVaults(target);
sender.sendMessage(Lang.TITLE.toString() + Lang.DELETE_OTHER_VAULT_ALL.toString().replaceAll("%p", target)); sender.sendMessage(Lang.TITLE.toString() + Lang.DELETE_OTHER_VAULT_ALL.toString().replaceAll("%p", target));
break; PlayerVaults.getInstance().getLogger().info(String.format("%s deleted ALL vaults belonging to %s", sender.getName(), target));
} else {
sender.sendMessage(Lang.TITLE.toString() + Lang.NO_PERMS);
}
} }
VaultOperations.deleteOtherVault(sender, target, args[1]); VaultOperations.deleteOtherVault(sender, target, args[1]);
break; break;