Browse Source

CosttoOpenBankMsg has no handler

combat-2
MagicBot 8 months ago
parent
commit
35ba4bbeec
  1. 9
      src/engine/net/client/ClientMessagePump.java
  2. 2
      src/engine/net/client/Protocol.java
  3. 4
      src/engine/net/client/handlers/VendorDialogMsgHandler.java
  4. 6
      src/engine/net/client/msg/CostOpenBankMsg.java

9
src/engine/net/client/ClientMessagePump.java

@ -48,12 +48,6 @@ public class ClientMessagePump implements NetMsgHandler {
this.server = server; this.server = server;
} }
private static void ackBankWindowOpened(AckBankWindowOpenedMsg msg, ClientConnection origin) {
// According to the Wiki, the client should not send this message.
// Log the instance to investigate, and modify Wiki accordingly.
Logger.error(msg.toString());
}
// called when player clicks respawn button // called when player clicks respawn button
private static void respawn(RespawnMsg msg, ClientConnection origin) throws MsgSendException { private static void respawn(RespawnMsg msg, ClientConnection origin) throws MsgSendException {
@ -235,9 +229,6 @@ public class ClientMessagePump implements NetMsgHandler {
break; break;
case OPENVAULT: case OPENVAULT:
break; break;
case COSTTOOPENBANK:
ackBankWindowOpened((AckBankWindowOpenedMsg) msg, origin);
break;
case RESETAFTERDEATH: case RESETAFTERDEATH:
respawn((RespawnMsg) msg, origin); respawn((RespawnMsg) msg, origin);
break; break;

2
src/engine/net/client/Protocol.java

@ -81,7 +81,7 @@ public enum Protocol {
CLIENTUPDATEVAULT(0x66EDBECD, UpdateVaultMsg.class, null), CLIENTUPDATEVAULT(0x66EDBECD, UpdateVaultMsg.class, null),
COMBATMODE(0xFE4BF353, ToggleCombatMsg.class, ToggleCombatMsgHandler.class), //Toggle Combat mode COMBATMODE(0xFE4BF353, ToggleCombatMsg.class, ToggleCombatMsgHandler.class), //Toggle Combat mode
CONFIRMPROMOTE(0x153BB5F9, ConfirmPromoteMsg.class, null), CONFIRMPROMOTE(0x153BB5F9, ConfirmPromoteMsg.class, null),
COSTTOOPENBANK(0x135BE5E8, AckBankWindowOpenedMsg.class, null), // ACK Bank Window Opened COSTTOOPENBANK(0x135BE5E8, CostOpenBankMsg.class, null), // ACK Bank Window Opened
CREATECHAR(0x5D18B5C8, CommitNewCharacterMsg.class, null), // Commit New Character, CREATECHAR(0x5D18B5C8, CommitNewCharacterMsg.class, null), // Commit New Character,
CREATEPETITION(0xD489CFED, GuildCreationFinalizeMsg.class, GuildCreationFinalizeHandler.class), //Confirm guild creation CREATEPETITION(0xD489CFED, GuildCreationFinalizeMsg.class, GuildCreationFinalizeHandler.class), //Confirm guild creation
CUSTOMERPETITION(0x7F9D7D6D, PetitionReceivedMsg.class, PetitionReceivedMsgHandler.class), CUSTOMERPETITION(0x7F9D7D6D, PetitionReceivedMsg.class, PetitionReceivedMsgHandler.class),

4
src/engine/net/client/handlers/VendorDialogMsgHandler.java

@ -510,9 +510,9 @@ public class VendorDialogMsgHandler extends AbstractClientMsgHandler {
// TODO When do we setBankOpen(false)? I don't think the client sends a // TODO When do we setBankOpen(false)? I don't think the client sends a
// "CloseBank" message. // "CloseBank" message.
AckBankWindowOpenedMsg ackBankWindowOpenedMsg = new AckBankWindowOpenedMsg(playerCharacter, 0L, 0L); CostOpenBankMsg costOpenBankMsg = new CostOpenBankMsg(playerCharacter, 0L, 0L);
Dispatch dispatch = Dispatch.borrow(playerCharacter, ackBankWindowOpenedMsg); Dispatch dispatch = Dispatch.borrow(playerCharacter, costOpenBankMsg);
DispatchMessage.dispatchMsgDispatch(dispatch, DispatchChannel.SECONDARY); DispatchMessage.dispatchMsgDispatch(dispatch, DispatchChannel.SECONDARY);
ReqBankInventoryMsg reqBankInventoryMsg = new ReqBankInventoryMsg(playerCharacter, 0L); ReqBankInventoryMsg reqBankInventoryMsg = new ReqBankInventoryMsg(playerCharacter, 0L);

6
src/engine/net/client/msg/AckBankWindowOpenedMsg.java → src/engine/net/client/msg/CostOpenBankMsg.java

@ -22,7 +22,7 @@ import engine.objects.PlayerCharacter;
* *
* @author Burfo * @author Burfo
*/ */
public class AckBankWindowOpenedMsg extends ClientNetMsg { public class CostOpenBankMsg extends ClientNetMsg {
private int playerType; private int playerType;
private int playerID; private int playerID;
@ -32,7 +32,7 @@ public class AckBankWindowOpenedMsg extends ClientNetMsg {
/** /**
* This is the general purpose constructor. * This is the general purpose constructor.
*/ */
public AckBankWindowOpenedMsg(PlayerCharacter pc, long unknown01, long unknown02) { public CostOpenBankMsg(PlayerCharacter pc, long unknown01, long unknown02) {
super(Protocol.COSTTOOPENBANK); super(Protocol.COSTTOOPENBANK);
this.playerType = pc.getObjectType().ordinal(); this.playerType = pc.getObjectType().ordinal();
this.playerID = pc.getObjectUUID(); this.playerID = pc.getObjectUUID();
@ -46,7 +46,7 @@ public class AckBankWindowOpenedMsg extends ClientNetMsg {
* past the limit) then this constructor Throws that Exception to the * past the limit) then this constructor Throws that Exception to the
* caller. * caller.
*/ */
public AckBankWindowOpenedMsg(AbstractConnection origin, ByteBufferReader reader) { public CostOpenBankMsg(AbstractConnection origin, ByteBufferReader reader) {
super(Protocol.COSTTOOPENBANK, origin, reader); super(Protocol.COSTTOOPENBANK, origin, reader);
} }
Loading…
Cancel
Save