You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ju...@apache.org on 2013/06/27 21:29:00 UTC

svn commit: r1497518 - in /sling/trunk/bundles/commons/json/src: main/java/org/apache/sling/commons/json/JSONObject.java test/java/org/apache/sling/commons/json/JSONObjectTest.java

Author: justin
Date: Thu Jun 27 19:28:59 2013
New Revision: 1497518

URL: http://svn.apache.org/r1497518
Log:
SLING-2937 - using Long.valueOf() in getLong() to ensure correct parsing

Modified:
    sling/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/JSONObject.java
    sling/trunk/bundles/commons/json/src/test/java/org/apache/sling/commons/json/JSONObjectTest.java

Modified: sling/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/JSONObject.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/JSONObject.java?rev=1497518&r1=1497517&r2=1497518&view=diff
==============================================================================
--- sling/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/JSONObject.java (original)
+++ sling/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/JSONObject.java Thu Jun 27 19:28:59 2013
@@ -484,7 +484,8 @@ public class JSONObject {
     public long getLong(String key) throws JSONException {
         Object o = get(key);
         return o instanceof Number ?
-                ((Number)o).longValue() : (long)getDouble(key);
+                ((Number)o).longValue() :
+                Long.valueOf((String)o).longValue();
     }
 
 

Modified: sling/trunk/bundles/commons/json/src/test/java/org/apache/sling/commons/json/JSONObjectTest.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/commons/json/src/test/java/org/apache/sling/commons/json/JSONObjectTest.java?rev=1497518&r1=1497517&r2=1497518&view=diff
==============================================================================
--- sling/trunk/bundles/commons/json/src/test/java/org/apache/sling/commons/json/JSONObjectTest.java (original)
+++ sling/trunk/bundles/commons/json/src/test/java/org/apache/sling/commons/json/JSONObjectTest.java Thu Jun 27 19:28:59 2013
@@ -57,4 +57,10 @@ public class JSONObjectTest extends Test
         TestCase.assertTrue(obj.get(KEY).equals(null));
         TestCase.assertEquals("{\"" + KEY + "\":null}", obj.toString());
     }
+    
+    public void testParseLong() throws JSONException {
+        String jsonStr = "{\"longvalue\":\"13857270119014401\"}";
+        JSONObject obj = new JSONObject(jsonStr);
+        TestCase.assertEquals(13857270119014401L, obj.getLong("longvalue"));
+    }
 }