DATAMONGO-617 - Fixed potential NullPointerException in MongoTemplate…

….insert(…).

If MongoTemplate.insert(…) was called with a Mongo-simple type (such as a raw DBBobject) it caused a NullPointerException during the lookup of a version property. This is now fixed by correcting the guard.

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/MongoTemplate.java

@@ -666,7 +666,7 @@ private void initializeVersionProperty(Object entity) {
 
 		MongoPersistentEntity<?> mongoPersistentEntity = getPersistentEntity(entity.getClass());
 
-		if (mongoPersistentEntity == null || mongoPersistentEntity.hasVersionProperty()) {
+		if (mongoPersistentEntity != null && mongoPersistentEntity.hasVersionProperty()) {
 			BeanWrapper<PersistentEntity<Object, ?>, Object> wrapper = BeanWrapper.create(entity, null);
 			wrapper.setProperty(mongoPersistentEntity.getVersionProperty(), 0);
 		}

spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/MongoTemplateTests.java

@@ -1318,6 +1318,18 @@ public void optimisticLockingHandlingWithExistingId() {
 		template.save(person);
 	}
 
+	/**
+	 * @see DATAMONGO-617
+	 */
+	@Test
+	public void doesNotFailOnVersionInitForUnversionedEntity() {
+
+		DBObject dbObject = new BasicDBObject();
+		dbObject.put("firstName", "Oliver");
+
+		template.insert(dbObject, template.determineCollectionName(PersonWithVersionPropertyOfTypeInteger.class));
+	}
+
 	/**
 	 * @see DATAMONGO-539
 	 */