DATAJDBC-545 - Fix support of setting properties via constructor.

Author: schauder

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

@@ -255,6 +255,17 @@ public void changeReferencedEntity() {
 assertThat(manual.content).isEqualTo("new content");
 }
 
+@Test // DATAJDBC-545
+public void setIdViaConstructor() {
+
+WithCopyConstructor entity = new WithCopyConstructor(null, "Alfred");
+
+WithCopyConstructor saved = template.save(entity);
+
+assertThat(saved).isNotEqualTo(entity);
+assertThat(saved.id).isNotNull();
+}
+
 private static LegoSet createLegoSet(Manual manual) {
 
 return new LegoSet(null, "Star Destroyer", manual, null);
@@ -296,6 +307,17 @@ static class Author {
 String name;
 }
 
+static class WithCopyConstructor {
+@Id
+private final Long id;
+private final String name;
+
+WithCopyConstructor(Long id, String name) {
+this.id = id;
+this.name = name;
+}
+}
+
 @Configuration
 @Import(TestConfiguration.class)
 static class Config {

spring-data-jdbc/src/test/resources/org.springframework.data.jdbc.core/ImmutableAggregateTemplateHsqlIntegrationTests-hsql.sql

@@ -23,3 +23,9 @@ CREATE TABLE AUTHOR
 ALTER TABLE AUTHOR
  ADD FOREIGN KEY (LEGO_SET)
  REFERENCES LEGO_SET (id);
+
+CREATE TABLE WITH_COPY_CONSTRUCTOR
+(
+ ID BIGINT GENERATED BY DEFAULT AS IDENTITY (START WITH 1) PRIMARY KEY,
+ NAME VARCHAR(30)
+);

spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/RelationalPersistentEntityImpl.java

@@ -105,13 +105,4 @@ public SqlIdentifier getIdColumn() {
 public String toString() {
 return String.format("JdbcPersistentEntityImpl<%s>", getType());
 }
-
-/*
- * (non-Javadoc)
- * @see org.springframework.data.mapping.model.BasicPersistentEntity#setPersistentPropertyAccessorFactory(org.springframework.data.mapping.model.PersistentPropertyAccessorFactory)
- */
-@Override
-public void setPersistentPropertyAccessorFactory(PersistentPropertyAccessorFactory factory) {
-
-}
 }