Merge pull request #1 from johnprisco/numeric-strings

Prevent NumberFormatException when strings consist only of numbers

Author: terma

src/main/java/com/github/terma/sqlonjson/SqlOnJson.java

@@ -94,7 +94,8 @@ private static LinkedHashMap<String, ColumnType> getColumns(JsonTable jsonTable)
                     }
                 }
 
-                cls.put(part.getKey(), columnType);
+                ColumnType current = cls.get(part.getKey());
+                if (current == null || current != ColumnType.STRING) cls.put(part.getKey(), columnType);
             }
         }
         return cls;

src/test/java/com/github/terma/sqlonjson/SqlOnJsonTest.java

@@ -162,6 +162,18 @@ public void representEmbeddedObjectAsString() throws Exception {
         }
     }
 
+    @Test
+    public void representNumericStringAsString() throws Exception {
+        try (Connection c = sqlOnJson.convertPlain("{t:[{a:\"super\"},{a:\"5\"}]}")) {
+            ResultSet rs = c.prepareStatement("select * from t").executeQuery();
+            rs.next();
+            Assert.assertEquals("super", rs.getString("a"));
+            rs.next();
+            Assert.assertEquals("5", rs.getString("a"));
+            Assert.assertFalse(rs.next());
+        }
+    }
+
     @Test
     public void supportEmbeddedArrayToTable() throws Exception {
         try (Connection c = sqlOnJson.convertPlain("{orders:[{em:[{a:-7}]}]}")) {