New JSON class changes

missionpinball · Jun 18, 2019 · 9a71e1b · 9a71e1b
1 parent 1aad0a1
commit 9a71e1b
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/Assets/BCP/Scripts/BcpMessage.cs b/Assets/BCP/Scripts/BcpMessage.cs
@@ -202,7 +202,7 @@ public static JSONNode ConvertBcpParameterStringToNode(string value)
             return new JSONBool(value.Substring(5));
 
         else if (value.ToLower().StartsWith("nonetype:"))
-            return new JSONNull();
+            return JSONNull.CreateOrGet();
 
         else if (value.ToLower().StartsWith("int:"))
             return new JSONNumber(value.Substring(4));
@@ -241,7 +241,7 @@ public static BcpMessage CreateFromRawMessage(string rawMessage)
             // BCP commands are not case sensitive so we convert to lower case
             // BCP parameter names are not case sensitive, but parameter values are
             bcpMessage.Command = rawMessage.Substring(0, messageDelimiterPos).Trim().ToLower();
-            rawMessage = rawMessage.Substring(rawMessage.IndexOf('?') + 1);
+            rawMessage = rawMessage.Substring(messageDelimiterPos + 1);
 
             foreach (string parameter in Regex.Split(rawMessage, "&"))
             {
@@ -261,7 +261,7 @@ public static BcpMessage CreateFromRawMessage(string rawMessage)
                         }
                         catch
                         {
-                            bcpMessage.Parameters.Add(name, new JSONNull());
+                            bcpMessage.Parameters.Add(name, JSONNull.CreateOrGet());
                         }
                     else
                     {
@@ -272,7 +272,7 @@ public static BcpMessage CreateFromRawMessage(string rawMessage)
                 else
                 {
                     // only one key with no value specified in query string
-                    bcpMessage.Parameters.Add(name, new JSONNull());
+                    bcpMessage.Parameters.Add(name, JSONNull.CreateOrGet());
                 }
             }