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

DATAJPA-1404 - Polishing.

Browse Source 
Simplify if-empty comparison. Fix ticket references.

Original pull request: #304.
This commit is contained in:
Mark Paluch
2019-02-28 10:21:01 +01:00
parent 877db7477c
commit 3e95d64150
2 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -614,7 +614,7 @@ public abstract class QueryUtils {
* @param isPluralAttribute is the attribute of Collection type?
* @param isLeafProperty is this the final property navigated by a {@link PropertyPath}?
* @param isForSelection is the property navigated for the selection part of the query?
* @return wether an outer join is to be used for integrating this attribute in a query.
* @return whether an outer join is to be used for integrating this attribute in a query.
*/
private static boolean requiresOuterJoin(@Nullable Bindable<?> propertyPathModel, boolean isPluralAttribute,
boolean isLeafProperty, boolean isForSelection) {
@@ -637,7 +637,7 @@ public abstract class QueryUtils {
// outer join to avoid https://hibernate.atlassian.net/browse/HHH-12712 and
// https://github.com/eclipse-ee4j/jpa-api/issues/170
boolean isInverseOptionalOneToOne = PersistentAttributeType.ONE_TO_ONE == attribute.getPersistentAttributeType()
&& !getAnnotationProperty(attribute, "mappedBy", "").isEmpty();
&& StringUtils.hasText(getAnnotationProperty(attribute, "mappedBy", ""));
// if this path is part of the select list we need to generate an explicit outer join in order to prevent Hibernate
// to use an inner join instead.
@@ -650,6 +650,7 @@ public abstract class QueryUtils {
}
private static <T> T getAnnotationProperty(Attribute<?, ?> attribute, String propertyName, T defaultValue) {
Class<? extends Annotation> associationAnnotation = ASSOCIATION_TYPES.get(attribute.getPersistentAttributeType());
if (associationAnnotation == null) {

4
src/test/java/org/springframework/data/jpa/repository/query/QueryUtilsIntegrationTests.java
View File

@@ -95,7 +95,7 @@ public class QueryUtilsIntegrationTests {
assertThat(getNonInnerJoins(root)).hasSize(1);
}
@Test // DATAJPA-1238
@Test // DATAJPA-1404
public void createsJoinForOptionalOneToOneInReverseDirection() {
doInMerchantContext(emf -> {
@@ -110,7 +110,7 @@ public class QueryUtilsIntegrationTests {
});
}
@Test // DATAJPA-1238
@Test // DATAJPA-1404
public void createsNoJoinForOptionalOneToOneInNormalDirection() {
doInMerchantContext(emf -> {