diff --git a/src/main/java/com/avaje/ebean/plugin/ExpressionPath.java b/src/main/java/com/avaje/ebean/plugin/ExpressionPath.java index 2d81119b0..a136aaead 100644 --- a/src/main/java/com/avaje/ebean/plugin/ExpressionPath.java +++ b/src/main/java/com/avaje/ebean/plugin/ExpressionPath.java @@ -67,10 +67,10 @@ public interface ExpressionPath { * Typically used to produce id = ? expression strings. *

*/ - String getAssocOneIdExpr(String propName, String bindOperator); + String getAssocIdExpression(String propName, String bindOperator); /** * Return the Id values for the given bean value. */ - Object[] getAssocOneIdValues(EntityBean bean); + Object[] getAssocIdValues(EntityBean bean); } diff --git a/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanFkeyProperty.java b/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanFkeyProperty.java index d2fe040d8..77ce914a7 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanFkeyProperty.java +++ b/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanFkeyProperty.java @@ -98,14 +98,14 @@ public final class BeanFkeyProperty implements ElPropertyValue { /** * Returns null as not an AssocOne. */ - public Object[] getAssocOneIdValues(EntityBean value) { + public Object[] getAssocIdValues(EntityBean value) { return null; } /** * Returns null as not an AssocOne. */ - public String getAssocOneIdExpr(String prefix, String operator) { + public String getAssocIdExpression(String prefix, String operator) { return null; } diff --git a/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanProperty.java b/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanProperty.java index 9e6229901..25a9fbf83 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanProperty.java +++ b/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanProperty.java @@ -834,12 +834,12 @@ public class BeanProperty implements ElPropertyValue, Property { return false; } - public Object[] getAssocOneIdValues(EntityBean bean) { + public Object[] getAssocIdValues(EntityBean bean) { // Returns null as not an AssocOne. return null; } - public String getAssocOneIdExpr(String prefix, String operator) { + public String getAssocIdExpression(String prefix, String operator) { // Returns null as not an AssocOne. return null; } diff --git a/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanPropertyAssocMany.java b/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanPropertyAssocMany.java index 02e302d18..b79bf0dcf 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanPropertyAssocMany.java +++ b/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanPropertyAssocMany.java @@ -485,14 +485,14 @@ public class BeanPropertyAssocMany extends BeanPropertyAssoc { * Return the Id values from the given bean. */ @Override - public Object[] getAssocOneIdValues(EntityBean bean) { + public Object[] getAssocIdValues(EntityBean bean) { return targetDescriptor.getIdBinder().getIdValues(bean); } /** * Return the Id expression to add to where clause etc. */ - public String getAssocOneIdExpr(String prefix, String operator) { + public String getAssocIdExpression(String prefix, String operator) { return targetDescriptor.getIdBinder().getAssocOneIdExpr(prefix, operator); } diff --git a/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanPropertyAssocOne.java b/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanPropertyAssocOne.java index 39ce11026..7c0f705e0 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanPropertyAssocOne.java +++ b/src/main/java/com/avaje/ebeaninternal/server/deploy/BeanPropertyAssocOne.java @@ -415,14 +415,14 @@ public class BeanPropertyAssocOne extends BeanPropertyAssoc { * Return the Id values from the given bean. */ @Override - public Object[] getAssocOneIdValues(EntityBean bean) { + public Object[] getAssocIdValues(EntityBean bean) { return targetDescriptor.getIdBinder().getIdValues(bean); } /** * Return the Id expression to add to where clause etc. */ - public String getAssocOneIdExpr(String prefix, String operator) { + public String getAssocIdExpression(String prefix, String operator) { return targetDescriptor.getIdBinder().getAssocOneIdExpr(prefix, operator); } diff --git a/src/main/java/com/avaje/ebeaninternal/server/el/ElPropertyChain.java b/src/main/java/com/avaje/ebeaninternal/server/el/ElPropertyChain.java index 024778233..7631eab47 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/el/ElPropertyChain.java +++ b/src/main/java/com/avaje/ebeaninternal/server/el/ElPropertyChain.java @@ -154,14 +154,14 @@ public class ElPropertyChain implements ElPropertyValue { return lastElPropertyValue.isLocalEncrypted(); } - public Object[] getAssocOneIdValues(EntityBean bean) { + public Object[] getAssocIdValues(EntityBean bean) { // Don't navigate the object graph as bean // is assumed to be the appropriate type - return lastElPropertyValue.getAssocOneIdValues(bean); + return lastElPropertyValue.getAssocIdValues(bean); } - public String getAssocOneIdExpr(String prefix, String operator) { - return lastElPropertyValue.getAssocOneIdExpr(expression, operator); + public String getAssocIdExpression(String prefix, String operator) { + return lastElPropertyValue.getAssocIdExpression(expression, operator); } public String getAssocIdInExpr(String prefix) { diff --git a/src/main/java/com/avaje/ebeaninternal/server/expression/InExpression.java b/src/main/java/com/avaje/ebeaninternal/server/expression/InExpression.java index ebd31b768..b9ae17902 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/expression/InExpression.java +++ b/src/main/java/com/avaje/ebeaninternal/server/expression/InExpression.java @@ -46,7 +46,7 @@ class InExpression extends AbstractExpression { } else { // extract the id values from the bean - Object[] ids = prop.getAssocOneIdValues((EntityBean) values[i]); + Object[] ids = prop.getAssocIdValues((EntityBean) values[i]); if (ids != null) { for (int j = 0; j < ids.length; j++) { request.addBindValue(ids[j]); diff --git a/src/main/java/com/avaje/ebeaninternal/server/expression/NullExpression.java b/src/main/java/com/avaje/ebeaninternal/server/expression/NullExpression.java index 213b59b3f..87d593c5d 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/expression/NullExpression.java +++ b/src/main/java/com/avaje/ebeaninternal/server/expression/NullExpression.java @@ -39,7 +39,7 @@ class NullExpression extends AbstractExpression { ElPropertyValue prop = getElProp(request); if (prop != null && prop.isAssocId()) { - request.append(prop.getAssocOneIdExpr(propertyName, nullExpr)); + request.append(prop.getAssocIdExpression(propertyName, nullExpr)); return; } diff --git a/src/main/java/com/avaje/ebeaninternal/server/expression/SimpleExpression.java b/src/main/java/com/avaje/ebeaninternal/server/expression/SimpleExpression.java index f373c0390..7cfe71ec6 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/expression/SimpleExpression.java +++ b/src/main/java/com/avaje/ebeaninternal/server/expression/SimpleExpression.java @@ -28,8 +28,8 @@ public class SimpleExpression extends AbstractExpression { } ExpressionPath prop = context.getExpressionPath(propName); if (prop != null && prop.isAssocId()) { - String idName = prop.getAssocOneIdExpr(propName, ""); - Object[] ids = prop.getAssocOneIdValues((EntityBean) value); + String idName = prop.getAssocIdExpression(propName, ""); + Object[] ids = prop.getAssocIdValues((EntityBean) value); if (ids == null || ids.length != 1) { throw new IllegalArgumentException("Expecting 1 Id value for " + idName + " but got " + ids); } @@ -57,7 +57,7 @@ public class SimpleExpression extends AbstractExpression { ElPropertyValue prop = getElProp(request); if (prop != null) { if (prop.isAssocId()) { - Object[] ids = prop.getAssocOneIdValues((EntityBean) value); + Object[] ids = prop.getAssocIdValues((EntityBean) value); if (ids != null) { for (int i = 0; i < ids.length; i++) { request.addBindValue(ids[i]); @@ -84,7 +84,7 @@ public class SimpleExpression extends AbstractExpression { ElPropertyValue prop = getElProp(request); if (prop != null) { if (prop.isAssocId()) { - request.append(prop.getAssocOneIdExpr(propName, type.bind())); + request.append(prop.getAssocIdExpression(propName, type.bind())); return; } if (prop.isDbEncrypted()) { diff --git a/src/main/java/com/avaje/ebeaninternal/server/type/CtCompoundPropertyElAdapter.java b/src/main/java/com/avaje/ebeaninternal/server/type/CtCompoundPropertyElAdapter.java index 3ccf8030f..4f12100ba 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/type/CtCompoundPropertyElAdapter.java +++ b/src/main/java/com/avaje/ebeaninternal/server/type/CtCompoundPropertyElAdapter.java @@ -48,11 +48,11 @@ public class CtCompoundPropertyElAdapter implements ElPropertyValue { prop.setValue(bean, value); } - public String getAssocOneIdExpr(String prefix, String operator) { + public String getAssocIdExpression(String prefix, String operator) { throw new RuntimeException("Not Supported or Expected"); } - public Object[] getAssocOneIdValues(EntityBean bean) { + public Object[] getAssocIdValues(EntityBean bean) { throw new RuntimeException("Not Supported or Expected"); }