skip all ignored json properties

to avoid to accidentally set them to null

Revert "fixed with skipping ignored properties"

This reverts commit 8d85c61.

fixed with skipping ignored properties
only working with jackson 2.8.7-SNAPSHOT
see FasterXML/jackson-databind#935

reproduce DATAREST-1006

Author: Mathias Düsterhöft

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

@@ -64,6 +64,9 @@ private MappedProperties(PersistentEntity<?, ?> entity, BeanDescription descript
 
 		for (BeanPropertyDefinition property : description.findProperties()) {
 
+			if (description.getIgnoredPropertyNames().contains(property.getName())) {
+				continue;
+			}
 			PersistentProperty<?> persistentProperty = entity.getPersistentProperty(property.getInternalName());
 
 			if (persistentProperty != null) {

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

@@ -15,9 +15,15 @@
  */
 package org.springframework.data.rest.webmvc.json;
 
-import static org.hamcrest.Matchers.*;
-import static org.junit.Assert.*;
-import static org.mockito.Mockito.*;
+import static com.fasterxml.jackson.annotation.JsonProperty.Access.READ_ONLY;
+import static org.hamcrest.Matchers.contains;
+import static org.hamcrest.Matchers.instanceOf;
+import static org.hamcrest.Matchers.is;
+import static org.hamcrest.Matchers.notNullValue;
+import static org.hamcrest.Matchers.nullValue;
+import static org.hamcrest.Matchers.sameInstance;
+import static org.junit.Assert.assertThat;
+import static org.mockito.Mockito.mock;
 
 import lombok.AllArgsConstructor;
 import lombok.EqualsAndHashCode;
@@ -58,6 +64,7 @@
 import com.fasterxml.jackson.annotation.JsonAutoDetect;
 import com.fasterxml.jackson.annotation.JsonAutoDetect.Visibility;
 import com.fasterxml.jackson.annotation.JsonIgnore;
+import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.PropertyNamingStrategy;
@@ -194,6 +201,22 @@ public void doesNotWipeIdAndVersionPropertyForPut() throws Exception {
 		assertThat(result.version, is(1L));
 	}
 
+	@Test //DATAREST-1006
+	public void doesNotWipeReadOnlyJsonPropertyForPut() throws Exception {
+
+		SampleUser sampleUser = new SampleUser("name", "password");
+		sampleUser.lastLogin = new Date();
+
+		ObjectMapper mapper = new ObjectMapper();
+		ObjectNode node = (ObjectNode) mapper.readTree("{ \"name\" : \"another\" }");
+
+		SampleUser result = reader.readPut(node, sampleUser, mapper);
+
+		assertThat(result.name, is("another"));
+		assertThat(result.password, notNullValue());
+		assertThat(result.lastLogin, notNullValue());
+	}
+
 	@Test // DATAREST-873
 	public void doesNotApplyInputToReadOnlyFields() throws Exception {
 
@@ -510,6 +533,9 @@ static class SampleUser {
 		@JsonIgnore String password;
 		Map<String, SampleUser> relatedUsers;
 
+		@JsonProperty(access = READ_ONLY)
+		private Date lastLogin;
+
 		public SampleUser(String name, String password) {
 
 			this.name = name;