fixed VPack serializing of array with null values (issue #88)

Author: Mark

ChangeLog

@@ -1,6 +1,7 @@
-v4.1.5 (2016-01-xx)
+v4.1.5 (2017-01-xx)
 ---------------------------
 * fixed VPack parsing of fields of type Object
+* fixed VPack serializing of array with null values (issue #88)
 
 v4.1.4 (2016-12-19)
 ---------------------------

src/main/java/com/arangodb/velocypack/VPack.java

@@ -579,7 +579,9 @@ private void serializeArray(
 		builder.add(name, ValueType.ARRAY);
 		for (int i = 0; i < Array.getLength(value); i++) {
 			final Object element = Array.get(value, i);
-			addValue(null, element.getClass(), element, builder, null, additionalFields);
+			if (element != null) {
+				addValue(null, element.getClass(), element, builder, null, additionalFields);
+			}
 		}
 		builder.close();
 	}

src/test/java/com/arangodb/velocypack/VPackSerializeDeserializeTest.java

@@ -677,6 +677,22 @@ public void toArray() throws VPackException {
 		}
 	}
 
+	@Test
+	public void fromArrayWithNull() {
+		final TestEntityArray entity = new TestEntityArray();
+		entity.a1 = new String[] { "foo", null };
+
+		final VPackSlice vpack = new VPack.Builder().build().serialize(entity);
+		assertThat(vpack, is(notNullValue()));
+		assertThat(vpack.isObject(), is(true));
+
+		final VPackSlice a1 = vpack.get("a1");
+		assertThat(a1.isArray(), is(true));
+		assertThat(a1.size(), is(1));
+		assertThat(a1.get(0).isString(), is(true));
+		assertThat(a1.get(0).getAsString(), is("foo"));
+	}
+
 	protected enum TestEnum {
 		A, B, C
 	}