Properly append all superfluous elements to an array during merge patch.

Fixes GH-2357.

Author: odrotbohm

spring-data-rest-webmvc/src/main/java/org/springframework/data/rest/webmvc/json/DomainObjectReader.java

@@ -390,7 +390,7 @@ private boolean handleArrayNode(ArrayNode array, Collection<Object> collection,
 						? rawValues.apply(current)
 						: mapper.treeToValue(jsonNode, componentType.getType()));
 
-				break;
+				continue;
 			}
 
 			Object next = value.next();

spring-data-rest-webmvc/src/test/java/org/springframework/data/rest/webmvc/json/DomainObjectReaderUnitTests.java

@@ -770,6 +770,40 @@ void replacesNestedArrays() throws Exception {
 		assertThat(result.map.get("array")).isEqualTo(new String[] { "second", "update" });
 	}
 
+	@Test // GH-2357
+	void addsElementToPreviouslyEmptyCollectionForPatch() throws Exception {
+
+		Child child = new Child();
+		child.items = new ArrayList<>();
+
+		JsonNode node = new ObjectMapper()
+				.readTree("{ \"items\" : [ { \"some\" : \"value\" }, { \"some\" : \"otherValue\" } ] }");
+
+		Child result = reader.doMerge((ObjectNode) node, child, new ObjectMapper());
+
+		assertThat(result.items).hasSize(2);
+		assertThat(result.items.get(0).some).isEqualTo("value");
+		assertThat(result.items.get(1).some).isEqualTo("otherValue");
+	}
+
+	@Test // GH-2357
+	void augmentsCollectionForPatch() throws Exception {
+
+		Child child = new Child();
+		child.items = new ArrayList<>(Arrays.asList(new Item("old")));
+
+		JsonNode node = new ObjectMapper()
+				.readTree(
+						"{ \"items\" : [ { \"some\" : \"value\" }, { \"some\" : \"otherValue\" }, { \"some\" : \"yetAnotherValue\" } ] }");
+
+		Child result = reader.doMerge((ObjectNode) node, child, new ObjectMapper());
+
+		assertThat(result.items).hasSize(3);
+		assertThat(result.items.get(0).some).isEqualTo("value");
+		assertThat(result.items.get(1).some).isEqualTo("otherValue");
+		assertThat(result.items.get(2).some).isEqualTo("yetAnotherValue");
+	}
+
 	@SuppressWarnings("unchecked")
 	private static <T> T as(Object source, Class<T> type) {