Spring/spring-data-jpa
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

DATAJPA-1061 - Polishing.

Browse Source 
Tweaked the formatting and the naming of variables.

Original pull request: #276.
This commit is contained in:
Jens Schauder
2019-06-24 14:00:44 +02:00
parent db39c48b9a
commit a4b3ca82ed
2 changed files with 6 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/main/java/org/springframework/data/jpa/repository/query/QueryUtils.java
View File

@@ -258,12 +258,12 @@ public abstract class QueryUtils {
builder.append(", ");
}
Set<String> aliases = getOuterJoinAliases(query);
Set<String> fieldAliases = getFunctionAliases(query);
fieldAliases.addAll(getFieldAliases(query));
Set<String> joinAliases = getOuterJoinAliases(query);
Set<String> selectionAliases = getFunctionAliases(query);
selectionAliases.addAll(getFieldAliases(query));
for (Order order : sort) {
builder.append(getOrderClause(aliases, fieldAliases, alias, order)).append(", ");
builder.append(getOrderClause(joinAliases, selectionAliases, alias, order)).append(", ");
}
builder.delete(builder.length() - 2, builder.length());
@@ -280,13 +280,13 @@ public abstract class QueryUtils {
* @param order the order object to build the clause for.
* @return
*/
private static String getOrderClause(Set<String> joinAliases, Set<String> fieldAlias, String alias, Order order) {
private static String getOrderClause(Set<String> joinAliases, Set<String> selectionAlias, String alias, Order order) {
String property = order.getProperty();
checkSortExpression(order);
if (fieldAlias.contains(property)) {
if (selectionAlias.contains(property)) {
return String.format("%s %s", property, toJpaDirection(order));
}

3
src/test/java/org/springframework/data/jpa/repository/query/QueryUtilsUnitTests.java
View File

@@ -427,7 +427,6 @@ public class QueryUtilsUnitTests {
String query = "SELECT m.price, lower(m.title) AS title, a.name as authorName FROM Magazine m INNER JOIN m.author a";
Sort sort = new Sort("authorName");
String fullQuery = applySorting(query, sort);
assertThat(fullQuery, endsWith("order by authorName asc"));
@@ -438,7 +437,6 @@ public class QueryUtilsUnitTests {
String query = "SELECT m.price, lower(m.title) AS title, a.name as authorName FROM Magazine m INNER JOIN m.author a";
Sort sort = new Sort("title");
String fullQuery = applySorting(query, sort);
assertThat(fullQuery, endsWith("order by title asc"));
@@ -449,7 +447,6 @@ public class QueryUtilsUnitTests {
String query = "SELECT m.price, lower(m.title) AS title, a.name as authorName FROM Magazine m INNER JOIN m.author a";
Sort sort = new Sort("price");
String fullQuery = applySorting(query, sort);
assertThat(fullQuery, endsWith("order by m.price asc"));