Skip to content

Commit 9f7990d

Browse files
committed
#30 - Polishing.
Minor formatting.
1 parent 384f3b4 commit 9f7990d

File tree

3 files changed

+6
-1
lines changed

3 files changed

+6
-1
lines changed

src/main/java/org/springframework/data/r2dbc/config/AbstractR2dbcConfiguration.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ public RelationalMappingContext r2dbcMappingContext(Optional<NamingStrategy> nam
118118
}
119119

120120
/**
121-
* Creates a {@link ReactiveDataAccessStrategy} using the configured {@link #r2dbcMappingContext(Optional)
121+
* Creates a {@link ReactiveDataAccessStrategy} using the configured {@link #r2dbcMappingContext(Optional, R2dbcCustomConversions)}
122122
* RelationalMappingContext}.
123123
*
124124
* @param mappingContext the configured {@link RelationalMappingContext}.

src/main/java/org/springframework/data/r2dbc/function/DefaultReactiveDataAccessStrategy.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -261,11 +261,13 @@ private Object getWriteValue(PersistentPropertyAccessor propertyAccessor, Relati
261261
}
262262

263263
if (!dialect.supportsArrayColumns()) {
264+
264265
throw new InvalidDataAccessResourceUsageException(
265266
"Dialect " + dialect.getClass().getName() + " does not support array columns");
266267
}
267268

268269
if (!property.isArray()) {
270+
269271
Object zeroLengthArray = Array.newInstance(property.getActualType(), 0);
270272
return relationalConverter.getConversionService().convert(value, zeroLengthArray.getClass());
271273
}

src/test/java/org/springframework/data/r2dbc/function/DefaultReactiveDataAccessStrategyUnitTests.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,7 @@ public void shouldUpdateArray() {
112112
.getColumnsToUpdate(new WithCollectionTypes(new String[] { "one", "two" }, null));
113113

114114
Object stringArray = columnsToUpdate.get("string_array").getValue();
115+
115116
assertThat(stringArray).isInstanceOf(String[].class);
116117
assertThat((String[]) stringArray).hasSize(2).contains("one", "two");
117118
}
@@ -123,6 +124,7 @@ public void shouldConvertListToArray() {
123124
.getColumnsToUpdate(new WithCollectionTypes(null, Arrays.asList("one", "two")));
124125

125126
Object stringArray = columnsToUpdate.get("string_collection").getValue();
127+
126128
assertThat(stringArray).isInstanceOf(String[].class);
127129
assertThat((String[]) stringArray).hasSize(2).contains("one", "two");
128130
}
@@ -134,6 +136,7 @@ static class WithCollectionTypes {
134136
List<String> stringCollection;
135137

136138
WithCollectionTypes(String[] stringArray, List<String> stringCollection) {
139+
137140
this.stringArray = stringArray;
138141
this.stringCollection = stringCollection;
139142
}

0 commit comments

Comments
 (0)