Merge remote-tracking branch 'origin/minetime_bugfix' into post-wipe-merge

src/engine/db/handlers/dbGuildHandler.java

@@ -19,6 +19,7 @@ import org.pmw.tinylog.Logger;
 
 import java.sql.ResultSet;
 import java.sql.SQLException;
+import java.time.LocalDateTime;
 import java.util.ArrayList;
 
 public class dbGuildHandler extends dbHandlerBase {
@@ -114,8 +115,12 @@ public class dbGuildHandler extends dbHandlerBase {
 		return outputStr;
 	}
 
-
-
+	public boolean SET_LAST_WOO_UPDATE(Guild guild, LocalDateTime lastEditTime) {
+		prepareCallable("UPDATE `obj_guild` SET `lastWooEditTime`=? WHERE `UID`=?");
+		setLocalDateTime(1, lastEditTime);
+		setLong(2, (long) guild.getObjectUUID());
+		return (executeUpdate() > 0);
+	}
 
 	public ArrayList<Guild> GET_GUILD_ALLIES(final int id) {
 		prepareCallable("SELECT g.* FROM `obj_guild` g, `dyn_guild_allianceenemylist` l "

src/engine/db/handlers/dbMineHandler.java

@@ -6,19 +6,6 @@
 //      Magicbane Emulator Project © 2013 - 2022
 //                www.magicbane.com
 
-
-
-
-
-// • ▌ ▄ ·.  ▄▄▄·  ▄▄ • ▪   ▄▄· ▄▄▄▄·  ▄▄▄·  ▐▄▄▄  ▄▄▄ .
-// ·██ ▐███▪▐█ ▀█ ▐█ ▀ ▪██ ▐█ ▌▪▐█ ▀█▪▐█ ▀█ •█▌ ▐█▐▌·
-// ▐█ ▌▐▌▐█·▄█▀▀█ ▄█ ▀█▄▐█·██ ▄▄▐█▀▀█▄▄█▀▀█ ▐█▐ ▐▌▐▀▀▀
-// ██ ██▌▐█▌▐█ ▪▐▌▐█▄▪▐█▐█▌▐███▌██▄▪▐█▐█ ▪▐▌██▐ █▌▐█▄▄▌
-// ▀▀  █▪▀▀▀ ▀  ▀ ·▀▀▀▀ ▀▀▀·▀▀▀ ·▀▀▀▀  ▀  ▀ ▀▀  █▪ ▀▀▀
-//      Magicbane Emulator Project © 2013 - 2022
-//                www.magicbane.com
-
-
 package engine.db.handlers;
 
 import engine.Enum;

src/engine/net/client/ClientMessagePump.java

@@ -265,9 +265,6 @@ public class ClientMessagePump implements NetMsgHandler {
 			case ARCMINEWINDOWAVAILABLETIME:
 				MineWindowAvailableTime((ArcMineWindowAvailableTimeMsg) msg, origin);
 				break;
-			case ARCMINEWINDOWCHANGE:
-				MineWindowChange((ArcMineWindowChangeMsg) msg, origin);
-				break;
 			case ARCOWNEDMINESLIST:
 				ListOwnedMines((ArcOwnedMinesListMsg) msg, origin);
 				break;

src/engine/net/client/Protocol.java

@@ -36,7 +36,7 @@ public enum Protocol {
     ARCMINECHANGEPRODUCTION(0x1EAA993F,  ArcMineChangeProductionMsg.class, null),
     ARCMINETOWERCRESTUPDATE(0x34164D0D, null, null),
     ARCMINEWINDOWAVAILABLETIME(0x6C909DE7, ArcMineWindowAvailableTimeMsg.class, null),
-    ARCMINEWINDOWCHANGE(0x92B2148A, ArcMineWindowChangeMsg.class, null),
+    ARCMINEWINDOWCHANGE(0x92B2148A, ArcMineWindowChangeMsg.class, MineWindowChangeHandler.class),
     ARCOWNEDMINESLIST(0x59184455, ArcOwnedMinesListMsg.class, null),
     ARCPETATTACK(0x18CD61AD, PetAttackMsg.class, null), // Pet Attack
     ARCPETCMD(0x4E80E001, PetCmdMsg.class, null), // Stop ArcPetAttack, Toggle Assist, Toggle Rest

src/engine/net/client/handlers/MineWindowChangeHandler.java

@@ -0,0 +1,115 @@
+// • ▌ ▄ ·.  ▄▄▄·  ▄▄ • ▪   ▄▄· ▄▄▄▄·  ▄▄▄·  ▐▄▄▄  ▄▄▄ .
+// ·██ ▐███▪▐█ ▀█ ▐█ ▀ ▪██ ▐█ ▌▪▐█ ▀█▪▐█ ▀█ •█▌ ▐█▐▌·
+// ▐█ ▌▐▌▐█·▄█▀▀█ ▄█ ▀█▄▐█·██ ▄▄▐█▀▀█▄▄█▀▀█ ▐█▐ ▐▌▐▀▀▀
+// ██ ██▌▐█▌▐█ ▪▐▌▐█▄▪▐█▐█▌▐███▌██▄▪▐█▐█ ▪▐▌██▐ █▌▐█▄▄▌
+// ▀▀  █▪▀▀▀ ▀  ▀ ·▀▀▀▀ ▀▀▀·▀▀▀ ·▀▀▀▀  ▀  ▀ ▀▀  █▪ ▀▀▀
+//      Magicbane Emulator Project © 2013 - 2022
+//                www.magicbane.com
+
+package engine.net.client.handlers;
+
+import engine.Enum;
+import engine.Enum.DispatchChannel;
+import engine.exception.MsgSendException;
+import engine.gameManager.BuildingManager;
+import engine.gameManager.ChatManager;
+import engine.gameManager.DbManager;
+import engine.gameManager.SessionManager;
+import engine.net.Dispatch;
+import engine.net.DispatchMessage;
+import engine.net.client.ClientConnection;
+import engine.net.client.msg.ArcMineWindowChangeMsg;
+import engine.net.client.msg.ClientNetMsg;
+import engine.net.client.msg.ErrorPopupMsg;
+import engine.net.client.msg.KeepAliveServerClientMsg;
+import engine.objects.Building;
+import engine.objects.Guild;
+import engine.objects.GuildStatusController;
+import engine.objects.PlayerCharacter;
+import engine.server.MBServerStatics;
+import org.pmw.tinylog.Logger;
+
+import java.time.LocalDateTime;
+
+/*
+ * @Author:
+ * @Summary: Processes requests to change a mine's opendate
+ */
+
+public class MineWindowChangeHandler extends AbstractClientMsgHandler {
+
+	public MineWindowChangeHandler() {
+		super(ArcMineWindowChangeMsg.class);
+	}
+
+	@Override
+	protected boolean _handleNetMsg(ClientNetMsg baseMsg, ClientConnection origin) throws MsgSendException {
+
+		PlayerCharacter playerCharacter = SessionManager.getPlayerCharacter(origin);
+		ArcMineWindowChangeMsg mineWindowChangeMsg = (ArcMineWindowChangeMsg)baseMsg;
+		int newMineTime;
+
+		if (playerCharacter == null)
+			return true;
+
+		Building treeOfLife =  BuildingManager.getBuildingFromCache(mineWindowChangeMsg.getBuildingID());
+
+		if (treeOfLife == null)
+			return true;
+
+		if (treeOfLife.getBlueprintUUID() == 0)
+			return true;
+
+		if (treeOfLife.getBlueprint().getBuildingGroup() != Enum.BuildingGroup.TOL)
+			return true;
+
+		Guild mineGuild = treeOfLife.getGuild();
+		if (mineGuild == null)
+			return true;
+
+		if (!Guild.sameGuild(mineGuild, playerCharacter.getGuild()))
+			return true;  //must be same guild
+
+		if (GuildStatusController.isInnerCouncil(playerCharacter.getGuildStatus()) == false) // is this only GL?
+			return true;
+
+		newMineTime = mineWindowChangeMsg.getTime();
+
+		// Enforce 15hr restriction between WOO edits
+
+		if (LocalDateTime.now().isBefore(mineGuild.lastWooEditTime.plusHours(14))) {
+			ErrorPopupMsg.sendErrorMsg(playerCharacter, "You must wait 15 hours between WOO changes.");
+			return true;
+		}
+
+		//hodge podge sanity check to make sure they don't set it before early window and is not set at late window.
+
+		if (newMineTime < MBServerStatics.MINE_EARLY_WINDOW &&
+				newMineTime != MBServerStatics.MINE_LATE_WINDOW)
+			return true;    //invalid mine time, must be in range
+
+		// Update guild mine time
+
+		if (!DbManager.GuildQueries.UPDATE_MINETIME(mineGuild.getObjectUUID(), newMineTime)) {
+			Logger.error("MineWindowChange", "Failed to update mine time for guild " + mineGuild.getObjectUUID());
+			ChatManager.chatGuildError(playerCharacter, "Failed to update the mine time");
+			return true;
+		}
+
+		mineGuild.setMineTime(newMineTime);
+		mineGuild.lastWooEditTime = LocalDateTime.now();
+
+		// Update guild WOO timer for reboot persistence
+
+		if (!DbManager.GuildQueries.SET_LAST_WOO_UPDATE(mineGuild, mineGuild.lastWooEditTime)) {
+			Logger.error("MineWindowChange", "Failed to update woo timer for guild " + mineGuild.getObjectUUID());
+			ChatManager.chatGuildError(playerCharacter, "A Serious error has for to occurred.");
+			return true;
+		}
+
+		ChatManager.chatGuildInfo(playerCharacter, "Mine time updated.");
+            
+            return true;
+	}
+
+}

src/engine/objects/Guild.java

@@ -40,7 +40,10 @@ import org.pmw.tinylog.Logger;
 
 import java.sql.ResultSet;
 import java.sql.SQLException;
+import java.sql.Timestamp;
+import java.time.LocalDateTime;
 import java.util.ArrayList;
+import java.util.Date;
 import java.util.HashMap;
 import java.util.concurrent.ConcurrentHashMap;
 
@@ -79,6 +82,7 @@ public class Guild extends AbstractWorldObject {
 	private String hash;
 	private boolean ownerIsNPC;
 
+	public LocalDateTime lastWooEditTime;
 	public HashMap<Integer,GuildAlliances> guildAlliances = new HashMap<>();
 
 	/**
@@ -176,8 +180,13 @@ public class Guild extends AbstractWorldObject {
 		this.teleportMax = rs.getInt("teleportMax");
 
 		this.mineTime = rs.getInt("mineTime");
-		this.hash = rs.getString("hash");
 
+		Timestamp lastWooRequest = rs.getTimestamp("lastWooEditTime");
+
+		if (lastWooRequest != null)
+			this.lastWooEditTime = lastWooRequest.toLocalDateTime();
+
+		this.hash = rs.getString("hash");
 	}
 
 	public void setNation(Guild nation) {

src/engine/objects/Mine.java

@@ -35,6 +35,7 @@ import org.pmw.tinylog.Logger;
 import java.net.UnknownHostException;
 import java.sql.ResultSet;
 import java.sql.SQLException;
+import java.sql.Timestamp;
 import java.time.LocalDateTime;
 import java.util.ArrayList;
 import java.util.concurrent.ConcurrentHashMap;
@@ -141,47 +142,11 @@ public class Mine extends AbstractGameObject {
 		this.lastClaimerID = 0;
 		this.lastClaimerSessionID = null;
 
-	java.sql.Timestamp mineTimeStamp = rs.getTimestamp("mine_openDate");
-	
-	
-	Building building = BuildingManager.getBuildingFromCache(this.buildingID);
-	
 
-	if (mineTimeStamp == null && (this.owningGuild == null || this.owningGuild.isErrant() || building.getRank() < 1)){
-		if (building != null){
-			String zoneName = building.getParentZone().getName();
-			String parentZoneName = building.getParentZone().getParent().getName();
-			Logger.info(zoneName + " in " + parentZoneName + " has a dirty mine, setting active.");
-		}
-		this.dirtyMine = true;
-		openDate = LocalDateTime.now().withMinute(0).withSecond(0).withNano(0);
-		return;
-	}else if (this.owningGuild.isErrant() || nation.isErrant()){
-		openDate = LocalDateTime.now().withMinute(0).withSecond(0).withNano(0);
-		return;
-	}else if (mineTimeStamp == null){
-	
-		this.openDate = LocalDateTime.now().withHour(mineTime).withMinute(0).withSecond(0).withNano(0);
-		
-		if (LocalDateTime.now().isAfter(this.openDate.plusHours(1)))
-			this.openDate = this.openDate.plusDays(1);
-		return;
-	}else{
-		this.openDate =  mineTimeStamp.toLocalDateTime().withHour(mineTime);
-		
-		if (LocalDateTime.now().isAfter(this.openDate.plusHours(1))){
-			this.openDate = this.openDate.plusDays(1);
-			return;
-		}
-	}
-	
-	//after 1 day...
-	if(this.openDate.getDayOfYear() - LocalDateTime.now().getDayOfYear() > 1){
-		this.openDate = this.openDate.withDayOfYear(LocalDateTime.now().getDayOfYear());
-		if (LocalDateTime.now().isAfter(this.openDate.plusHours(1)))
-			this.openDate = this.openDate.plusDays(1);
-		return;
-	}
+	Timestamp mineOpenDateTime = rs.getTimestamp("mine_openDate");
+
+		if (mineOpenDateTime != null)
+			this.openDate = mineOpenDateTime.toLocalDateTime();
 	
 	}
 
@@ -258,10 +223,6 @@ try{
 	 */
 	private void initializeMineTime(){
 
-		//Mine time has already been set at loading from the database. skip.
-
-		if (this.openDate != null)
-			return;
 
 		Guild nation = null;