DATAJDBC-341 - fixed, added tests, some polish.

Author: THD-Thomas-Lang

spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/BasicJdbcConverter.java

@@ -324,8 +324,6 @@ private T populateProperties(T instance, @Nullable Object idValue) {
                 try {
                     if (resultSet.findColumn(property.getColumnName()) > 0) {
                         propertyAccessor.setProperty(property, readOrLoadProperty(idValue, property));
-                    } else {
-                        System.out.println(String.format("Column %s not found in result set. Use standard values.", property.getColumnName()));
                     }
                 } catch (SQLException e) {
                     e.printStackTrace();
@@ -381,15 +379,12 @@ private Object readFrom(RelationalPersistentProperty property) {
             }
             Object value = null;
             try {
-				// check if property is in the result set
+                // check if property is in the result set
                 // if not - leave it out
                 // DATAJDBC-341
                 if (resultSet.findColumn(property.getColumnName()) > 0) {
                     value = getObjectFromResultSet(path.extendBy(property).getColumnAlias());
-                } else {
-                    System.out.println(String.format("Column %s not found in result set. Use standard values.", property.getColumnName()));
                 }
-
             } catch (SQLException e) {
                 e.printStackTrace();
             }

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/convert/EntityRowMapperUnitTests.java

@@ -128,6 +128,20 @@ public void mapNotNeededPrimitiveTypePropertiesToNull() throws SQLException {
 
 	}
 
+    @Test // DATAJDBC-341
+    public void mapNotNeededMixedTypePropertiesToNull() throws SQLException {
+        ResultSet rs = mockResultSet(singletonList("id"), //
+                ID_FOR_ENTITY_NOT_REFERENCING_MAP, "alpha");
+        rs.next();
+        TrivialMapPropertiesToNullIfNotNeeded extracted = createRowMapper(TrivialMapPropertiesToNullIfNotNeeded.class).mapRow(rs, 1);
+
+        assertThat(extracted) //
+                .isNotNull() //
+                .extracting(e -> e.id, e -> e.age, e -> e.isSupreme) //
+                .containsExactly(ID_FOR_ENTITY_NOT_REFERENCING_MAP, 0, null);
+
+    }
+
     @Test // DATAJDBC-181
     public void namingStrategyGetsHonored() throws SQLException {
 
@@ -480,6 +494,9 @@ static class TrivialMapPropertiesToNullIfNotNeeded {
         @Id
         Long id;
         int age;
+        String phone;
+        Boolean isSupreme;
+        long referenceToCustomer;
     }
 
     static class OneToOne {