diff --git a/ebean-core/src/main/java/io/ebeaninternal/server/querydefn/OrmQueryDetail.java b/ebean-core/src/main/java/io/ebeaninternal/server/querydefn/OrmQueryDetail.java index ba03b7c78..6f96fbbea 100644 --- a/ebean-core/src/main/java/io/ebeaninternal/server/querydefn/OrmQueryDetail.java +++ b/ebean-core/src/main/java/io/ebeaninternal/server/querydefn/OrmQueryDetail.java @@ -111,7 +111,7 @@ public class OrmQueryDetail implements Serializable { */ public String asStringDebug() { StringBuilder sb = new StringBuilder(); - if (baseProps.notEmpty()) { + if (baseProps.hasProperties()) { baseProps.asStringDebug("select ", sb); } if (fetchPaths != null) { diff --git a/ebean-core/src/main/java/io/ebeaninternal/server/querydefn/OrmQueryProperties.java b/ebean-core/src/main/java/io/ebeaninternal/server/querydefn/OrmQueryProperties.java index eb41bb40f..d1a3aeafc 100644 --- a/ebean-core/src/main/java/io/ebeaninternal/server/querydefn/OrmQueryProperties.java +++ b/ebean-core/src/main/java/io/ebeaninternal/server/querydefn/OrmQueryProperties.java @@ -214,7 +214,7 @@ public class OrmQueryProperties implements Serializable { */ @SuppressWarnings("unchecked") public void configureBeanQuery(SpiQuery query) { - if (notEmpty()) { + if (hasProperties()) { query.selectProperties(this); } @@ -242,17 +242,13 @@ public class OrmQueryProperties implements Serializable { } public boolean hasSelectClause() { - if (allProperties) { - return true; - } - // explicitly selected some properties - return included != null || filterMany != null; + return allProperties || included != null || filterMany != null; } /** - * Return true if the properties and configuration are empty. + * Return true if explicit properties have been specified. */ - boolean notEmpty() { + boolean hasProperties() { return allProperties || included != null; }