diff --git a/src/main/java/com/avaje/ebeaninternal/server/querydefn/DefaultOrmQuery.java b/src/main/java/com/avaje/ebeaninternal/server/querydefn/DefaultOrmQuery.java index 83c6b4f1e..71ec91532 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/querydefn/DefaultOrmQuery.java +++ b/src/main/java/com/avaje/ebeaninternal/server/querydefn/DefaultOrmQuery.java @@ -344,20 +344,6 @@ public class DefaultOrmQuery implements SpiQuery { this.beanDescriptor = beanDescriptor; } - /** - * Return true if select all properties was used to ensure the property invoking a lazy load was - * included in the query. - */ - public boolean selectAllForLazyLoadProperty() { - if (lazyLoadProperty != null) { - if (!detail.containsProperty(lazyLoadProperty)) { - detail.select("*"); - return true; - } - } - return false; - } - public RawSql getRawSql() { return rawSql; } @@ -404,6 +390,20 @@ public class DefaultOrmQuery implements SpiQuery { return manyWhereJoins; } + /** + * Return true if select all properties was used to ensure the property invoking a lazy load was + * included in the query. + */ + public boolean selectAllForLazyLoadProperty() { + if (lazyLoadProperty != null) { + if (!detail.containsProperty(lazyLoadProperty)) { + detail.select("*"); + return true; + } + } + return false; + } + public List removeQueryJoins() { List queryJoins = detail.removeSecondaryQueries(); if (queryJoins != null) { @@ -447,6 +447,39 @@ public class DefaultOrmQuery implements SpiQuery { detail.convertManyFetchJoinsToQueryJoins(beanDescriptor, lazyLoadManyPath, allowOne, queryBatch); } + protected void setOrmQueryDetail(OrmQueryDetail detail) { + this.detail = detail; + } + + public void setDefaultSelectClause() { + detail.setDefaultSelectClause(beanDescriptor); + } + + public void setDetail(OrmQueryDetail detail) { + this.detail = detail; + } + + public boolean tuneFetchProperties(OrmQueryDetail tunedDetail) { + return detail.tuneFetchProperties(tunedDetail); + } + + public OrmQueryDetail getDetail() { + return detail; + } + + public ExpressionList filterMany(String prop) { + + OrmQueryProperties chunk = detail.getChunk(prop, true); + return chunk.filterMany(this); + } + + public void setFilterMany(String prop, ExpressionList filterMany) { + if (filterMany != null) { + OrmQueryProperties chunk = detail.getChunk(prop, true); + chunk.setFilterMany((SpiExpressionList) filterMany); + } + } + /** * Setup to be a delete query. */ @@ -465,7 +498,6 @@ public class DefaultOrmQuery implements SpiQuery { public void setSelectId() { // clear select and fetch joins.. detail.clear(); - select(beanDescriptor.getIdBinder().getIdProperty()); } @@ -932,18 +964,11 @@ public class DefaultOrmQuery implements SpiQuery { parser.assign(this); } - protected void setOrmQueryDetail(OrmQueryDetail detail) { - this.detail = detail; - } protected void setRawWhereClause(String rawWhereClause) { this.rawWhereClause = rawWhereClause; } - public void setDefaultSelectClause() { - detail.setDefaultSelectClause(beanDescriptor); - } - public DefaultOrmQuery select(String columns) { detail.select(columns); return this; @@ -1165,18 +1190,6 @@ public class DefaultOrmQuery implements SpiQuery { return beanType; } - public void setDetail(OrmQueryDetail detail) { - this.detail = detail; - } - - public boolean tuneFetchProperties(OrmQueryDetail tunedDetail) { - return detail.tuneFetchProperties(tunedDetail); - } - - public OrmQueryDetail getDetail() { - return detail; - } - public String toString() { return "Query [" + whereExpressions + "]"; } @@ -1270,19 +1283,6 @@ public class DefaultOrmQuery implements SpiQuery { return whereExpressions; } - public ExpressionList filterMany(String prop) { - - OrmQueryProperties chunk = detail.getChunk(prop, true); - return chunk.filterMany(this); - } - - public void setFilterMany(String prop, ExpressionList filterMany) { - if (filterMany != null) { - OrmQueryProperties chunk = detail.getChunk(prop, true); - chunk.setFilterMany((SpiExpressionList) filterMany); - } - } - public DefaultOrmQuery having(String addToHavingClause) { if (additionalHaving == null) { additionalHaving = addToHavingClause;