Skip to content

Commit 721c696

Browse files
DavideDSanne
authored andcommitted
[#1078] Fix LGTM regression
In ORM 5.6.2.Final, the `IdentifierGenerator` extends `Configurable` so we can skip some casting and `instance of` checks.
1 parent 9d83c71 commit 721c696

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

hibernate-reactive-core/src/main/java/org/hibernate/reactive/id/impl/ReactiveGeneratorWrapper.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@ public Serializable generate(SharedSessionContractImplementor session, Object ob
4848

4949
@Override
5050
public void registerExportables(Database database) {
51-
if (generator instanceof ExportableProducer) {
52-
((ExportableProducer) generator).registerExportables( database );
51+
if ( generator != null ) {
52+
generator.registerExportables( database );
5353
}
5454
if (reactiveGenerator instanceof ExportableProducer) {
5555
((ExportableProducer) reactiveGenerator).registerExportables( database );

hibernate-reactive-core/src/main/java/org/hibernate/reactive/id/impl/ReactiveIdentifierGeneratorFactory.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,7 @@ else if ( generator instanceof ReactiveIdentifierGenerator) {
6868
throw new MappingException( String.format( "Not an id generator [entity-name=%s]", entityName ) );
6969
}
7070

71-
if ( result instanceof Configurable) {
72-
( (Configurable) result ).configure( type, config, serviceRegistry );
73-
}
71+
result.configure( type, config, serviceRegistry );
7472

7573
return result;
7674
}

0 commit comments

Comments
 (0)