Rename and clean up MapleTrade

src/main/java/client/Character.java

@@ -167,7 +167,7 @@ public class Character extends AbstractCharacterObject {
     private Shop shop = null;
     private SkinColor skinColor = SkinColor.NORMAL;
     private Storage storage = null;
-    private MapleTrade trade = null;
+    private Trade trade = null;
     private MonsterBook monsterbook;
     private CashShop cashshop;
     private final Set<NewYearCardRecord> newyears = new LinkedHashSet<>();
@@ -1676,7 +1676,7 @@ public class Character extends AbstractCharacterObject {
 
         this.unregisterChairBuff();
         this.clearBanishPlayerData();
-        MapleTrade.cancelTrade(this, MapleTrade.TradeResult.UNSUCCESSFUL_ANOTHER_MAP);
+        Trade.cancelTrade(this, Trade.TradeResult.UNSUCCESSFUL_ANOTHER_MAP);
         this.closePlayerInteractions();
 
         Party e = null;
@@ -5532,7 +5532,7 @@ public class Character extends AbstractCharacterObject {
     }
 
     public void closeTrade() {
-        MapleTrade.cancelTrade(this, MapleTrade.TradeResult.PARTNER_CANCEL);
+        Trade.cancelTrade(this, Trade.TradeResult.PARTNER_CANCEL);
     }
 
     public void closePlayerShop() {
@@ -5887,7 +5887,7 @@ public class Character extends AbstractCharacterObject {
         return summons.containsValue(summon);
     }
 
-    public MapleTrade getTrade() {
+    public Trade getTrade() {
         return trade;
     }
 
@@ -9574,7 +9574,7 @@ public class Character extends AbstractCharacterObject {
         slots = slotid;
     }
 
-    public void setTrade(MapleTrade trade) {
+    public void setTrade(Trade trade) {
         this.trade = trade;
     }
 

src/main/java/client/processor/npc/DueyProcessor.java

@@ -37,7 +37,7 @@ import constants.inventory.ItemConstants;
 import net.server.channel.Channel;
 import server.DueyPackage;
 import server.ItemInformationProvider;
-import server.MapleTrade;
+import server.Trade;
 import tools.DatabaseConnection;
 import tools.FilePrinter;
 import tools.PacketCreator;
@@ -281,7 +281,7 @@ public class DueyProcessor {
     public static void dueySendItem(Client c, byte invTypeId, short itemPos, short amount, int sendMesos, String sendMessage, String recipient, boolean quick) {
         if (c.tryacquireClient()) {
             try {
-                int fee = MapleTrade.getFee(sendMesos);
+                int fee = Trade.getFee(sendMesos);
                 if (!quick) {
                     fee += 5000;
                 } else if (!c.getPlayer().haveItem(5330000)) {

src/main/java/net/server/channel/handlers/ChangeMapHandler.java

@@ -27,7 +27,7 @@ import client.inventory.InventoryType;
 import client.inventory.manipulator.InventoryManipulator;
 import net.AbstractPacketHandler;
 import net.packet.InPacket;
-import server.MapleTrade;
+import server.Trade;
 import server.maps.MapleMap;
 import server.maps.Portal;
 import tools.FilePrinter;
@@ -53,7 +53,7 @@ public final class ChangeMapHandler extends AbstractPacketHandler {
             return;
         }
         if (chr.getTrade() != null) {
-            MapleTrade.cancelTrade(chr, MapleTrade.TradeResult.UNSUCCESSFUL_ANOTHER_MAP);
+            Trade.cancelTrade(chr, Trade.TradeResult.UNSUCCESSFUL_ANOTHER_MAP);
         }
         if (p.available() == 0) { //Cash Shop :)
             if (!chr.getCashShop().isOpened()) {

src/main/java/net/server/channel/handlers/ChangeMapSpecialHandler.java

@@ -24,8 +24,8 @@ package net.server.channel.handlers;
 import client.Client;
 import net.AbstractPacketHandler;
 import net.packet.InPacket;
-import server.MapleTrade;
+import server.Trade;
-import server.MapleTrade.TradeResult;
+import server.Trade.TradeResult;
 import server.maps.Portal;
 import tools.PacketCreator;
 
@@ -45,7 +45,7 @@ public final class ChangeMapSpecialHandler extends AbstractPacketHandler {
                     return;
             }
             if (c.getPlayer().getTrade() != null) {
-                    MapleTrade.cancelTrade(c.getPlayer(), TradeResult.UNSUCCESSFUL_ANOTHER_MAP);
+                    Trade.cancelTrade(c.getPlayer(), TradeResult.UNSUCCESSFUL_ANOTHER_MAP);
             }
             portal.enterPortal(c);   
     }

src/main/java/net/server/channel/handlers/PlayerInteractionHandler.java

@@ -35,7 +35,7 @@ import constants.inventory.ItemConstants;
 import net.AbstractPacketHandler;
 import net.packet.InPacket;
 import server.ItemInformationProvider;
-import server.MapleTrade;
+import server.Trade;
 import server.maps.*;
 import server.maps.MiniGame.MiniGameType;
 import tools.FilePrinter;
@@ -144,7 +144,7 @@ public final class PlayerInteractionHandler extends AbstractPacketHandler {
 
                 byte createType = p.readByte();
                 if (createType == 3) {  // trade
-                    MapleTrade.startTrade(chr);
+                    Trade.startTrade(chr);
                 } else if (createType == 1) { // omok mini game
                     int status = establishMiniroomStatus(chr, true);
                     if (status > 0) {
@@ -266,13 +266,13 @@ public final class PlayerInteractionHandler extends AbstractPacketHandler {
                     return;
                 }
                 
-                MapleTrade.inviteTrade(chr, other);
+                Trade.inviteTrade(chr, other);
             } else if (mode == Action.DECLINE.getCode()) {
-                MapleTrade.declineTrade(chr);
+                Trade.declineTrade(chr);
             } else if (mode == Action.VISIT.getCode()) {
                 if (chr.getTrade() != null && chr.getTrade().getPartner() != null) {
                     if (!chr.getTrade().isFullTrade() && !chr.getTrade().getPartner().isFullTrade()) {
-                        MapleTrade.visitTrade(chr, chr.getTrade().getPartner().getChr());
+                        Trade.visitTrade(chr, chr.getTrade().getPartner().getChr());
                     } else {
                         chr.sendPacket(PacketCreator.getMiniRoomError(2));
                         return;
@@ -332,7 +332,7 @@ public final class PlayerInteractionHandler extends AbstractPacketHandler {
                 }
             } else if (mode == Action.EXIT.getCode()) {
                 if (chr.getTrade() != null) {
-                    MapleTrade.cancelTrade(chr, MapleTrade.TradeResult.PARTNER_CANCEL);
+                    Trade.cancelTrade(chr, Trade.TradeResult.PARTNER_CANCEL);
                 } else {
                     chr.closePlayerShop();
                     chr.closeMiniGame(false);
@@ -507,7 +507,7 @@ public final class PlayerInteractionHandler extends AbstractPacketHandler {
                     return;
                 }
                 
-                MapleTrade trade = chr.getTrade();
+                Trade trade = chr.getTrade();
                 if (trade != null) {
                     if ((quantity <= item.getQuantity() && quantity >= 0) || ItemConstants.isRechargeable(item.getItemId())) {
                         if (ii.isDropRestricted(item.getItemId())) { // ensure that undroppable items do not make it to the trade window
@@ -552,7 +552,7 @@ public final class PlayerInteractionHandler extends AbstractPacketHandler {
                     }
                 }
             } else if (mode == Action.CONFIRM.getCode()) {
-                MapleTrade.completeTrade(chr);
+                Trade.completeTrade(chr);
             } else if (mode == Action.ADD_ITEM.getCode() || mode == Action.PUT_ITEM.getCode()) {
                 if (isTradeOpen(chr)) return;
 

src/main/java/server/Trade.java

@@ -43,11 +43,10 @@ import java.util.List;
 import java.util.concurrent.atomic.AtomicBoolean;
 
 /**
- *
  * @author Matze
  * @author Ronan - concurrency safety + check available slots + trade results
  */
-public class MapleTrade {
+public class Trade {
 
     public enum TradeResult {
         NO_RESPONSE(1),
@@ -60,7 +59,7 @@ public class MapleTrade {
 
         private final int res;
 
-        private TradeResult(int res) {
+        TradeResult(int res) {
             this.res = res;
         }
 
@@ -69,17 +68,17 @@ public class MapleTrade {
         }
     }
 
-    private MapleTrade partner = null;
+    private Trade partner = null;
-    private List<Item> items = new ArrayList<>();
+    private final List<Item> items = new ArrayList<>();
     private List<Item> exchangeItems;
     private int meso = 0;
     private int exchangeMeso;
-    private AtomicBoolean locked = new AtomicBoolean(false);
+    private final AtomicBoolean locked = new AtomicBoolean(false);
-    private Character chr;
+    private final Character chr;
-    private byte number;
+    private final byte number;
     private boolean fullTrade = false;
 
-    public MapleTrade(byte number, Character chr) {
+    public Trade(byte number, Character chr) {
         this.chr = chr;
         this.number = number;
     }
@@ -127,7 +126,7 @@ public class MapleTrade {
             int fee = getFee(exchangeMeso);
 
             chr.gainMeso(exchangeMeso - fee, show, true, show);
-            if(fee > 0) {
+            if (fee > 0) {
                 chr.dropMessage(1, "Transaction completed. You received " + GameConstants.numberWithCommas(exchangeMeso - fee) + " mesos due to trade fees.");
             } else {
                 chr.dropMessage(1, "Transaction completed. You received " + GameConstants.numberWithCommas(exchangeMeso) + " mesos.");
@@ -218,11 +217,11 @@ public class MapleTrade {
         }
     }
 
-    public MapleTrade getPartner() {
+    public Trade getPartner() {
         return partner;
     }
 
-    public void setPartner(MapleTrade partner) {
+    public void setPartner(Trade partner) {
         if (locked.get()) {
             return;
         }
@@ -237,8 +236,8 @@ public class MapleTrade {
         return new LinkedList<>(items);
     }
 
-    public int getExchangeMesos(){
+    public int getExchangeMesos() {
-    	return exchangeMeso;
+        return exchangeMeso;
     }
 
     private boolean fitsMeso() {
@@ -264,7 +263,7 @@ public class MapleTrade {
     }
 
     private synchronized boolean checkTradeCompleteHandshake(boolean updateSelf) {
-        MapleTrade self, other;
+        Trade self, other;
 
         if (updateSelf) {
             self = this;
@@ -291,8 +290,8 @@ public class MapleTrade {
     }
 
     public static void completeTrade(Character chr) {
-        MapleTrade local = chr.getTrade();
+        Trade local = chr.getTrade();
-        MapleTrade partner = local.getPartner();
+        Trade partner = local.getPartner();
         if (local.checkCompleteHandshake()) {
             local.fetchExchangedItems();
             partner.fetchExchangedItems();
@@ -359,8 +358,10 @@ public class MapleTrade {
     }
 
     private static void cancelTradeInternal(Character chr, byte selfResult, byte partnerResult) {
-        MapleTrade trade = chr.getTrade();
+        Trade trade = chr.getTrade();
-        if(trade == null) return;
+        if (trade == null) {
+            return;
+        }
 
         trade.cancel(selfResult);
         if (trade.getPartner() != null) {
@@ -392,7 +393,7 @@ public class MapleTrade {
 
     private synchronized void tradeCancelHandshake(boolean updateSelf, byte result) {
         byte selfResult, partnerResult;
-        MapleTrade self;
+        Trade self;
 
         byte[] pairedResult = tradeResultsPair(result);
         selfResult = pairedResult[0];
@@ -408,7 +409,7 @@ public class MapleTrade {
     }
 
     private void cancelHandshake(byte result) {  // handshake checkout thanks to Ronan
-        MapleTrade partner = this.getPartner();
+        Trade partner = this.getPartner();
         if (partner == null || this.getChr().getId() < partner.getChr().getId()) {
             this.tradeCancelHandshake(true, result);
         } else {
@@ -417,26 +418,26 @@ public class MapleTrade {
     }
 
     public static void cancelTrade(Character chr, TradeResult result) {
-        MapleTrade trade = chr.getTrade();
+        Trade trade = chr.getTrade();
-        if(trade == null) return;
+        if (trade == null) {
+            return;
+        }
 
         trade.cancelHandshake(result.getValue());
     }
 
     public static void startTrade(Character chr) {
         if (chr.getTrade() == null) {
-            chr.setTrade(new MapleTrade((byte) 0, chr));
+            chr.setTrade(new Trade((byte) 0, chr));
         }
     }
 
     private static boolean hasTradeInviteBack(Character c1, Character c2) {
-        MapleTrade other = c2.getTrade();
+        Trade other = c2.getTrade();
         if (other != null) {
-            MapleTrade otherPartner = other.getPartner();
+            Trade otherPartner = other.getPartner();
             if (otherPartner != null) {
-                if (otherPartner.getChr().getId() == c1.getId()) {
+                return otherPartner.getChr().getId() == c1.getId();
-                    return true;
-                }
             }
         }
 
@@ -459,7 +460,7 @@ public class MapleTrade {
 
         if (InviteCoordinator.createInvite(InviteType.TRADE, c1, c1.getId(), c2.getId())) {
             if (c2.getTrade() == null) {
-                c2.setTrade(new MapleTrade((byte) 1, c2));
+                c2.setTrade(new Trade((byte) 1, c2));
                 c2.getTrade().setPartner(c1.getTrade());
                 c1.getTrade().setPartner(c2.getTrade());
 
@@ -496,7 +497,7 @@ public class MapleTrade {
     }
 
     public static void declineTrade(Character chr) {
-        MapleTrade trade = chr.getTrade();
+        Trade trade = chr.getTrade();
         if (trade != null) {
             if (trade.getPartner() != null) {
                 Character other = trade.getPartner().getChr();
@@ -513,11 +514,11 @@ public class MapleTrade {
         }
     }
 
-	public boolean isFullTrade() {
+    public boolean isFullTrade() {
-		return fullTrade;
+        return fullTrade;
-	}
+    }
 
-	public void setFullTrade(boolean fullTrade) {
+    public void setFullTrade(boolean fullTrade) {
-		this.fullTrade = fullTrade;
+        this.fullTrade = fullTrade;
-	}
+    }
 }

src/main/java/server/maps/HiredMerchant.java

@@ -36,7 +36,7 @@ import net.server.Server;
 import net.server.audit.locks.MonitoredLockType;
 import net.server.audit.locks.factory.MonitoredReentrantLockFactory;
 import server.ItemInformationProvider;
-import server.MapleTrade;
+import server.Trade;
 import tools.DatabaseConnection;
 import tools.PacketCreator;
 import tools.Pair;
@@ -279,7 +279,7 @@ public class HiredMerchant extends AbstractMapObject {
             if (c.getPlayer().getMeso() >= price) {
                 if (canBuy(c, newItem)) {
                     c.getPlayer().gainMeso(-price, false);
-                    price -= MapleTrade.getFee(price);  // thanks BHB for pointing out trade fees not applying here
+                    price -= Trade.getFee(price);  // thanks BHB for pointing out trade fees not applying here
 
                     synchronized (sold) {
                         sold.add(new SoldItem(c.getPlayer().getName(), pItem.getItem().getItemId(), newItem.getQuantity(), price));

src/main/java/server/maps/PlayerShop.java

@@ -31,7 +31,7 @@ import client.inventory.manipulator.KarmaManipulator;
 import net.packet.Packet;
 import net.server.audit.locks.MonitoredLockType;
 import net.server.audit.locks.factory.MonitoredReentrantLockFactory;
-import server.MapleTrade;
+import server.Trade;
 import tools.PacketCreator;
 import tools.Pair;
 
@@ -283,7 +283,7 @@ public class PlayerShop extends AbstractMapObject {
 
                         if (canBuy(c, newItem)) {
                             c.getPlayer().gainMeso(-price, false);
-                            price -= MapleTrade.getFee(price);  // thanks BHB for pointing out trade fees not applying here
+                            price -= Trade.getFee(price);  // thanks BHB for pointing out trade fees not applying here
                             owner.gainMeso(price, true);
 
                             SoldItem soldItem = new SoldItem(c.getPlayer().getName(), pItem.getItem().getItemId(), quantity, price);

src/main/java/tools/LogHelper.java

@@ -5,7 +5,7 @@ import client.Client;
 import client.inventory.Item;
 import net.server.Server;
 import server.ItemInformationProvider;
-import server.MapleTrade;
+import server.Trade;
 import server.expeditions.Expedition;
 
 import java.text.SimpleDateFormat;
@@ -14,7 +14,7 @@ import java.util.Date;
 
 public class LogHelper {
 
-	public static void logTrade(MapleTrade trade1, MapleTrade trade2) {
+	public static void logTrade(Trade trade1, Trade trade2) {
 		String name1 = trade1.getChr().getName();
 		String name2 = trade2.getChr().getName();
 		String log = "TRADE BETWEEN " + name1 + " AND " + name2 + "\r\n";

src/main/java/tools/PacketCreator.java

@@ -3243,7 +3243,7 @@ public class PacketCreator {
         return p;
     }
 
-    public static Packet getTradeStart(Client c, MapleTrade trade, byte number) {
+    public static Packet getTradeStart(Client c, Trade trade, byte number) {
         final OutPacket p = OutPacket.create(SendOpcode.PLAYER_INTERACTION);
         p.writeByte(PlayerInteractionHandler.Action.ROOM.getCode());
         p.writeByte(3);