DATACMNS-447 - Soften too strict strickt-check in MappingContext.

Previously we directly threw a MappingException in MappingContext.getPersistentEntity(TypeInformation) when the MappingContext was in strict mode and we were given a TypeInformation for which we didn't have a PersistentEntity already.

We now first check whether we should actually create a PersistentEntity for the given TypeInformation if no PersistentEntity is available, before throwing a  MappingException - if we should not then we simply return null (e.g. for simple types like Integer or Double).

Original pull requests: #66, #71.

Author: jexp

src/main/java/org/springframework/data/mapping/context/AbstractMappingContext.java

@@ -61,6 +61,8 @@
  * @param P the concrete {@link PersistentProperty} type the {@link MappingContext} implementation creates
  * @author Jon Brisbin <jbrisbin@vmware.com>
  * @author Oliver Gierke
+ * @author Michael Hunger
+ * @author Thomas Darimont
  */
 public abstract class AbstractMappingContext<E extends MutablePersistentEntity<?, P>, P extends PersistentProperty<P>>
 		implements MappingContext<E, P>, ApplicationEventPublisherAware, InitializingBean {
@@ -159,14 +161,14 @@ public E getPersistentEntity(TypeInformation<?> type) {
 			read.unlock();
 		}
 
-		if (strict) {
-			throw new MappingException("Unknown persistent entity " + type);
-		}
-
 		if (!shouldCreatePersistentEntityFor(type)) {
 			return null;
 		}
 
+		if (strict) {
+			throw new MappingException("Unknown persistent entity " + type);
+		}
+
 		return addPersistentEntity(type);
 	}
 

src/test/java/org/springframework/data/mapping/context/AbstractMappingContextUnitTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2012 the original author or authors.
+ * Copyright 2011-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -42,6 +42,7 @@
  * Unit test for {@link AbstractMappingContext}.
  * 
  * @author Oliver Gierke
+ * @author Thomas Darimont
  */
 public class AbstractMappingContextUnitTests {
 
@@ -213,6 +214,16 @@ public void exposesCopyOfPersistentEntitiesToAvoidConcurrentModificationExceptio
 		}
 	}
 
+	/**
+	 * @see DATACMNS-447
+	 */
+	@Test
+	public void shouldReturnNullForSimpleTypesIfInStrictIsEnabled() {
+
+		context.setStrict(true);
+		assertThat(context.getPersistentEntity(Integer.class), is(nullValue()));
+	}
+
 	class Person {
 		String name;
 	}