From fa0cfa5191a79a9819c8d6a5902981f9c7bb81ae Mon Sep 17 00:00:00 2001
From: MagicBot <MagicBot@magicbane.com>
Date: Sat, 21 Jan 2023 12:27:33 -0500
Subject: [PATCH] More context in method names.

---
 src/engine/devcmd/cmds/MineActiveCmd.java   |  4 ++--
 src/engine/workthreads/HourlyJobThread.java | 15 ++++++++-------
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/src/engine/devcmd/cmds/MineActiveCmd.java b/src/engine/devcmd/cmds/MineActiveCmd.java
index 83a5d2e8..2263128c 100644
--- a/src/engine/devcmd/cmds/MineActiveCmd.java
+++ b/src/engine/devcmd/cmds/MineActiveCmd.java
@@ -41,11 +41,11 @@ public class MineActiveCmd extends AbstractDevCmd {
 		String trigger = args[0];
 		switch (trigger){
 		case "true":
-			HourlyJobThread.handleStartMineWindow(mine);
+			HourlyJobThread.mineWindowOpen(mine);
 			Mine.setLastChange(System.currentTimeMillis());
 			break;
 		case "false":
-			HourlyJobThread.handleEndMineWindow(mine);
+			HourlyJobThread.mineWindowClose(mine);
 			Mine.setLastChange(System.currentTimeMillis());
 			break;
 		default:
diff --git a/src/engine/workthreads/HourlyJobThread.java b/src/engine/workthreads/HourlyJobThread.java
index 7589049d..58562e8d 100644
--- a/src/engine/workthreads/HourlyJobThread.java
+++ b/src/engine/workthreads/HourlyJobThread.java
@@ -187,7 +187,7 @@ public class HourlyJobThread implements Runnable {
                     // Open Errant Mines
 
                     if (mine.getOwningGuild().isEmptyGuild()) {
-                        HourlyJobThread.handleStartMineWindow(mine);
+                        HourlyJobThread.mineWindowOpen(mine);
                         Mine.setLastChange(System.currentTimeMillis());
                         continue;
                     }
@@ -197,14 +197,14 @@ public class HourlyJobThread implements Runnable {
 
                     if (mine.getOwningGuild().getNation().getMineTime() ==
                             LocalDateTime.now().getHour()) {
-                        HourlyJobThread.handleStartMineWindow(mine);
+                        HourlyJobThread.mineWindowOpen(mine);
                         Mine.setLastChange(System.currentTimeMillis());
                         continue;
                     }
 
                     // Close all the remaining mines
 
-                    if (handleEndMineWindow(mine))
+                    if (mineWindowClose(mine))
                         Mine.setLastChange(System.currentTimeMillis());
 
                 } catch (Exception e) {
@@ -216,14 +216,14 @@ public class HourlyJobThread implements Runnable {
         }
     }
 
-    public static void handleStartMineWindow(Mine mine) {
+    public static void mineWindowOpen(Mine mine) {
 
         mine.setActive(true);
         ChatManager.chatSystemChannel(mine.getZoneName() + "'s Mine is now Active!");
         Logger.info(mine.getZoneName() + "'s Mine is now Active!");
     }
 
-    public static boolean handleEndMineWindow(Mine mine) {
+    public static boolean mineWindowClose(Mine mine) {
 
         // No need to end the window of a mine which never opened.
 
@@ -237,9 +237,10 @@ public class HourlyJobThread implements Runnable {
             return false;
         }
 
+        // Mine building still stands.
+        // It was never knocked down
+
         if (mineBuilding.getRank() > 0) {
-            //never knocked down, let's just move on.
-            //hasn't been claimed since server start.
             mine.setActive(false);
             mine.lastClaimer = null;
             return true;