EnversRevisionRepositoryFactoryBean doesn't use custom RevisionEntity about spring-data-envers HOT 7 CLOSED

The EnversRepositoryFactoryBean already exposes a property to customize the revision entity class. If this property is set the ReflectionRevisionEntityInformation information will be used. So the problem currently is that the infrastructure does not know about your special RevEntity class, and the namespace configuration you probably use does not expose the property. You should be able to workaround the issue by extending EnversRepositoryFactoryBean as follows:

public class CustomEnversRevisionRepositoryFactoryBean extends EnversRevisionRepositoryFactoryBean {
  public FooCustomEnversRevisionRepositoryFactoryBean() {
    setRevisionEntityClass(RevEntity.class);
  }
}

then you can alter your Spring Data JPA config as follows:

<jpa:repositories base-package="…" factory-class="….CustomEnversRevisionRepositoryFactoryBean" />

We'll probably add a custom <envers:repositories /> element to the Spring Data Envers project that will expose the revision entity class attribute directly to ease the setup.

from spring-data-envers.

We just created #289.

from spring-data-envers.

Thnx, extending is a good idea.
I was already wondering how to use the setRevisionEntityClass()-Method in the EnversRepositoryFactoryBean, as it is not used as a plain bean ref but as a factory-class....

from spring-data-envers.

public class CustomEnversRevisionRepositoryFactoryBean extends EnversRevisionRepositoryFactoryBean { public FooCustomEnversRevisionRepositoryFactoryBean() { setRevisionEntityClass(RevEntity.class); } }

In the version of hibernate-envers 5.1.1.Final and spring-data-envers 1.1.15.RELEASE this code doesn't work since EnversRevisionRepositoryFactoryBean has no default constructor (and keeping in mind that "Foo" in constructor name is a typo).
This code worked for me:

public class CustomEnversRevisionRepositoryFactoryBean extends EnversRevisionRepositoryFactoryBean {
    public CustomEnversRevisionRepositoryFactoryBean(Class repositoryInterface) {
        super(repositoryInterface);
        setRevisionEntityClass(AbonRevision.class);
    }
}

from spring-data-envers.

We'll probably add a custom <envers:repositories /> element to the Spring Data Envers project that will expose the revision entity class attribute directly to ease the setup.

Hi @odrotbohm, I couldn't find anything about it so I guess this was never added. Is it something you would still consider or is subclassing the preferred approach?

from spring-data-envers.

We have no plans to extend our XML based configurations at the moment.

from spring-data-envers.

Thanks for the fast feedback!

What about something for Java config? I could imagine a custom annotation to avoid the EnversRevisionRepositoryFactoryBean custom class wired into @EnableJpaRepositories.

I'm happy to contribute if the idea is acceptable.

from spring-data-envers.