Skip to content

Commit ac88f1d

Browse files
committed
DATAJDBC-530 - Polishing.
Original pull request: #210.
1 parent f9bc40c commit ac88f1d

File tree

1 file changed

+10
-8
lines changed
  • spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert

1 file changed

+10
-8
lines changed

spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/convert/SqlGenerator.java

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -659,17 +659,19 @@ private String renderReference(SqlIdentifier identifier) {
659659
}
660660

661661
private List<OrderByField> extractOrderByFields(Sort sort) {
662-
return sort.stream()
663-
.map(order -> {
664-
Column column = Column.create(
665-
this.entity.getRequiredPersistentProperty(order.getProperty()).getColumnName(),
666-
this.getTable()
667-
);
668-
return OrderByField.from(column, order.getDirection());
669-
})
662+
663+
return sort.stream() //
664+
.map(this::orderToOrderByField) //
670665
.collect(Collectors.toList());
671666
}
672667

668+
private OrderByField orderToOrderByField(Sort.Order order) {
669+
670+
SqlIdentifier columnName = this.entity.getRequiredPersistentProperty(order.getProperty()).getColumnName();
671+
Column column = Column.create(columnName, this.getTable());
672+
return OrderByField.from(column, order.getDirection());
673+
}
674+
673675
/**
674676
* Value object representing a {@code JOIN} association.
675677
*/

0 commit comments

Comments
 (0)