diff --git a/spring-data-envers/src/main/java/org/springframework/data/envers/repository/support/EnversRevisionRepositoryImpl.java b/spring-data-envers/src/main/java/org/springframework/data/envers/repository/support/EnversRevisionRepositoryImpl.java index 259f510ed..ef55a7880 100755 --- a/spring-data-envers/src/main/java/org/springframework/data/envers/repository/support/EnversRevisionRepositoryImpl.java +++ b/spring-data-envers/src/main/java/org/springframework/data/envers/repository/support/EnversRevisionRepositoryImpl.java @@ -169,8 +169,8 @@ public class EnversRevisionRepositoryImpl orderMapped = (pageable.getSort() instanceof RevisionSort) ? - Collections.singletonList(mapRevisionSort((RevisionSort) pageable.getSort())) : + List orderMapped = (pageable.getSort() instanceof RevisionSort revisionSort) ? + Collections.singletonList(mapRevisionSort(revisionSort)) : mapPropertySort(pageable.getSort()); orderMapped.forEach(baseQuery::addOrder); @@ -232,8 +232,8 @@ public class EnversRevisionRepositoryImpl createRevisionMetadata() { - return metadata instanceof DefaultRevisionEntity // - ? new DefaultRevisionMetadata((DefaultRevisionEntity) metadata, revisionType) // + return metadata instanceof DefaultRevisionEntity defaultRevisionEntity // + ? new DefaultRevisionMetadata(defaultRevisionEntity, revisionType) // : new AnnotationRevisionMetadata<>(Hibernate.unproxy(metadata), RevisionNumber.class, RevisionTimestamp.class, revisionType); } diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/provider/HibernateUtils.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/provider/HibernateUtils.java index 92983f134..98e65c2d9 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/provider/HibernateUtils.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/provider/HibernateUtils.java @@ -45,8 +45,8 @@ public abstract class HibernateUtils { try { // Try the new Hibernate implementation first - if (query instanceof SqmQuery) { - return ((SqmQuery) query).getSqmStatement().toHqlString(); + if (query instanceof SqmQuery sqmQuery) { + return sqmQuery.getSqmStatement().toHqlString(); } // Couple of cases in which this still breaks, see HHH-15389 diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/provider/JpaClassUtils.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/provider/JpaClassUtils.java index 771979c56..10de0aad2 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/provider/JpaClassUtils.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/provider/JpaClassUtils.java @@ -48,8 +48,8 @@ abstract class JpaClassUtils { EntityManager entityManagerToUse = em; Object delegate = em.getDelegate(); - if (delegate instanceof EntityManager) { - entityManagerToUse = (EntityManager) delegate; + if (delegate instanceof EntityManager delegateEntityManager) { + entityManagerToUse = delegateEntityManager; } return isOfType(entityManagerToUse, type, entityManagerToUse.getClass().getClassLoader()); diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/cdi/JpaRepositoryExtension.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/cdi/JpaRepositoryExtension.java index 9a218c4ab..2e345c22a 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/cdi/JpaRepositoryExtension.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/cdi/JpaRepositoryExtension.java @@ -68,7 +68,7 @@ public class JpaRepositoryExtension extends CdiRepositoryExtensionSupport { Bean bean = processBean.getBean(); for (Type type : bean.getTypes()) { // Check if the bean is an EntityManager. - if (type instanceof Class && EntityManager.class.isAssignableFrom((Class) type)) { + if (type instanceof Class classType && EntityManager.class.isAssignableFrom(classType)) { Set qualifiers = new HashSet<>(bean.getQualifiers()); if (bean.isAlternative() || !entityManagers.containsKey(qualifiers)) { LOGGER.debug(String.format("Discovered '%s' with qualifiers %s", EntityManager.class.getName(), qualifiers)); diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/JpaQueryTransformerSupport.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/JpaQueryTransformerSupport.java index b791c4771..13ad183fd 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/JpaQueryTransformerSupport.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/JpaQueryTransformerSupport.java @@ -85,7 +85,7 @@ class JpaQueryTransformerSupport { */ private void checkSortExpression(Sort.Order order) { - if (order instanceof JpaSort.JpaOrder && ((JpaSort.JpaOrder) order).isUnsafe()) { + if (order instanceof JpaSort.JpaOrder jpaOrder && jpaOrder.isUnsafe()) { return; } diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/Meta.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/Meta.java index a3e07cf14..771d8b582 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/Meta.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/Meta.java @@ -103,7 +103,7 @@ public class Meta { values = new LinkedHashMap<>(2); } - if (value == null || (value instanceof String && !StringUtils.hasText((String) value))) { + if (value == null || (value instanceof String stringValue && !StringUtils.hasText(stringValue))) { this.values.remove(key); } this.values.put(key, value); diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/ParameterMetadataProvider.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/ParameterMetadataProvider.java index b3737940f..52cc95156 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/ParameterMetadataProvider.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/ParameterMetadataProvider.java @@ -274,9 +274,8 @@ class ParameterMetadataProvider { return null; } - if (value instanceof Collection) { + if (value instanceof Collection collection) { - Collection collection = (Collection) value; return collection.isEmpty() ? null : collection; } diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/QueryUtils.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/QueryUtils.java index 0d2c65072..0b781b33d 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/QueryUtils.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/QueryUtils.java @@ -846,12 +846,10 @@ public abstract class QueryUtils { return true; } - if (!(propertyPathModel instanceof Attribute)) { + if (!(propertyPathModel instanceof Attribute attribute)) { return false; } - Attribute attribute = (Attribute) propertyPathModel; - // not a persistent attribute type association (@OneToOne, @ManyToOne) if (!ASSOCIATION_TYPES.containsKey(attribute.getPersistentAttributeType())) { return false; @@ -882,11 +880,11 @@ public abstract class QueryUtils { Member member = attribute.getJavaMember(); - if (!(member instanceof AnnotatedElement)) { + if (!(member instanceof AnnotatedElement annotatedMember)) { return defaultValue; } - Annotation annotation = AnnotationUtils.getAnnotation((AnnotatedElement) member, associationAnnotation); + Annotation annotation = AnnotationUtils.getAnnotation(annotatedMember, associationAnnotation); return annotation == null ? defaultValue : (T) AnnotationUtils.getValue(annotation, propertyName); } @@ -945,7 +943,7 @@ public abstract class QueryUtils { */ static void checkSortExpression(Order order) { - if (order instanceof JpaOrder && ((JpaOrder) order).isUnsafe()) { + if (order instanceof JpaOrder jpaOrder && jpaOrder.isUnsafe()) { return; } diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/EntityManagerBeanDefinitionRegistrarPostProcessor.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/EntityManagerBeanDefinitionRegistrarPostProcessor.java index f4e953576..cf3ed03ff 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/EntityManagerBeanDefinitionRegistrarPostProcessor.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/EntityManagerBeanDefinitionRegistrarPostProcessor.java @@ -64,12 +64,11 @@ public class EntityManagerBeanDefinitionRegistrarPostProcessor implements BeanFa BeanFactory definitionFactory = definition.getBeanFactory(); - if (!(definitionFactory instanceof BeanDefinitionRegistry)) { + if (!(definitionFactory instanceof BeanDefinitionRegistry definitionRegistry)) { continue; } String entityManagerBeanName = "jpaSharedEM_AWC_" + definition.getBeanName(); - BeanDefinitionRegistry definitionRegistry = (BeanDefinitionRegistry) definitionFactory; if (!beanFactory.containsBeanDefinition(entityManagerBeanName) && !definitionRegistry.containsBeanDefinition(entityManagerBeanName)) { diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/Querydsl.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/Querydsl.java index 25fb4abec..9cc1017a4 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/Querydsl.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/Querydsl.java @@ -137,8 +137,8 @@ public class Querydsl { return query; } - if (sort instanceof QSort) { - return addOrderByFrom((QSort) sort, query); + if (sort instanceof QSort qsort) { + return addOrderByFrom(qsort, query); } return addOrderByFrom(sort, query); diff --git a/spring-data-jpa/src/main/java/org/springframework/data/jpa/util/BeanDefinitionUtils.java b/spring-data-jpa/src/main/java/org/springframework/data/jpa/util/BeanDefinitionUtils.java index 779be1445..322cf3095 100644 --- a/spring-data-jpa/src/main/java/org/springframework/data/jpa/util/BeanDefinitionUtils.java +++ b/spring-data-jpa/src/main/java/org/springframework/data/jpa/util/BeanDefinitionUtils.java @@ -107,8 +107,8 @@ public final class BeanDefinitionUtils { BeanFactory parentBeanFactory = beanFactory.getParentBeanFactory(); - if (parentBeanFactory instanceof ConfigurableListableBeanFactory) { - definitions.addAll(getEntityManagerFactoryBeanDefinitions((ConfigurableListableBeanFactory) parentBeanFactory)); + if (parentBeanFactory instanceof ConfigurableListableBeanFactory parentConfigurableListableBeanFactory) { + definitions.addAll(getEntityManagerFactoryBeanDefinitions(parentConfigurableListableBeanFactory)); } return definitions; @@ -157,8 +157,8 @@ public final class BeanDefinitionUtils { BeanFactory parentBeanFactory = beanFactory.getParentBeanFactory(); - if (parentBeanFactory instanceof ConfigurableListableBeanFactory) { - return getBeanDefinition(name, (ConfigurableListableBeanFactory) parentBeanFactory); + if (parentBeanFactory instanceof ConfigurableListableBeanFactory parentConfigurableListableBeanFactory) { + return getBeanDefinition(name, parentConfigurableListableBeanFactory); } throw o_O;