From 2bc0f9f7a2eb0e97c5f8cf947f34d6877702bc3c Mon Sep 17 00:00:00 2001 From: "Dylan K. Taylor" Date: Tue, 27 Apr 2021 22:39:13 +0100 Subject: [PATCH] TimingsCommand: do not enable timings if they are already enabled this causes timings to be reset, which is potentially undesirable (loss of already-collected timing data). closes #4194 --- resources/locale | 2 +- src/command/defaults/TimingsCommand.php | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/resources/locale b/resources/locale index 3d1c70b9f..ca7fecaf2 160000 --- a/resources/locale +++ b/resources/locale @@ -1 +1 @@ -Subproject commit 3d1c70b9f3d7650cef7cac25db27397a5708d22c +Subproject commit ca7fecaf24d8a8f9b947b69f1a8772475e03c379 diff --git a/src/command/defaults/TimingsCommand.php b/src/command/defaults/TimingsCommand.php index 601688690..3c3fbf670 100644 --- a/src/command/defaults/TimingsCommand.php +++ b/src/command/defaults/TimingsCommand.php @@ -75,6 +75,10 @@ class TimingsCommand extends VanillaCommand{ $mode = strtolower($args[0]); if($mode === "on"){ + if(TimingsHandler::isEnabled()){ + $sender->sendMessage(new TranslationContainer("pocketmine.command.timings.alreadyEnabled")); + return true; + } TimingsHandler::setEnabled(); Command::broadcastCommandMessage($sender, new TranslationContainer("pocketmine.command.timings.enable"));