JSONObject#getString() returns a string literal - "null" instead of null when the value is null

[yusuke]

This test case fails:

public void testNull() throws Exception {
    JSONObject j;

    j = new JSONObject("{\"message\":\"null\"}");
    assertFalse(j.isNull("message"));
    assertEquals("null", j.getString("message"));

    j = new JSONObject("{\"message\":null}");
    assertTrue(j.isNull("message"));
    assertEquals(null, j.getString("message")); // fail here: getString should return null instead of "null"
}

---

[douglascrockford]

Thanks.

[mjb-starleaf]

Was this issue fixed? If not, will it be? It's a correctness issue.

[stleary]

@mjb-starleaf This issue should have been fixed. Parsing a JSON doc with null as an object value should store a JSONObject.Null instance for the null value.