From 759cd230a1f82cf29a2b5ff3d7305e5bebc0a951 Mon Sep 17 00:00:00 2001
From: Brokkonaut <hannos17@gmx.de>
Date: Tue, 31 Jul 2018 15:54:21 +0200
Subject: [PATCH] sanity checks for some command params

rollback/redo of chat or kills is not possible
---
 .../de/diddiz/LogBlock/CommandsHandler.java   | 24 +++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/src/main/java/de/diddiz/LogBlock/CommandsHandler.java b/src/main/java/de/diddiz/LogBlock/CommandsHandler.java
index aaf4a72..400920f 100755
--- a/src/main/java/de/diddiz/LogBlock/CommandsHandler.java
+++ b/src/main/java/de/diddiz/LogBlock/CommandsHandler.java
@@ -634,6 +634,18 @@ public class CommandsHandler implements CommandExecutor {
         @Override
         public void run() {
             try {
+                if(params.bct == BlockChangeType.CHAT) {
+                    sender.sendMessage(ChatColor.RED + "Chat cannot be rolled back");
+                    return;
+                }
+                if(params.bct == BlockChangeType.KILLS) {
+                    sender.sendMessage(ChatColor.RED + "Kills cannot be rolled back");
+                    return;
+                }
+                if(params.sum != SummarizationMode.NONE) {
+                    sender.sendMessage(ChatColor.RED + "Cannot rollback summarized changes");
+                    return;
+                }
                 params.needCoords = true;
                 params.needType = true;
                 params.needData = true;
@@ -712,6 +724,18 @@ public class CommandsHandler implements CommandExecutor {
         @Override
         public void run() {
             try {
+                if(params.bct == BlockChangeType.CHAT) {
+                    sender.sendMessage(ChatColor.RED + "Chat cannot be redone");
+                    return;
+                }
+                if(params.bct == BlockChangeType.KILLS) {
+                    sender.sendMessage(ChatColor.RED + "Kills cannot be redone");
+                    return;
+                }
+                if(params.sum != SummarizationMode.NONE) {
+                    sender.sendMessage(ChatColor.RED + "Cannot redo summarized changes");
+                    return;
+                }
                 params.needCoords = true;
                 params.needType = true;
                 params.needData = true;