Add constants for message action type.

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

@@ -13,6 +13,10 @@ import engine.objects.PlayerCharacter;
 
 public class PetitionReceivedMsgHandler extends AbstractClientMsgHandler {
 
+    public static final int PETITION_NEW = 1;
+    public static final int PETITION_CANCEL = 2;
+    public static final int PETITION_CLOSE = 4;
+
     public PetitionReceivedMsgHandler() {
         super(PetitionReceivedMsg.class);
     }
@@ -35,18 +39,16 @@ public class PetitionReceivedMsgHandler extends AbstractClientMsgHandler {
 
         Petition petition = new Petition(msg, origin);
 
-        if (petition == null)
-            return false;
-
         try {
             // Write petition to database
 
-            DbManager.PetitionQueries.WRITE_PETITION_TO_TABLE(petition);
+            if (petitionReceivedMsg.petition == PETITION_NEW)
+                DbManager.PetitionQueries.WRITE_PETITION_TO_TABLE(petition);
 
             // Close the petition window
 
-            if (petitionReceivedMsg.petition != 2)
-                petitionReceivedMsg.petition = 4;
+            if (petitionReceivedMsg.petition == PETITION_NEW)
+                petitionReceivedMsg.petition = PETITION_CLOSE;
 
             petitionReceivedMsg.unknownByte01 = 0;
             petitionReceivedMsg.unknown04 = 0;

src/engine/net/client/msg/PetitionReceivedMsg.java

@@ -15,13 +15,11 @@ import engine.net.ByteBufferReader;
 import engine.net.ByteBufferWriter;
 import engine.net.client.Protocol;
 
+import static engine.net.client.handlers.PetitionReceivedMsgHandler.PETITION_NEW;
+
 
 public class PetitionReceivedMsg extends ClientNetMsg {
 
-    // TODO pull these statics out into SBEmuStatics.java
-    public static final int PETITION_NEW = 1;
-    public static final int PETITION_CANCEL = 2;
-
     public int petition;
     public int unknown01;
     public int unknown02;
@@ -97,7 +95,7 @@ public class PetitionReceivedMsg extends ClientNetMsg {
             this.language = reader.getString();
             this.unknown07 = reader.getInt();
             this.message = reader.getUnicodeString();
-        } else if (petition == PETITION_CANCEL) {
+        } else {
             this.unknown01 = reader.getInt();
             this.unknown02 = reader.getInt();
             this.unknownByte01 = reader.get();