fix: make number parsing more strict

Author: samdozor

android-core/src/androidTest/kotlin/com.mparticle/MPUserTest.kt

@@ -61,11 +61,11 @@ class MPUserTest : BaseCleanStartedEachTest() {
                 ) {
                     assertEquals(6, userAttributes.size)
                     assertEquals("bar", userAttributes["foo"])
-                    assertEquals(123L, userAttributes["fooInt"])
-                    assertEquals(12345L, userAttributes["fooLong"])
+                    assertEquals(123, userAttributes["fooInt"])
+                    assertEquals(12345, userAttributes["fooLong"])
                     assertEquals(10.15, userAttributes["fooDouble"])
-                    assertEquals(-10L, userAttributes["fooNegInt"])
-                    assertEquals(-1010L, userAttributes["fooNegLong"])
+                    assertEquals(-10, userAttributes["fooNegInt"])
+                    assertEquals(-1010, userAttributes["fooNegLong"])
                     assertEquals(null, userAttributes["fooNull"])
                 }
             })
@@ -83,12 +83,12 @@ class MPUserTest : BaseCleanStartedEachTest() {
             incrementUserAttribute("foo", 3)
 
             android_test_hack()
-            assertEquals(4L, userAttributes["foo"])
+            assertEquals(4, userAttributes["foo"])
 
             // test negative increment
             incrementUserAttribute("foo", -2)
             android_test_hack()
-            assertEquals(2L, userAttributes["foo"])
+            assertEquals(2, userAttributes["foo"])
 
             // test remove incremented attribute
             removeUserAttribute("foo")
@@ -130,20 +130,20 @@ class MPUserTest : BaseCleanStartedEachTest() {
     fun testIncrementLongAttribute() {
         MParticle.getInstance()!!.Identity().currentUser!!.apply {
             assertTrue { getUserAttributes().isEmpty() }
-            setUserAttribute("foo", 10L)
+            setUserAttribute("foo", 10)
 
             android_test_hack()
             assertEquals(1, userAttributes.size)
-            assertEquals(10L, userAttributes["foo"])
-            incrementUserAttribute("foo", 37L)
+            assertEquals(10, userAttributes["foo"])
+            incrementUserAttribute("foo", 37)
 
             android_test_hack()
-            assertEquals(47L, userAttributes["foo"])
+            assertEquals(47, userAttributes["foo"])
 
             // test negative increment
-            incrementUserAttribute("foo", -21L)
+            incrementUserAttribute("foo", -21)
             android_test_hack()
-            assertEquals(26L, userAttributes["foo"])
+            assertEquals(26, userAttributes["foo"])
 
             // test remove incremented attribute
             removeUserAttribute("foo")

android-core/src/main/java/com/mparticle/internal/MPUtility.java

@@ -833,17 +833,20 @@ public static Number addNumbers(Number number1, Number number2) {
 
     public static Object toNumberOrString(String stringValue) {
         if (stringValue == null) {
-            return stringValue;
+            return null;
         }
         for (Character c : stringValue.toCharArray()) {
             if (!Character.isDigit(c) && c != '.' && c != '-') {
                 return stringValue;
             }
         }
         try {
-            return NumberFormat.getInstance().parse(stringValue);
-        } catch (ParseException e) {
-        }
+            return Integer.parseInt(stringValue);
+        } catch (NumberFormatException ignored){}
+        try {
+           return Double.parseDouble(stringValue);
+        } catch (NumberFormatException ignored){}
+
         return stringValue;
     }