Skip to content

Commit c735b58

Browse files
mp911deschauder
mp911de
authored and
schauder
committed
Migrate code to Java 17 style.
Use var instead of explicit local types where applicable. Use pattern variable instead instanceof and cast. Prefer loops and nullable types over Stream and Optional. Convert classes to records where applicable. See #2465
1 parent d4036ec commit c735b58

File tree

463 files changed

+3670
-4014
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

463 files changed

+3670
-4014
lines changed

‎src/main/java/org/springframework/data/auditing/AnnotationAuditingMetadata.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ private void assertValidDateFieldType(Optional<Field> field) {
101101
return;
102102
}
103103

104-
Class<?> type = it.getType();
104+
var type = it.getType();
105105

106106
if (Jsr310Converters.supports(type)) {
107107
return;

‎src/main/java/org/springframework/data/auditing/AuditingHandlerSupport.java‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ protected final boolean isAuditable(Object source) {
107107
* @param auditor can be {@literal null}.
108108
* @param source must not be {@literal null}.
109109
*/
110-
<T> T markCreated(Auditor auditor, T source) {
110+
<T> T markCreated(Auditor<?> auditor, T source) {
111111

112112
Assert.notNull(source, "Source entity must not be null!");
113113

@@ -120,16 +120,16 @@ <T> T markCreated(Auditor auditor, T source) {
120120
* @param auditor
121121
* @param source
122122
*/
123-
<T> T markModified(Auditor auditor, T source) {
123+
<T> T markModified(Auditor<?> auditor, T source) {
124124

125125
Assert.notNull(source, "Source entity must not be null!");
126126

127127
return touch(auditor, source, false);
128128
}
129129

130-
private <T> T touch(Auditor auditor, T target, boolean isNew) {
130+
private <T> T touch(Auditor<?> auditor, T target, boolean isNew) {
131131

132-
Optional<AuditableBeanWrapper<T>> wrapper = factory.getBeanWrapperFor(target);
132+
var wrapper = factory.getBeanWrapperFor(target);
133133

134134
return wrapper.map(it -> {
135135

@@ -157,7 +157,7 @@ private <T> T touch(Auditor auditor, T target, boolean isNew) {
157157
* @param isNew
158158
* @return
159159
*/
160-
private void touchAuditor(Auditor auditor, AuditableBeanWrapper<?> wrapper, boolean isNew) {
160+
private void touchAuditor(Auditor<?> auditor, AuditableBeanWrapper<?> wrapper, boolean isNew) {
161161

162162
if(!auditor.isPresent()) {
163163
return;
@@ -185,7 +185,7 @@ private Optional<TemporalAccessor> touchDate(AuditableBeanWrapper<?> wrapper, bo
185185

186186
Assert.notNull(wrapper, "AuditableBeanWrapper must not be null!");
187187

188-
Optional<TemporalAccessor> now = dateTimeProvider.getNow();
188+
var now = dateTimeProvider.getNow();
189189

190190
Assert.notNull(now, () -> String.format("Now must not be null! Returned by: %s!", dateTimeProvider.getClass()));
191191

‎src/main/java/org/springframework/data/auditing/Auditor.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ public boolean equals(Object o) {
110110
if (o == null || getClass() != o.getClass())
111111
return false;
112112

113-
Auditor<?> auditor = (Auditor<?>) o;
113+
var auditor = (Auditor<?>) o;
114114

115115
return ObjectUtils.nullSafeEquals(value, auditor.value);
116116
}

‎src/main/java/org/springframework/data/auditing/DefaultAuditableBeanWrapperFactory.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ class DefaultAuditableBeanWrapperFactory implements AuditableBeanWrapperFactory
4646

4747
public DefaultAuditableBeanWrapperFactory() {
4848

49-
DefaultFormattingConversionService conversionService = new DefaultFormattingConversionService();
49+
var conversionService = new DefaultFormattingConversionService();
5050

5151
Jsr310Converters.getConvertersToRegister().forEach(conversionService::addConverter);
5252

@@ -75,7 +75,7 @@ public <T> Optional<AuditableBeanWrapper<T>> getBeanWrapperFor(T source) {
7575
return (AuditableBeanWrapper<T>) new AuditableInterfaceBeanWrapper(conversionService, (Auditable<Object, ?, TemporalAccessor>) it);
7676
}
7777

78-
AnnotationAuditingMetadata metadata = AnnotationAuditingMetadata.getMetadata(it.getClass());
78+
var metadata = AnnotationAuditingMetadata.getMetadata(it.getClass());
7979

8080
if (metadata.isAuditable()) {
8181
return new ReflectionAuditingBeanWrapper<T>(conversionService, it);
@@ -215,7 +215,7 @@ protected Object getDateValueToSet(TemporalAccessor value, Class<?> targetType,
215215
value.getClass(), targetType));
216216
}
217217

218-
Date date = conversionService.convert(value, Date.class);
218+
var date = conversionService.convert(value, Date.class);
219219
return conversionService.convert(date, targetType);
220220
}
221221

@@ -317,7 +317,7 @@ public Optional<TemporalAccessor> getLastModifiedDate() {
317317

318318
return getAsTemporalAccessor(metadata.getLastModifiedDateField().map(field -> {
319319

320-
Object value = org.springframework.util.ReflectionUtils.getField(field, target);
320+
var value = org.springframework.util.ReflectionUtils.getField(field, target);
321321
return value instanceof Optional ? ((Optional<?>) value).orElse(null) : value;
322322

323323
}), TemporalAccessor.class);

‎src/main/java/org/springframework/data/auditing/IsNewAwareAuditingHandler.java‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
package org.springframework.data.auditing;
1717

1818
import org.springframework.data.mapping.PersistentEntity;
19-
import org.springframework.data.mapping.PersistentProperty;
2019
import org.springframework.data.mapping.context.MappingContext;
2120
import org.springframework.data.mapping.context.PersistentEntities;
2221
import org.springframework.data.support.IsNewStrategy;
@@ -62,7 +61,7 @@ public Object markAudited(Object object) {
6261
return object;
6362
}
6463

65-
PersistentEntity<?, ? extends PersistentProperty<?>> entity = entities
64+
var entity = entities
6665
.getRequiredPersistentEntity(object.getClass());
6766

6867
return entity.isNew(object) ? markCreated(object) : markModified(object);

‎src/main/java/org/springframework/data/auditing/MappingAuditableBeanWrapperFactory.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ public <T> Optional<AuditableBeanWrapper<T>> getBeanWrapperFor(T source) {
8585

8686
return entities.mapOnContext(it.getClass(), (context, entity) -> {
8787

88-
MappingAuditingMetadata metadata = metadataCache.computeIfAbsent(it.getClass(),
88+
var metadata = metadataCache.computeIfAbsent(it.getClass(),
8989
key -> new MappingAuditingMetadata(context, it.getClass()));
9090

9191
return Optional.<AuditableBeanWrapper<T>> ofNullable(metadata.isAuditable() //
@@ -229,7 +229,7 @@ public Object setLastModifiedBy(Object value) {
229229
@Override
230230
public Optional<TemporalAccessor> getLastModifiedDate() {
231231

232-
Optional<Object> firstValue = metadata.lastModifiedDatePaths.getFirst() //
232+
var firstValue = metadata.lastModifiedDatePaths.getFirst() //
233233
.map(accessor::getProperty);
234234

235235
return getAsTemporalAccessor(firstValue, TemporalAccessor.class);
@@ -266,7 +266,7 @@ private TemporalAccessor setDateProperty(
266266

267267
property.forEach(it -> {
268268

269-
Class<?> type = it.getRequiredLeafProperty().getType();
269+
var type = it.getRequiredLeafProperty().getType();
270270

271271
this.accessor.setProperty(it, getDateValueToSet(value, type, accessor.getBean()), OPTIONS);
272272
});

‎src/main/java/org/springframework/data/auditing/ReactiveIsNewAwareAuditingHandler.java‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818
import reactor.core.publisher.Mono;
1919

2020
import org.springframework.data.mapping.PersistentEntity;
21-
import org.springframework.data.mapping.PersistentProperty;
2221
import org.springframework.data.mapping.context.MappingContext;
2322
import org.springframework.data.mapping.context.PersistentEntities;
2423
import org.springframework.data.support.IsNewStrategy;
@@ -62,7 +61,7 @@ public Mono<Object> markAudited(Object object) {
6261
return Mono.just(object);
6362
}
6463

65-
PersistentEntity<?, ? extends PersistentProperty<?>> entity = entities
64+
var entity = entities
6665
.getRequiredPersistentEntity(object.getClass());
6766

6867
return entity.isNew(object) ? markCreated(object) : markModified(object);

‎src/main/java/org/springframework/data/auditing/config/AnnotationAuditingConfiguration.java‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
package org.springframework.data.auditing.config;
1717

1818
import java.lang.annotation.Annotation;
19-
import java.util.Map;
2019

2120
import org.springframework.core.annotation.AnnotationAttributes;
2221
import org.springframework.core.type.AnnotationMetadata;
@@ -47,7 +46,7 @@ public AnnotationAuditingConfiguration(AnnotationMetadata metadata, Class<? exte
4746
Assert.notNull(metadata, "AnnotationMetadata must not be null!");
4847
Assert.notNull(annotation, "Annotation must not be null!");
4948

50-
Map<String, Object> attributesSource = metadata.getAnnotationAttributes(annotation.getName());
49+
var attributesSource = metadata.getAnnotationAttributes(annotation.getName());
5150

5251
if (attributesSource == null) {
5352
throw new IllegalArgumentException(String.format(MISSING_ANNOTATION_ATTRIBUTES, annotation, metadata));

‎src/main/java/org/springframework/data/auditing/config/AuditingBeanDefinitionRegistrarSupport.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ public void registerBeanDefinitions(AnnotationMetadata annotationMetadata, BeanD
6060
Assert.notNull(annotationMetadata, "AnnotationMetadata must not be null!");
6161
Assert.notNull(registry, "BeanDefinitionRegistry must not be null!");
6262

63-
AbstractBeanDefinition ahbd = registerAuditHandlerBeanDefinition(registry, getConfiguration(annotationMetadata));
63+
var ahbd = registerAuditHandlerBeanDefinition(registry, getConfiguration(annotationMetadata));
6464
registerAuditListenerBeanDefinition(ahbd, registry);
6565
}
6666

@@ -77,7 +77,7 @@ private AbstractBeanDefinition registerAuditHandlerBeanDefinition(BeanDefinition
7777
Assert.notNull(registry, "BeanDefinitionRegistry must not be null!");
7878
Assert.notNull(configuration, "AuditingConfiguration must not be null!");
7979

80-
AbstractBeanDefinition ahbd = getAuditHandlerBeanDefinitionBuilder(configuration).getBeanDefinition();
80+
var ahbd = getAuditHandlerBeanDefinitionBuilder(configuration).getBeanDefinition();
8181
registry.registerBeanDefinition(getAuditingHandlerBeanName(), ahbd);
8282
return ahbd;
8383
}
@@ -178,10 +178,10 @@ protected void registerInfrastructureBeanWithId(AbstractBeanDefinition definitio
178178

179179
private BeanDefinition createLazyInitTargetSourceBeanDefinition(String auditorAwareRef) {
180180

181-
BeanDefinitionBuilder targetSourceBuilder = rootBeanDefinition(LazyInitTargetSource.class);
181+
var targetSourceBuilder = rootBeanDefinition(LazyInitTargetSource.class);
182182
targetSourceBuilder.addPropertyValue("targetBeanName", auditorAwareRef);
183183

184-
BeanDefinitionBuilder builder = rootBeanDefinition(ProxyFactoryBean.class);
184+
var builder = rootBeanDefinition(ProxyFactoryBean.class);
185185
builder.addPropertyValue("targetSource", targetSourceBuilder.getBeanDefinition());
186186

187187
return builder.getBeanDefinition();

‎src/main/java/org/springframework/data/auditing/config/AuditingHandlerBeanDefinitionParser.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -95,11 +95,11 @@ protected boolean shouldGenerateId() {
9595
@Override
9696
protected void doParse(Element element, BeanDefinitionBuilder builder) {
9797

98-
final BeanDefinitionBuilder persistentEntities = rootBeanDefinition(PersistentEntitiesFactoryBean.class);
98+
final var persistentEntities = rootBeanDefinition(PersistentEntitiesFactoryBean.class);
9999
persistentEntities.addConstructorArgReference(mappingContextBeanName);
100100
builder.addConstructorArgValue(persistentEntities.getBeanDefinition());
101101

102-
String auditorAwareRef = element.getAttribute(AUDITOR_AWARE_REF);
102+
var auditorAwareRef = element.getAttribute(AUDITOR_AWARE_REF);
103103

104104
if (StringUtils.hasText(auditorAwareRef)) {
105105
builder.addPropertyValue("auditorAware", createLazyInitTargetSourceBeanDefinition(auditorAwareRef));
@@ -123,10 +123,10 @@ protected String resolveId(Element element, AbstractBeanDefinition definition, P
123123

124124
private BeanDefinition createLazyInitTargetSourceBeanDefinition(String auditorAwareRef) {
125125

126-
BeanDefinitionBuilder targetSourceBuilder = rootBeanDefinition(LazyInitTargetSource.class);
126+
var targetSourceBuilder = rootBeanDefinition(LazyInitTargetSource.class);
127127
targetSourceBuilder.addPropertyValue("targetBeanName", auditorAwareRef);
128128

129-
BeanDefinitionBuilder builder = rootBeanDefinition(ProxyFactoryBean.class);
129+
var builder = rootBeanDefinition(ProxyFactoryBean.class);
130130
builder.addPropertyValue("targetSource", targetSourceBuilder.getBeanDefinition());
131131

132132
return builder.getBeanDefinition();

0 commit comments

Comments
 (0)