Merge pull request bitly#26 from Quantisan/feature/null-string-in-array

StringArray handle JSON null string as "" and add test for it

Author: mreiferson

simplejson.go

@@ -176,6 +176,10 @@ func (j *Json) StringArray() ([]string, error) {
 	}
 	retArr := make([]string, 0, len(arr))
 	for _, a := range arr {
+		if a == nil {
+			retArr = append(retArr, "")
+			continue
+		}
 		s, ok := a.(string)
 		if !ok {
 			return nil, err

simplejson_test.go

@@ -13,6 +13,7 @@ func TestSimplejson(t *testing.T) {
 	js, err := NewJson([]byte(`{
 		"test": {
 			"string_array": ["asdf", "ghjk", "zxcv"],
+			"string_array_null": ["abc", null, "efg"],
 			"array": [1, "2", 3],
 			"arraywithsubs": [{"subkeyone": 1},
 			{"subkeytwo": 2, "subkeythree": 3}],
@@ -78,6 +79,12 @@ func TestSimplejson(t *testing.T) {
 	assert.Equal(t, strs[1], "ghjk")
 	assert.Equal(t, strs[2], "zxcv")
 
+	strs2, err := js.Get("test").Get("string_array_null").StringArray()
+	assert.Equal(t, err, nil)
+	assert.Equal(t, strs2[0], "abc")
+	assert.Equal(t, strs2[1], "")
+	assert.Equal(t, strs2[2], "efg")
+
 	gp, _ := js.GetPath("test", "string").String()
 	assert.Equal(t, "simplejson", gp)