Skip to content

Commit 9d3540c

Browse files
schauderodrotbohm
authored andcommitted
DATACMNS-1259 - Polishing.
Refactored lambdas to method references. Simplified type parameter. Added comment to test to indicate related issue. Original pull request: #273.
1 parent 5251f72 commit 9d3540c

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ public Object setCreatedBy(Object value) {
113113
public TemporalAccessor setCreatedDate(TemporalAccessor value) {
114114

115115
auditable.setCreatedDate(
116-
getAsTemporalAccessor(Optional.of(value), type).orElseThrow(() -> new IllegalStateException()));
116+
getAsTemporalAccessor(Optional.of(value), type).orElseThrow(IllegalStateException::new));
117117

118118
return value;
119119
}
@@ -147,7 +147,7 @@ public Optional<TemporalAccessor> getLastModifiedDate() {
147147
public TemporalAccessor setLastModifiedDate(TemporalAccessor value) {
148148

149149
auditable.setLastModifiedDate(
150-
getAsTemporalAccessor(Optional.of(value), type).orElseThrow(() -> new IllegalStateException()));
150+
getAsTemporalAccessor(Optional.of(value), type).orElseThrow(IllegalStateException::new));
151151

152152
return value;
153153
}
@@ -220,7 +220,7 @@ protected Object getDateValueToSet(TemporalAccessor value, Class<?> targetType,
220220
* @return
221221
*/
222222
@SuppressWarnings("unchecked")
223-
protected <T extends TemporalAccessor> Optional<T> getAsTemporalAccessor(Optional<? extends Object> source,
223+
protected <T extends TemporalAccessor> Optional<T> getAsTemporalAccessor(Optional<?> source,
224224
Class<? extends T> target) {
225225

226226
return source.map(it -> {

src/test/java/org/springframework/data/auditing/MappingAuditableBeanWrapperFactoryUnitTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ public void returnsLastModificationThreeTenBpDateTimeAsCalendar() {
163163
ThreeTenBackPortConverters.LocalDateTimeToJsr310LocalDateTimeConverter.INSTANCE.convert(reference));
164164
}
165165

166-
@Test
166+
@Test // DATACMNS-1109
167167
public void exposesInstantAsModificationDate() {
168168

169169
SampleWithInstant sample = new SampleWithInstant();

0 commit comments

Comments
 (0)